Open
Conversation
added 6 commits
July 29, 2025 20:01
Previously, calling cmake with -DCMAKE_INSTALL_PREFIX=/my/path wouldn't be honored.
Modern Unix systems bake the library paths in the ELF binaries directly via runpaths, avoiding the need to manually set LD_LIBRARY_PATH. This simplifies the problem and avoid the brittleness of running potentially non-portable commands in a Shell script wrapper. Take the opportunity to reuse Linux.cmake for every supported platform but Windows and MacOS.
Specify the CMake targets to be installed directly.
A new INSTALL_C_SOURCE option is added guarding their installation, OFF by default.
a872be8 to
2adf905
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This work was made while adding a
pharo-vmpackage to GNU Guix. With it, packaging should become trivial for most GNU/Linux distributions. The installation layout looks like: