Skip to content

Feature: Add new backend: nix#91

Draft
PhilTaken wants to merge 1 commit intolspcontainers:mainfrom
PhilTaken:main
Draft

Feature: Add new backend: nix#91
PhilTaken wants to merge 1 commit intolspcontainers:mainfrom
PhilTaken:main

Conversation

@PhilTaken
Copy link

This PR introduces a new optional backend to pull the language servers from: nix.
Nix does not necessarily fit the containerization theme of this plugin, but provides a very similar user experience to the docker backend. It also cuts down on the (albeit very slim) containerization overhead.

The nix backend provides the same interface as the docker/podman approach in that it can pull/build and delete language servers dynamically to save on disk space.

Since this is quite a big change relatively speaking, I consider this more of a draft for now.

This PR introduces a new optional backend to pull the language servers
from: nix.
Nix does not necessarily fit the containerization theme of this plugin,
but provides a very similar user experience to the docker backend.
It also cuts down on the (albeit very slim) containerization overhead.

The nix backend provides the same interface as the docker/podman
approach in that it can pull/build and delete language servers dynamically
to save on disk space.
@WhyNotHugo
Copy link
Contributor

WhyNotHugo commented Nov 8, 2022

nix-provided LSPs can be sandboxed with bubblewrap. I used this briefly and it worked pretty well. I eventually moved from nix to just using my distro's packages, so don't have many references on it.

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.

2 participants