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

[release/9.0.1xx-rc2] Correctly handle different architectures for devdeviceID #43521

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 17, 2024

Backport of #43471 to release/9.0.1xx-rc2
Summary

DevDeviceID was introduced in RC2 as a way to be consistent with Visual Studio on how to track individual devices. There was a bug in the initial implementation where on windows, we would use the SDK architecture version of the registry for reading and storing the cached value

Customer Impact

x86 SDKs would have a completely unique devdevice ID to the x64 version on the same box. The spec calls for the same id regardless of architecture.

Regression

No, implemented incorrectly.

Testing

Minimal

Risk

Very low

@marcpopMSFT marcpopMSFT changed the title [release/9.0.1xx-rc2] Correctly handle different architectures of the SDK [release/9.0.1xx-rc2] Correctly handle different architectures for devdeviceID Sep 18, 2024
@marcpopMSFT marcpopMSFT merged commit 7163c0a into release/9.0.1xx-rc2 Sep 18, 2024
31 checks passed
@marcpopMSFT marcpopMSFT deleted the backport/pr-43471-to-release/9.0.1xx-rc2 branch September 18, 2024 18:39
@marcpopMSFT marcpopMSFT added Servicing-approved and removed Servicing-consider untriaged Request triage from a team member labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant