kicad: include desktop, icon and mime files#119055
kicad: include desktop, icon and mime files#119055SuperSandro2000 merged 2 commits intoNixOS:masterfrom eduardosm:kicad
Conversation
ofborg
bot
added
10.rebuild-darwin: 1-10
There was a problem hiding this comment.
| install -d $out/share | |
| mkdir -p $out/share |
|
Result of 2 packages built successfully:
1 suggestion:
|
evils
left a comment
evils
left a comment
There was a problem hiding this comment.
sorry i didn't get around to addressing your issue
| in | ||
| (concatStringsSep "\n" | ||
| (flatten [ | ||
| "runHook preInstall" |
There was a problem hiding this comment.
Hooks need to be explicitly called by the installPhase script. Even if there isn't a preInstall hook, users may define their own using overrideAttrs.
https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md
| # link in the CLI utils | ||
| (map (util: "ln -s ${base}/bin/${util} $out/bin/${util}") utils) | ||
|
|
||
| "runHook postInstall" |
There was a problem hiding this comment.
any idea why this has to be added explicitly?
There was a problem hiding this comment.
Because you are overwriting the phase that normally runs those.
There was a problem hiding this comment.
LGTM
@evils LGTY? nvm, should have read the above more closely.
|
@ofborg eval |
4 participants
Motivation for this change
Fixes #106295.
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)