Skip to content
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

sbot: init at 9d686ee70c6b2073da626fa08e38a8e988b5c5b4 #21809

Closed
wants to merge 2 commits into from

Conversation

matthiasbeyer
Copy link
Contributor

Motivation for this change

Package sbot

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

I need help here. That's actually the first application from the node-js ecosystem I'm trying to package.


np.buildNodePackage rec {
version = "9.4.2";
name = "sbot-${version}";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple things,

  1. you're actually doing init at 9.4.2 -- can you update the commit message?
  2. can you dedent version back to the normal level here?

@matthiasbeyer
Copy link
Contributor Author

@grahamc fixed what you mentioned, it still does not build though, and I do not know how to resolve this error:

error: anonymous function at /home/m/archive/development/contrib/nixpkgs/pkgs/development/node-packages/default-v6.nix:1:1 called with unexpected argument ‘generated’, at /home/m/archive/development/contrib/nixpkgs/lib/customisation.nix:56:12

Copy link
Member

@fpletz fpletz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please merge the generated dependes with ./pkgs/top-level/node-packages-generated.nix if possible.

Adding more than 12k lines for a single package that isn't very popular is not acceptable.

@matthiasbeyer
Copy link
Contributor Author

@fpletz I would love to - is there a tool available for this or do I have to do it by hand?

@Mic92
Copy link
Member

Mic92 commented Jan 15, 2017

@matthiasbeyer add it to pkgs/development/node-packages/node-packages.json and run generate.sh in the same directory.

@matthiasbeyer
Copy link
Contributor Author

@Mic92 umm... I'm not sure what you mean? Add the .nix file to the .json file? Sorry if I'm dumb...

@Mic92
Copy link
Member

Mic92 commented Jan 16, 2017

Add your package to node-packages.json and then regenerate node-packages-v4.nix and node-packages-v6.nix with the script generate.sh

@matthiasbeyer
Copy link
Contributor Author

@Mic92 Hope that is what you suggested...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants