Skip to content
This repository was archived by the owner on Aug 24, 2026. It is now read-only.

Fix install check when multiple major versions installed - #190

Merged
David Barbet (dibarbet) merged 1 commit into
microsoft:mainfrom
dibarbet:fix_version_check
Aug 6, 2025
Merged

Fix install check when multiple major versions installed#190
David Barbet (dibarbet) merged 1 commit into
microsoft:mainfrom
dibarbet:fix_version_check

Conversation

@dibarbet

@dibarbet David Barbet (dibarbet) commented Aug 6, 2025

Copy link
Copy Markdown
Member

the original registry key being checked (e.g. VisualStudio.DTE.17.0) is overwritten when a new major version is installed. So if you have both installed side by side, the older instance is never found.

instead of using this registry key, just use the same discovery logic we use to find an instance to launch later on.

@dibarbet
David Barbet (dibarbet) marked this pull request as ready for review August 6, 2025 21:35
@jasonmalinowski

Copy link
Copy Markdown
Member

Looks like we're being broken by https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2521866

@dibarbet

Copy link
Copy Markdown
Member Author

failure is known, bypassing to unblock builds

@dibarbet
David Barbet (dibarbet) merged commit d9bef46 into microsoft:main Aug 6, 2025
3 of 5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants