Skip to content

Conversation

@jonthysell
Copy link
Contributor

@jonthysell jonthysell commented Feb 13, 2020

  • All builds moved to VS 2019 VMs
  • Only v142 toolset installed
  • Solutions and projects updated to VS 16.0, to not require v141
  • CLI scripts updated to not use v141

Addresses issues #3476, #3801, #3960

Microsoft Reviewers: Open in CodeFlow

@jonthysell jonthysell requested a review from a team as a code owner February 13, 2020 21:11
@asklar
Copy link
Member

asklar commented Feb 13, 2020

link this issue #3476

Copy link
Contributor

@JunielKatarn JunielKatarn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we ready for this change yet?

Last time this was discussed, there were dependent projects that were forced to use v141.
In reality, v142 and v141 are not binary-compatible (when linking with optimizations).

@ghost ghost added the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Feb 13, 2020
@asklar
Copy link
Member

asklar commented Feb 13, 2020

we couldn't get any details on why 141 is needed since October. I say we're ready for this change and if it breaks something, we'll hear about it and can react then.


In reply to: 358572903 [](ancestors = 358572903)

@JunielKatarn
Copy link
Contributor

we couldn't get any details on why 141 is needed since October. I say we're ready for this change and if it breaks something, we'll hear about it and can react then.

In reply to: 358572903 [](ancestors = 358572903)

We would hear about it almost immediately.
I have sent a message with details.
Can we discuss offline?

@asklar
Copy link
Member

asklar commented Feb 13, 2020

can do away with passing version and always assume 16


Refers to: vnext/local-cli/runWindows/utils/msbuildtools.js:165 in f3cd019. [](commit_id = f3cd019, deletion_comment = False)

@asklar
Copy link
Member

asklar commented Feb 13, 2020

    });

don't need this, just call checkMSBuildVersion(16, ...)

omit version if you'd like


Refers to: vnext/local-cli/runWindows/utils/msbuildtools.js:199 in f3cd019. [](commit_id = f3cd019, deletion_comment = False)

@jonthysell
Copy link
Contributor Author

@jonthysell Jon Thysell (JAUNTY) FTE

Is the build process not working without calling VCVars?

Does this happen locally, or only during CI?

It was a red herring - I got the build working for c# apps with msbuild-x64 by preventing the buggy target from running. #4462

Copy link
Member

@asklar asklar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

Copy link
Contributor

@JunielKatarn JunielKatarn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unblocking, but need to review again before explicitly signing off.

@JunielKatarn JunielKatarn dismissed their stale review April 2, 2020 03:59

Reviewing again...

Copy link
Member

@vmoroz vmoroz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@jonthysell jonthysell merged commit 8ccb8eb into microsoft:master Apr 2, 2020
@jonthysell jonthysell deleted the vs2019 branch April 2, 2020 23:32
@rkokXXVV
Copy link

Hey guys. Is the support for VS2019 with v142 tools now live? I've just tried to install RNW for RN0.61.5 and it is still failing to build with VS2019 and v142

Cheers

@asklar
Copy link
Member

asklar commented May 16, 2020

Hey guys. Is the support for VS2019 with v142 tools now live? I've just tried to install RNW for RN0.61.5 and it is still failing to build with VS2019 and v142

Cheers

@rkokXXVV it is live but likely not something we'll backport. check out what's in master today instead! :)

@NickGerleman
Copy link
Contributor

Alex is right, that you can test bleeding edge features using our master branch. It's important to know that our master branch won't work with stable releases of React Native, and sees frequent breaking changes.

Visual Studio V142 support is included in React Native Windows 0.62, which we're in the process of taking out of prerelease. You should get it automatically if you run react-native-windows-init on a React Native 0.62 project.

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.

Use v142 for E2E test on vs2019 build machine Use windows-2019 as build machine Use latest toolset, optionally use v141

7 participants