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
nodePackages.purescript-psa: init at 0.7.3 #74727
nodePackages.purescript-psa: init at 0.7.3 #74727
Conversation
@GrahamcOfBorg build nodePackages.purescript-psa |
@@ -2,7 +2,7 @@ | |||
|
|||
{pkgs ? import <nixpkgs> { | |||
inherit system; | |||
}, system ? builtins.currentSystem, nodejs ? pkgs."nodejs-13_x"}: | |||
}, system ? builtins.currentSystem, nodejs ? pkgs."nodejs-8_x"}: |
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.
I'm quite worried about this change. It seems strange that generate.sh
would replace pkgs.nodejs-13_x
with pkgs.nodejs-8_x
in this file (since it is composition-v13.nix).
I can remove this change from this PR if necessary.
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.
this seems like unwanted change to me as well
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.
Yeah, I guess the real question is, is this a problem in node2nix
(and should be reported upstream), or is this a problem in the generate.sh
script (and a new issue opened here on the nixpkgs repo).
@@ -363,7 +363,7 @@ let | |||
|
|||
npm ${forceOfflineFlag} --nodedir=${nodeSources} ${npmFlags} ${stdenv.lib.optionalString production "--production"} rebuild | |||
|
|||
if [ "''${dontNpmInstall-}" != "1" ] |
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.
This reverts treewide patch: 2811b03
node2nix should be patched to include that change to avoid generating that diff on each run.
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.
This has been reported upstream in svanderburg/node2nix#162.
|
bf6ba62
to
2b63a42
Compare
@GrahamcOfBorg build purescript-psa pulp |
I never ended up getting a review of this, so I fixed it up as seemed appropriate. If this ends up causing errors in some other package, let's revert it. |
|
Motivation for this change
Add the
purescript-psa
package tonodePackages
.This was packaged live during Nix Fridays with @zimbatm.
We followed the steps in the nixpkgs manual (adding
purescript-psa
topkgs/development/node-packages/node-packages-v10.json
and then runninggenerate.sh
).However, a bunch of files were modified, including
pkgs/development/node-packages/node-packages-v12.nix
,pkgs/development/node-packages/node-packages-v13.nix
,pkgs/development/node-packages/composition-v13.nix
, andpkgs/development/node-packages/node-env.nix
. I committed them all (since that is what the instructions say to do), but I imagine that might not be the correct thing to do.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 @