-
-
Notifications
You must be signed in to change notification settings - Fork 182
Add dynamically linked musl distributions #541
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
Merged
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
9663760
Dynamically link libc in musl distributions
zanieb 18ff4fc
Fix dynlib loading
zanieb f156f44
Update musl dist metadata
zanieb c78bb9b
Update validation to support dynamic musl
zanieb d590f87
Update comment
zanieb 51a5e29
Fix verify test for dynamic musl
zanieb 7b2b0f2
Add path context to test run failure
zanieb 2cef305
Format
zanieb a173de8
Install musl-dev for validation
zanieb 94cadca
Fix musl build mode loading
zanieb 7b8ff31
Downgrade to musl 1.2.2
zanieb 280f345
Debug `LD_LIBRARY_PATH`
zanieb 902fcd0
Add comment
zanieb b60e292
Refactor static / dynamic linking into build options
zanieb 28d5d6e
Merge branch 'zb/ref-static' into zb/musl-dynamic
zanieb 01172ad
Add dynamic CI targets
zanieb 75ff237
Use the musl version from downloads in the metadata
zanieb d05620a
Remove the `+shared` build option; enable implicitly
zanieb 567768d
Remove commented line
zanieb fd04e37
Apply `_blake2` musl fixes when doing non-static builds
zanieb 88b335b
Split the musl versions for static and shared buildsl
zanieb 6b6f103
Properly toggle `--enable-shared` when building the musl toolchain
zanieb 6f88ed4
Fix syntax error
zanieb c0b9ce9
Set `-x` in the musl build
zanieb 44c1959
Only enable `fPIC` during shared musl builds
zanieb 4385eb8
Trim some extraneous whitespace
zanieb 1990014
Change the triple during static musl tcl builds to get past duplicate…
zanieb 7153487
Format
zanieb 4c685db
Fix typo
zanieb 4670133
Fix missing variable
zanieb c4e081d
Fix configuration of tcl build
zanieb c410304
Update validation
zanieb a745841
Remove extra "
zanieb 6a79ece
Clean up ELF validation logic
zanieb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably a little overzealous including all these here, but of no real consequence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming we were previously implicitly doing this for musl?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. We were normalizing the triple to gnu beforehand.