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

nodePackages.{triton,manta}: repackage using buildNpmPackage #230158

Merged
merged 3 commits into from
May 6, 2023

Conversation

teutat3s
Copy link
Member

@teutat3s teutat3s commented May 5, 2023

Description of changes
  • convert nodePackages.triton to buildNpmPackage
  • convert nodePackages.manta to buildNpmPackage
  • add version tests for triton and manta
  • renamed nodePackages.manta to node-manta, because manta is already taken

If desired, can be split up into two separate PRs.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@teutat3s teutat3s changed the title Triton buildnpmpackage nodePackages.{triton,manta}: convert to buildNpmPackage May 5, 2023
@teutat3s
Copy link
Member Author

teutat3s commented May 5, 2023

Related #229475

@teutat3s teutat3s changed the title nodePackages.{triton,manta}: convert to buildNpmPackage nodePackages.{triton,manta}: repackage using buildNpmPackage May 5, 2023
@teutat3s teutat3s requested a review from winterqt May 5, 2023 18:23
@winterqt winterqt changed the title nodePackages.{triton,manta}: repackage using buildNpmPackage nodePackages.{triton,manta}: repackage using buildNpmPackage May 5, 2023
pkgs/tools/admin/manta/default.nix Outdated Show resolved Hide resolved
pkgs/tools/admin/manta/default.nix Outdated Show resolved Hide resolved
@ofborg ofborg bot added 8.has: package (new) This PR adds a new package 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 labels May 5, 2023
Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

Please rebase and add an alias to the new aliases file inside of node-packages.

pkgs/tools/admin/triton/default.nix Outdated Show resolved Hide resolved
pkgs/tools/admin/triton/default.nix Outdated Show resolved Hide resolved
pkgs/tools/admin/manta/default.nix Outdated Show resolved Hide resolved
pkgs/tools/admin/manta/default.nix Outdated Show resolved Hide resolved
pkgs/tools/admin/manta/default.nix Outdated Show resolved Hide resolved
@SuperSandro2000
Copy link
Member

@ofborg build triton node-manta

@SuperSandro2000 SuperSandro2000 merged commit 2b8f115 into NixOS:master May 6, 2023
@teutat3s teutat3s deleted the triton-buildnpmpackage branch May 8, 2023 06:51
@@ -37,4 +37,5 @@ mapAliases {
"@antora/cli" = pkgs.antora;
"@githubnext/github-copilot-cli" = pkgs.github-copilot-cli; # Added 2023-05-02
"@nestjs/cli" = pkgs.nest-cli;
trito = pkgs.triton; # Added 2023-05-06
Copy link
Member Author

Choose a reason for hiding this comment

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

I missed this typo: trito -> triton

Copy link
Member

Choose a reason for hiding this comment

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

@dotlambda
Copy link
Member

@teutat3s Can you make an upstream PR committing package-lock.json to their repo so we don't have to vendor 3k lines?

@teutat3s teutat3s mentioned this pull request Sep 19, 2023
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
Status: Re-packaged
Development

Successfully merging this pull request may close these issues.

4 participants