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

Applied latest Messaging and Hot Restart changes from main #15177

Merged
merged 5 commits into from
Jun 3, 2022

Conversation

mauroa
Copy link
Contributor

@mauroa mauroa commented Jun 1, 2022

These changes are needed to for 17.3 Preview 2 and net6

emaf and others added 3 commits June 1, 2022 17:50
The compiled entitlements should be placed in the intermediate Hot Restart app bundle so those can be picked up by the HotRestart Codesign task. Prior to this change, entitlements set in the project wouldn't be included in the app, making things like Keychain Access fail, even though it was configured.

Fixes https://developercommunity.visualstudio.com/t/Unable-to-use-MSAL-with-locally-connecte/1573064
…ac (#15026)

This should fix an annoying warning where the user builds an iOS project offline and get a message about that target couldn't be executed
@mauroa mauroa added the not-notes-worthy Ignore for release notes label Jun 1, 2022
@mauroa mauroa requested a review from dalexsoto June 1, 2022 21:58
@mauroa mauroa self-assigned this Jun 1, 2022
@rolfbjarne
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@dalexsoto
Copy link
Member

@mandel-macaque can you have a look?

@mandel-macaque
Copy link
Member

yes, this is because we do not run all branches, let me rebase to see if the new changes in the CI pipeline work, else I'll do a pr with the update that is needed.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 [PR Build] Build failed 🔥

Build failed for the job 'Generate API diff'

Pipeline on Agent
Hash:

@vs-mobiletools-engineering-service2
Copy link
Collaborator

📚 [PR Build] Artifacts 📚

Packages generated

View packages

Pipeline on Agent XAMBOT-1042.Monterey
Hash: 213c8a536eb9d4ebb2448ca9d15fedbcee793269

@vs-mobiletools-engineering-service2
Copy link
Collaborator

💻 [PR Build] Tests on macOS Mac Catalina (10.15) passed 💻

All tests on macOS Mac Catalina (10.15) passed.

Pipeline on Agent
Hash: 213c8a536eb9d4ebb2448ca9d15fedbcee793269

@mauroa
Copy link
Contributor Author

mauroa commented Jun 3, 2022

@rolfbjarne @dalexsoto is this ready for merge?

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [CI Build] Tests failed on VSTS: simulator tests iOS ❌

Tests failed on VSTS: simulator tests iOS.

Test results

2 tests failed, 147 tests passed.

Failed tests

  • monotouch-test/iOS Unified 64-bits - simulator/Debug: Failed
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): Failed

Pipeline on Agent XAMBOT-1042.Monterey'
Merge 213c8a5 into 320d083

@rolfbjarne
Copy link
Member

Test failures are unrelated (https://github.com/xamarin/maccore/issues/1265).

@rolfbjarne rolfbjarne merged commit c5eafcf into release/6.0.4xx Jun 3, 2022
@rolfbjarne rolfbjarne deleted the dev/mag/6.0.4xx branch June 3, 2022 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-notes-worthy Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants