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

feat: add nixd #6725

Closed
wants to merge 1 commit into from
Closed

feat: add nixd #6725

wants to merge 1 commit into from

Conversation

antoineco
Copy link

Describe your changes

Adds a package for nixd, a Nix language server.

Supersedes / closes #4822, which was seemlingly abandonned by its author.

Issue ticket number and link

Checklist before requesting a review

  • I have successfully tested installation of the package.
  • I have successfully tested the package after installation.

Screenshots

Screenshot 2024-08-02 at 14 21 04

@williambotman williambotman requested a review from a team August 2, 2024 12:21
@antoineco
Copy link
Author

@williamboman sorry for the ping, but would you mind taking a look?

Co-authored-by: plokid <910576949@qq.com>
@antoineco
Copy link
Author

The binary currently produced by nix build .#nixd links against two shared libraries produced by that same build: libnixd-lspserver.so and libnixd.so. This isn't well supported by the Mason registry, so there is no point leaving this open at the moment.

Besides, this PR requires an additional change on the CI infra to even pass the checks: mason-org/actions#9

Nix users are better off installing nixd from nixpkgs with nix profile install 'nixpkgs#nixd'.

Closing

@antoineco antoineco closed this Sep 14, 2024
@antoineco antoineco deleted the nixd branch September 14, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant