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

Cross-platform env variable syntax in vhdl_ls.toml #346

Open
davidgussler opened this issue Oct 14, 2024 · 1 comment
Open

Cross-platform env variable syntax in vhdl_ls.toml #346

davidgussler opened this issue Oct 14, 2024 · 1 comment

Comments

@davidgussler
Copy link

davidgussler commented Oct 14, 2024

I'm requesting that the syntax for environment variables be treated the same way for both Windows and Linux variables so that a vhdl_ls.toml can be checked into version control along with a project and work in any OS without modification.

This is mainly for handling vendor libraries that are installed on a machine, and not directly part of the project. See differences highlighted below:

Windows:

UNISIM.files = [
  '%XILINX_VIVADO%/data/vhdl/src/unisims/unisim_VCOMP.vhd',
]

Linux:

UNISIM.files = [
  '$XILINX_VIVADO/data/vhdl/src/unisims/unisim_VCOMP.vhd',
]

It would be ideal if Windows variables could be handled the same way as Linux variables. Thank you.

@Schottkyc137
Copy link
Contributor

So the decision to separate windows and unix-like systems was made based on #278. However, I think that there should be a more flexible solution. Probably the best way forward would be to use unix-style substitution (i.e., $VARIABLE), but allow the escape char ($) to be dynamically changeable from some config.

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

No branches or pull requests

2 participants