-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Deploy from layout #4945
Deploy from layout #4945
Conversation
Does this mean we can stop generating the .pfx key files as well? Or will that still be needed for layout installs? |
vnext/PropertySheets/External/Microsoft.ReactNative.Uwp.CSharpApp.targets
Outdated
Show resolved
Hide resolved
vnext/PropertySheets/External/Microsoft.ReactNative.Uwp.CSharpApp.targets
Outdated
Show resolved
Hide resolved
vnext/PropertySheets/External/Microsoft.ReactNative.Uwp.CSharpApp.targets
Show resolved
Hide resolved
@dannyvv it's not strictly needed for layout installs but they are needed for appx. We still produce the appx even in Debug (so that you can carry the appx to a different machine and install it there), but installing from layout avoids having to elevate to install the certificate and is faster than installing an APPX |
* Reduce need for custom metro configs * Change files * Fix for integration tests
…tive-windows into deployFromLayout
change/react-native-windows-2020-05-17-09-26-39-MS_CallbackMultiArgs.json
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* deploy from layout
Visual Studio is able to deploy a UWP app from layout (i.e. without installing the APPX). This has several benefits for inner-loop development: installing an APPX requires elevation to pre-install the certificate that was used to sign the APPX with, which is unnecessary and cumbersome for Debug builds while you're writing the app.
This workflow works well in VS, however the command-line experience is broken. There are docs for Add-AppxPackage which tell us how to register a UWP app from layout. This seems to work for native apps but it doesn't work for .net UWP apps: apps will crash on launch. The symptom is that the desktop .net framework gets loaded instead of the .net framework for UWP.
As part of #4749 we disabled layout registration due to these UWP dotnet issues.
This PR fixes #4760 (the follow up bug to reenable .net UWP layout registration in Debug mode)
More details here
Microsoft Reviewers: Open in CodeFlow