Conversation
Also added a configurable wrapper that puts a TeX Distribution into PATH. Defaults to texlive.combined.scheme-small. Without this wrapper, the following message is displayed on startup: Your TeX distribution could not be detected. Please make sure that your distribution is in your PATH.
|
Why do you need the wrapper? |
|
@veprbl The wrapper was added to ensure purity, so that the language server isn't dependent on the environment, and fully functionally without having to install additional packages. Why is this a problem? |
|
@Metadark You say "purity", but what you describe is actually "isolation". Latter, to some degree, can be achieved in different many ways depending on what task you have. It's really up to user to decide what kind of isolation they want to have. The problem is, I think, that putting |
|
@veprbl What I describe is isolation, but also purity. Relying on whatever is in The wrapper also doesn't prevent users from deciding what kind of "isolation" they want to have, it makes the decision more explicit: { config, pkgs, ... }:
{
environment.systemPackages = with pkgs; [
(texlab.override { tex = tectonic; })
];
}If you had multiple TeX distributions installed, then texlab would just select whatever one it finds first, which may not be the one you want. If anything, this should lead to fewer suprises. The same kind of thing is done for TeX distributions on:
Some projects do the patching in the patch / fixup phases, but I created a separate derivation so texlab doesn't need to be rebuilt to change the TeX distribution. |
|
Closing in favour of #85700 |
Also added a configurable wrapper that puts a TeX Distribution into
PATH.Defaults to
texlive.combined.scheme-small.Without this wrapper, the following message is displayed on startup:
Motivation for this change
Upgrade to latest version of texlab: https://github.com/latex-lsp/texlab/releases/tag/v2.0.0
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)@doronbehar