Skip to content

Conversation

rhythmize
Copy link

@rhythmize rhythmize commented Aug 17, 2025

It seems like when running SITL the environment variable override is not respected.
Proposing a fix for the same

@rhythmize
Copy link
Author

@bugobliterator Can you please check?
I'm not sure what's the right branch to merge this change. Please let me know in case it's not master

@bugobliterator
Copy link
Member

@bugobliterator Can you please check? I'm not sure what's the right branch to merge this change. Please let me know in case it's not master

Hi @rhythmize this is being fixed as part of larger set of changes here https://github.com/bugobliterator/ardupilot_vscode_devenv/tree/pr-e2e-testing .

This PR will add massive improvements to Tool configuration and management. Which will include the change similar to above.

@magicrub
Copy link
Collaborator

magicrub commented Sep 5, 2025

There's been lots of changes recently. Is this PR still useful?

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

Successfully merging this pull request may close these issues.

3 participants