Skip to content

Add GNU target platform support for configure argument #280

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

etienne02
Copy link

@etienne02 etienne02 commented Mar 18, 2025

GNU is supported for auto-configuration but was not when defining the platform with --target. This is now supported.

And fix the name of the shared library to follow the same as linux and bsd

GNU is supported for auto-configuration but was not when defining the
platform with --target. This is now supported.
@rhash
Copy link
Owner

rhash commented Mar 18, 2025

This commit supports triplets for the GNU/Hurd OS, such as:

  1. i686-pc-gnu (pc: Generic vendor)
  2. x86_64-pc-gnu
  3. i586-pc-gnu
  4. i686-unknown-gnu (Vendor is unspecified or unknown)
  5. x86_64-unknown-gnu
  6. i686-gnu
  7. x86_64-gnu

rhash pushed a commit that referenced this pull request Mar 18, 2025
Add support for GNU-based target platform. Update the configure script
to recognize --target triplets containing the "gnu" part. This also fixes
the shared library naming convention to match those used for Linux and BSD.
@rhash
Copy link
Owner

rhash commented Mar 18, 2025

@etienne02 , since you are still fixing the script, I'll wait for the final, tested version. :)

@etienne02
Copy link
Author

@etienne02 , since you are still fixing the script, I'll wait for the final, tested version. :)

It’s working now. You can test it.

@rhash
Copy link
Owner

rhash commented Mar 21, 2025

@etienne02 thank you for the improvement!

PR has been merged as 5dd4665, faa052a.

@rhash rhash closed this Mar 21, 2025
@rhash rhash added the done The issue is fixed/applied/implemented label Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done The issue is fixed/applied/implemented enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants