kdePackages.*: disable strictDeps#329822
kdePackages.*: disable strictDeps#329822paparodeo wants to merge 1 commit intoNixOS:stagingfrom paparodeo:disable-kdePackages-strictDeps
Conversation
cmake strictDeps are broken and searches for binaries in both buildInputs and PATH. strictDeps will eventually be fixed via PR #318226 or a derivative of and when that happens many of the 400 packages in the kdePackages attribute set will break due to binaries missing from nativeBuildInputs (but in the buildInputs)
|
I don't mind this as a solution, but also, things really shouldn't break here really, and most if not all the inputs should be in their right places already. I can try rebuilding kdePackages with your patch applied, or we can do it in a staging cycle. |
there are 400 packages and while not all of them break may of them do. i ran 112 of the packages. 41 passed, 71 failed and then i just skipped all of the rest of the kde packages. [edit] though, looking at the logs 70 of them were failing due to Could NOT find Gettext (missing: GETTEXT_MSGMERGE_EXECUTABLE but there could be other failures beyond that one. |
|
That's very surprising. Were those individual failures or cascading from a single one? Either way, I can look into it later today. |
so i think i just built one package at a time and i was overrideing cmake to cmake2 (which was the one with strictDeps fixed) so it shouldn't have been a cascade of failures. |
|
yeah, i hacked one kde package to get access to |
|
I've just applied your patch on top of master and am building it now. I'll see what the failures are. |
|
I'm only building |
|
here is the failed logs tarball |
|
Fixed everything: #329977 |
Description of changes
cmake strictDeps are broken and searches for binaries in both buildInputs and PATH. strictDeps will eventually be fixed via PR #318226 or a derivative of and when that happens many of the 400 packages in the kdePackages attribute set will break due to binaries missing from nativeBuildInputs (but in the buildInputs)
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.