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

angular-language-server: collision with typescript #387400

Closed
3 tasks done
xdinterface opened this issue Mar 5, 2025 · 5 comments · Fixed by #390084
Closed
3 tasks done

angular-language-server: collision with typescript #387400

xdinterface opened this issue Mar 5, 2025 · 5 comments · Fixed by #390084
Labels
0.kind: bug Something is broken

Comments

@xdinterface
Copy link

Nixpkgs version

  • Stable (24.11)

Describe the bug

I am trying to add the angular-language-server to my packages but it collides with my typescript package.

Steps to reproduce

Add typescript and angular-language-server to your packages and use nixos-rebuild switch to rebuild the system.

Expected behaviour

Both packages should be able to be installed alongside each other.

Screenshots

No response

Relevant log output

error: builder for '/nix/store/qkbvpaz7r4738h46l2nm6lw4jwfmi439-home-manager-path.drv' failed with exit code 2;
       last 2 log lines:
       > warning: different permissions in `/nix/store/gz2hn0l6dz4qbvjdliz5517aan87732x-typescript-5.6.3/lib/node_modules/typescript/ThirdPartyNoticeText.txt' and `/nix/store/16ia3wrrhknxckkh1g9h4y7x4dvg28xj-angular-language-server-19.1.0/lib/node_modules/typescript/ThirdPartyNoticeText.txt': 0444 <-> 0555 at /nix/store/g1yy0f7gq66w9bd5nd9jr2g3jsff9nil-builder.pl line 103.
       > error: collision between `/nix/store/16ia3wrrhknxckkh1g9h4y7x4dvg28xj-angular-language-server-19.1.0/lib/node_modules/typescript/ThirdPartyNoticeText.txt' and `/nix/store/gz2hn0l6dz4qbvjdliz5517aan87732x-typescript-5.6.3/lib/node_modules/typescript/ThirdPartyNoticeText.txt'
       For full logs, run 'nix log /nix/store/qkbvpaz7r4738h46l2nm6lw4jwfmi439-home-manager-path.drv'.
error: 1 dependencies of derivation '/nix/store/yyzflxb3iqp33d72r2z9jkfmkbyai77l-home-manager-generation.drv' failed to build
error: 1 dependencies of derivation '/nix/store/3zrr2zagxfbwipqp0r50xymfp29byr5i-user-environment.drv' failed to build
error: 1 dependencies of derivation '/nix/store/j0k8qgv8dr9q1fsv0l2hgp65q1da4jf1-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/2lgcygfp1g9c3hnr9ymf52c9qddr37f9-nixos-system-framework-24.11.20250302.1546c45.drv' failed to build

Additional context

No response

System metadata

  • system: "x86_64-linux"
  • host os: Linux 6.13.5, NixOS, 24.11 (Vicuna), 24.11.20250302.1546c45
  • multi-user?: yes
  • sandbox: yes
  • version: nix-env (Nix) 2.24.12
  • channels(root): "nixos-24.05, nixos-unstable"
  • nixpkgs: /nix/store/070ny7v4l1lbwx44ac3bsagzwx9mjdcy-source

Notify maintainers

@tricktron

Note for maintainers: Please tag this issue in your pull request description. (i.e. Resolves #ISSUE.)

I assert that this issue is relevant for Nixpkgs

Is this issue important to you?

Add a 👍 reaction to issues you find important.

@xdinterface xdinterface added the 0.kind: bug Something is broken label Mar 5, 2025
@eclairevoyant
Copy link
Contributor

Typescript isn't the kind of thing that goes in your system packages. Look into https://nix.dev/tutorials/first-steps/declarative-shell per project.

@xdinterface
Copy link
Author

Typescript isn't the kind of thing that goes in your system packages. Look into https://nix.dev/tutorials/first-steps/declarative-shell per project.

Nix packages should generally not have the license present at root. It should go in a directory like $out/share/$name/ or similar, to avoid collision.
This has nothing to do with shells.

@tricktron
Copy link
Member

@xdinterface Thanks for reporting this. I might need to adapt the file permissions of the node modules in the angular-language-server package.

I'll have a look at this.

@eclairevoyant
Copy link
Contributor

eclairevoyant commented Mar 6, 2025

Typescript isn't the kind of thing that goes in your system packages. Look into https://nix.dev/tutorials/first-steps/declarative-shell per project.

Nix packages should generally not have the license present at root.

While that is true, again, typescript doesn't go in one's config. That would avoid this becoming a blocking issue in the first place.

@xdinterface
Copy link
Author

Typescript isn't the kind of thing that goes in your system packages. Look into https://nix.dev/tutorials/first-steps/declarative-shell per project.

Nix packages should generally not have the license present at root.

While that is true, again, typescript doesn't go in one's config. That would avoid this becoming a blocking issue in the first place.

I see your point about nix-shells being good practice, but I still think properly packaged Nix modules shouldn't collide regardless of how they're installed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants