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
php.extensions.rdkafka: init at 4.0.3 #86492
Conversation
6d33861
to
5df9d34
Compare
5df9d34
to
a13be9f
Compare
@ofborg build php74.extensions.rdkafka php73.extensions.rdkafka php72.extensions.rdkafka |
|
||
postPhpize = '' | ||
substituteInPlace configure \ | ||
--replace 'SEARCH_PATH="/usr/local /usr"' 'SEARCH_PATH=${pkgs.rdkafka}' |
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.
Maybe a regex that looks for a line starting with SEARCH_PATH would be more robust.
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.
It's also more complex to read, they have had the same SEARCH_PATH
as a result from 3.0.3 to 4.0.3, so it doesn't seem to change that much.
Also, if it would fail to build. It's easy to spot that during the package update.
a13be9f
to
e683f5c
Compare
@ofborg build php.extensions.rdkafka |
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.
Builds and loads fine and @alebon confirmed that it worked for them.
Motivation for this change
Added by request from @alebon. This fixes #86463
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)