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

[Feature Request] Add predefined variable with separator for environment variable PATH (also PYTHONPATH, CLASSPATH etc.) #180830

Open
pawelszramowski opened this issue Apr 25, 2023 · 4 comments
Assignees
Labels
feature-request Request for new features or functionality variable-resolving
Milestone

Comments

@pawelszramowski
Copy link

pawelszramowski commented Apr 25, 2023

I would like to request a new predefined variable, e.g. ${pathListSeparator}, that would be substituted with the separator used in the operating system's PATH environment variable, e.g. : for POSIX or ; for Windows.

Similarly to this feature request regarding ${pathSeparator}, I would also like it to have a shorter alias for improved readability, e.g. ${:}.

The variable could be used for modifications of not only the OS's PATH but also other environment variables that follow the same convention, e.g. Python's PYTHONPATH, Java's CLASSPATH, etc.

@roblourens roblourens assigned alexr00 and unassigned roblourens Apr 25, 2023
@alexr00 alexr00 added feature-request Request for new features or functionality variable-resolving labels Apr 26, 2023
@alexr00 alexr00 added this to the Backlog Candidates milestone Apr 26, 2023
@VSCodeTriageBot
Copy link
Collaborator

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@VSCodeTriageBot
Copy link
Collaborator

🙂 This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.

Happy Coding!

@starball5
Copy link

Related: #143289

@pawelszramowski
Copy link
Author

pawelszramowski commented May 27, 2023

I think the shorter aliases I propose (${/} for ${pathSeparator} and ${:} for ${pathListSeparator} or whatever the new variable ends up being called) alleviate the readability concerns raised there, e.g. ${workspaceFolder}${/}path${/}to${/}dir${:}${env:PATH}.

I like the variable name proposed there (${pathListSeparator}) better than my original proposal (${searchPathSeparator}) since it doesn't include the pathSeparator string which is good for case-insensitive search so I edited the first message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality variable-resolving
Projects
None yet
Development

No branches or pull requests

5 participants