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

lltex-ls-plus: init at 17.0.1; vscode-extensions.ltex-plus.vscode-ltex-plus: init at 14.0.2 #342724

Closed

Conversation

PhilippWoelfel
Copy link
Contributor

@PhilippWoelfel PhilippWoelfel commented Sep 18, 2024

Description of changes

This adds ltex-ls-plus and vscode-extensions.ltex-plus.vscode-ltex-plus, see https://ltex-plus.github.io/ltex-plus/ and https://github.com/ltex-plus/vscode-ltex-plus
It is a fork of ltex-ls (see https://github.com/valentjn/vscode-ltex) which seems to be unmaintained.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 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.

Add a 👍 reaction to pull requests you find important.

@PhilippWoelfel PhilippWoelfel marked this pull request as draft September 18, 2024 06:28
@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: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels Sep 18, 2024
@PhilippWoelfel PhilippWoelfel marked this pull request as ready for review September 18, 2024 19:03
@nyabinary
Copy link
Contributor

Shouldn't this also add the vscode extension as well?

@PhilippWoelfel
Copy link
Contributor Author

Shouldn't this also add the vscode extension as well?

I was thinking that the extension (vscode-ltex-plus) could be a separate pull request, if anyone is interested. Personally, I am using github:nix-community/nix-vscode-extensions to get the extension, and then configure it so that it uses the binary from this package (by setting the vscode option ltex.ltex-ls.path and also ltex.java.path).

Note that if we want to package the extension vscode-ltex-plus, we'd probably want to use the "offline" source, which already contains the binary from this package. Otherwise, the extension will just try to download the binary into the store path, which is doomed to fail. So these two packages don't really go together.

@nyabinary
Copy link
Contributor

Shouldn't this also add the vscode extension as well?

I was thinking that the extension (vscode-ltex-plus) could be a separate pull request, if anyone is interested. Personally, I am using github:nix-community/nix-vscode-extensions to get the extension, and then configure it so that it uses the binary from this package (by setting the vscode option ltex.ltex-ls.path and also ltex.java.path).

Note that if we want to package the extension vscode-ltex-plus, we'd probably want to use the "offline" source, which already contains the binary from this package. Otherwise, the extension will just try to download the binary into the store path, which is doomed to fail. So these two packages don't really go together.

It could be the same PR, but has to be a different commit :3

@PhilippWoelfel PhilippWoelfel marked this pull request as draft September 30, 2024 16:11
@PhilippWoelfel PhilippWoelfel marked this pull request as ready for review September 30, 2024 16:19
@PhilippWoelfel
Copy link
Contributor Author

It could be the same PR, but has to be a different commit :3

I added the extension. Not sure what the naming convention for the pull request is, though...?

@nyabinary
Copy link
Contributor

It could be the same PR, but has to be a different commit :3

I added the extension. Not sure what the naming convention for the pull request is, though...?

ltex-ls-plus: init at 17.0.1; vscode-extensions.ltex-plus.vscode-ltex-plus: init at 14.0.2

should be oki

@PhilippWoelfel PhilippWoelfel changed the title ltex-ls-plus: init at 17.0.1 lltex-ls-plus: init at 17.0.1; vscode-extensions.ltex-plus.vscode-ltex-plus: init at 14.0.2 Sep 30, 2024
pkgs/by-name/lt/ltex-ls-plus/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/lt/ltex-ls-plus/package.nix Outdated Show resolved Hide resolved
@github-actions github-actions bot added 10.rebuild-darwin: 1-10 and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin labels Dec 4, 2024
@FliegendeWurst
Copy link
Member

FliegendeWurst commented Dec 5, 2024

Please squash your commits into two (one for each package). Then it is good to go.

@austinbutler
Copy link
Member

FYI there's also two other open PRs for ltex-ls-plus:

  1. ltex-ls-plus: init at 18.2.0 #357814
  2. ltex-ls-plus: init at 18.3.0 #363410

Plus one already merged for the VS Code extension: #362085

@FliegendeWurst FliegendeWurst added the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 9, 2024
@austinbutler austinbutler mentioned this pull request Dec 9, 2024
13 tasks
@austinbutler austinbutler mentioned this pull request Dec 9, 2024
13 tasks
Copy link
Member

Choose a reason for hiding this comment

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

#357814 has been merged, adding the package, so this should be removed. Feel free to add yourself as maintainer though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: vscode 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants