Skip to content

Commit

Permalink
[msbuild] Allow setting EnableCodeSigning=false to disable code signi…
Browse files Browse the repository at this point in the history
…ng for Hot Restart. (#18013)

This makes writing tests easier, because we won't need a functioning code signing
setup in order to verify other parts of a build.
  • Loading branch information
rolfbjarne committed Apr 11, 2023
1 parent 5a3248d commit f853788
Showing 1 changed file with 2 additions and 1 deletion.
Expand Up @@ -36,6 +36,7 @@
Condition="'$(_CanOutputAppBundle)' == 'true' And '$(IsHotRestartBuild)' == 'true' And '$(IsHotRestartEnvironmentReady)' == 'true'" >

<HotRestart.Tasks.DetectSigningIdentity
Condition="'$(EnableCodeSigning)' != 'false'"
GenerateApplicationManifest="$(GenerateApplicationManifest)"
ApplicationId="$(ApplicationId)"
ApplicationTitle="$(ApplicationTitle)"
Expand Down Expand Up @@ -326,7 +327,7 @@
AfterTargets="_CodesignAppBundle" />

<Target Name="_CodesignHotRestartAppBundle"
Condition="'$(_CanOutputAppBundle)' == 'true' And '$(IsAppExtension)' == 'false' And '$(IsHotRestartBuild)' == 'true'"
Condition="'$(_CanOutputAppBundle)' == 'true' And '$(IsAppExtension)' == 'false' And '$(IsHotRestartBuild)' == 'true' And '$(EnableCodeSigning)' != 'false'"
DependsOnTargets="_CreateHotRestartCachedBundle;_CopyFrameworksToHotRestartBundle;_CollectCodeSignHotRestartInputs"
Inputs="@(_CodeSignHotRestartInputs)"
Outputs="@(_CodeSignHotRestartInputs -> '%(Outputs)')">
Expand Down

2 comments on commit f853788

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

Please sign in to comment.