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
Update build phpPackages #75033
Update build phpPackages #75033
Conversation
8c56d88
to
a076b0b
Compare
6b82db2
to
b320bda
Compare
b320bda
to
5a7155d
Compare
For which package(s) does |
Only removed this warning. How this error affects the building I do not know. |
@Izorkin Sorry, but I don't see the point of this PR if it actually doesn't fix any real error. If we have a package that is broken due to this, it's worth doing. But this touch so many places to get rid of something that seems to be harmless... |
@Izorkin Since that commit only would add file as an additional dependency to the closure but not actually resolve any issue (that we know of)... I don't see the point of merging it. |
I agree with @etu that we should be solving a known problem when we increase closure size, not appeasing a warning which has no known issue. I would also point out that many packages mention a missing |
Motivation for this change
Fixed error:
./configure: line 5836: /usr/bin/file: No such file or directory
Clean-ip build configuration.
cc @etu @aanderse
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @