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

environmentSetupScript exit code not checked #3566

Closed
joelfrederico opened this issue Feb 1, 2024 · 1 comment · Fixed by #3747
Closed

environmentSetupScript exit code not checked #3566

joelfrederico opened this issue Feb 1, 2024 · 1 comment · Fixed by #3747
Labels
bug a bug in the product suggested Temporary label for internal tracking for suggested bugs to pick up in spare time.
Milestone

Comments

@joelfrederico
Copy link

Brief Issue Summary

I had an environmentSetupScript that worked. It started failing when IT removed a feature from the machine I needed. But since VSCode doesn't check the script's exit code, I never had a chance to notice that it was failing.

CMake Tools Diagnostics

No response

Debug Log

No response

Additional Information

The result of the script is here:

const res = await proc.execute(run_command, [], null, { shell: true, silent: true }).result; // run script

As far as I can tell, its exit code is never checked. If it was checked, and an error raised, I could have noticed that my script wasn't working properly.

@gcampbell-msft
Copy link
Collaborator

@joelfrederico Thanks for the call-out we should make a failure here more obvious. Adding it to the backlog.

@gcampbell-msft gcampbell-msft added bug a bug in the product and removed triage labels Feb 6, 2024
@gcampbell-msft gcampbell-msft moved this from Triage Needed to Pending Prioritization in CMake Tools Feb 6, 2024
@gcampbell-msft gcampbell-msft added this to the On Deck milestone Feb 6, 2024
@gcampbell-msft gcampbell-msft added the suggested Temporary label for internal tracking for suggested bugs to pick up in spare time. label Apr 24, 2024
@github-project-automation github-project-automation bot moved this from Pending Prioritization to Completed in CMake Tools May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug a bug in the product suggested Temporary label for internal tracking for suggested bugs to pick up in spare time.
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

2 participants