-
-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
fluidsynth: Fix CMake config #242657
fluidsynth: Fix CMake config #242657
Conversation
Would this supersede #242635? |
Yes. |
Upstream is concatenating CMAKE_INSTALL_LIBDIR onto CMAKE_INSTALL_PREFIX to get a directory to use for the install_name on Darwin. This is not the right way of making CMAKE_INSTALL_LIBDIR absolute. The config was first set up for breakage when the CMAKE_INSTALL_LIBDIR was made relative, in order to work around the wrong concatenation. This caused the _IMPORT_PREFIX to be computed by walking up the directory tree from the config file, instead of using the absolute CMAKE_INSTALL_LIBDIR. Then outputs were introduced to the derivation and the config was moved to a different output. Now the prefix it walks up to is the wrong prefix, and the config is busted. Fix by reverting the original workaround so we have an absolute CMAKE_INSTALL_LIBDIR, and pulling a patch that fixes the bad concatenation.
f427432
to
f43a46f
Compare
|
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.
Thank you!
Successfully created backport PR for |
Description of changes
Closes #242623
Upstream is concatenating
CMAKE_INSTALL_LIBDIR
ontoCMAKE_INSTALL_PREFIX
to get a directory to use for the install_name on Darwin. This is not the right way of makingCMAKE_INSTALL_LIBDIR
absolute.The config was first set up for breakage when the
CMAKE_INSTALL_LIBDIR
was made relative, in order to work around the wrong concatenation. This caused the_IMPORT_PREFIX
to be computed by walking up the directory tree from the config file, instead of using the absoluteCMAKE_INSTALL_PREFIX
.Then outputs were introduced to the derivation and the config was moved to a different output. Now the prefix it walks up to is the wrong prefix, and the config is busted.
Fix by reverting the original workaround so we have an absolute
CMAKE_INSTALL_LIBDIR
, and pulling a patch that fixes the bad concatenation.Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)