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
pugixml: 1.9 -> 1.10 #102926
pugixml: 1.9 -> 1.10 #102926
Conversation
* split to multiple outputs when using the shared library version * enabled the tests * validate the pkg-config files
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.
LGTM
failures are broken on target branch
https://github.com/NixOS/nixpkgs/pull/102926
1 package marked as broken and skipped:
photoflow
1 package failed to build:
appleseed
13 packages built:
PlistCpp darktable filezilla gerbera gnss-sdr kodi-wayland kodiPlainWayland mudlet osl poedit pugixml waylandpp xib2nib
export DYLD_LIBRARY_PATH="`pwd`''${DYLD_LIBRARY_PATH:+:}$DYLD_LIBRARY_PATH" | ||
'' else '' | ||
export LD_LIBRARY_PATH="`pwd`''${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH" |
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.
export DYLD_LIBRARY_PATH="`pwd`''${DYLD_LIBRARY_PATH:+:}$DYLD_LIBRARY_PATH" | |
'' else '' | |
export LD_LIBRARY_PATH="`pwd`''${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH" | |
export DYLD_LIBRARY_PATH="$(pwd)''${DYLD_LIBRARY_PATH:+:}$DYLD_LIBRARY_PATH" | |
'' else '' | |
export LD_LIBRARY_PATH="$(pwd)''${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH" |
I have a doubt, do i open another pr with the suggestion? |
go for it |
cmakeFlags = [ "-DBUILD_SHARED_LIBS=${if shared then "ON" else "OFF"}" ]; | ||
# Hack to be able to run the test, broken because we use | ||
# CMAKE_SKIP_BUILD_RPATH to avoid cmake resetting rpath on install | ||
preBuild = if stdenv.isDarwin then '' |
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.
if this is specific for tests, please do preCheck
to avoid this affecting the build or install
Result of 1 package marked as broken and skipped:
5 packages built:
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)