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

Make the PATH configurable #246

Open
fpoli opened this issue Jan 22, 2024 · 2 comments
Open

Make the PATH configurable #246

fpoli opened this issue Jan 22, 2024 · 2 comments
Labels
enhancement New feature or request good-first-issue Good for newcomers

Comments

@fpoli
Copy link
Member

fpoli commented Jan 22, 2024

Currently, rustup needs to be in the PATH used by VS Code. This is not flexible enough for some use cases. See the comment here.

For improved flexibility, we could add an optional configuration field specifying the PATH to be used in place of the one available to VS Code. When such field is empty, Prusti Assitant should still use VS Code's PATH as it currently does.
It is necessary to configure the full PATH instead of just rustup's path, because Prusti internally searches for rustup in the PATH that we pass via Prusti Assistant.

Relevant source files:

@fpoli fpoli added enhancement New feature or request good-first-issue Good for newcomers labels Jan 22, 2024
@fpoli fpoli changed the title Make the rustup path configurable Make the PATH configurable Jan 22, 2024
@jaivardhan-bhola
Copy link

Can I work on this?

@fpoli
Copy link
Member Author

fpoli commented Jan 23, 2024

Sure! Let me know if you have any questions or suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good-first-issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants