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

Fixes Hot Restart build issues #12500

Merged
merged 5 commits into from
Aug 24, 2021
Merged

Fixes Hot Restart build issues #12500

merged 5 commits into from
Aug 24, 2021

Conversation

emaf
Copy link
Contributor

@emaf emaf commented Aug 20, 2021

The changes are explained on each commit :), two the problems fixed on this PR are related to Maui Single Projects.

emaf added 3 commits August 20, 2021 18:25
This version contains fixes for building Maui projects with Hot Restart
The build was failing if `CodesignEntitlements` was not set, even though the CompileEntitlements task has a default value. That default value is not compatible with Hot Restart because it is a template file that exists on the Mac (and Hot Restart is an offline build from Windows).

So if that property is not set we get the xcent file from the Hot Restart PreBuilt app bundle, which is essentially an empty plist.
On a Maui Single Project the app title can be set on the project file using the `ApplicationTitle` property. If that's set Hot Restart should include that value in the compiled app manifest, so the app name is shown on the device when the app is deployed.
@emaf emaf requested a review from mauroa August 20, 2021 21:57
@emaf emaf requested a review from rolfbjarne as a code owner August 20, 2021 21:57
@emaf
Copy link
Contributor Author

emaf commented Aug 20, 2021

Oh, and all these changes should be included in RC1 if possible 😬

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Tests failed catastrophically on Build (no summary found). 🔥

Result file $(TEST_SUMMARY_PATH) not found.

Pipeline on Agent
Merge 566ab4e into 34a4d0d

@emaf
Copy link
Contributor Author

emaf commented Aug 23, 2021

/sudo backport release/6.0.1xx-rc.1

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Backport Job to branch release/6.0.1xx-rc.1 Created! The magic is happening here

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Hooray! Backport succeeded! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5121723 for more details.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (only version changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

1 tests failed, 142 tests passed.

Failed tests

  • framework-test/Mac Catalyst/Debug: TimedOut (Execution timed out after 1200 seconds.
    No test log file was produced)

Pipeline on Agent XAMBOT-1099.BigSur'
Merge 2b97b7d into 643a924

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (only version changes)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

1 tests failed, 142 tests passed.

Failed tests

  • link all/Mac Catalyst/Debug [dotnet]: Failed (Test run crashed (exit code: 134).
    Tests run: 90 Passed: 79 Inconclusive: 0 Failed: 0 Ignored: 11)

Pipeline on Agent XAMBOT-1104.BigSur'
Merge b59a769 into 28fc7bb

@rolfbjarne
Copy link
Member

Test failure is unrelated (https://github.com/xamarin/maccore/issues/2479).

@rolfbjarne rolfbjarne merged commit fbbaa7f into main Aug 24, 2021
@rolfbjarne rolfbjarne deleted the dev/ema/hr-fixes branch August 24, 2021 06:13
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.

8 participants