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

Remove explicit SDK version for Windows builds #483

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

chrisglein
Copy link
Contributor

Overview

Fix SDK build error if 10.0.18362.0 is not installed:

× Building Solution: C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft...
× Build failed with message C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.Cpp.WindowsSDK.targets(46,5): error MSB8036: The Windows SDK version 10.0.18362.0 was not found. Install the required version of Windows SDK or change the SDK version in the project property pages or by right-clicking the solution and selecting "Retarget solution".

See react-native-async-storage/async-storage#810 for a similar fix.

Previously, after bringing down package dependencies you'd have to retarget the SDK version for the module project.

Test Plan

Committed to a fork and redirected an app with a Windows build to this. Build error goes away.

@chrisglein chrisglein requested a review from Naturalclar as a code owner March 24, 2023 18:28
chrisglein added a commit to chrisglein/artificial-chat that referenced this pull request Apr 13, 2023
@Naturalclar Naturalclar merged commit fc52bca into react-native-picker:master Sep 20, 2023
10 of 12 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 20, 2023
## [2.5.1](v2.5.0...v2.5.1) (2023-09-20)

### Bug Fixes

* Remove explicit SDK version for Windows builds ([#483](#483)) ([fc52bca](fc52bca))
@Naturalclar
Copy link
Contributor

🎉 This PR is included in version 2.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants