-
Notifications
You must be signed in to change notification settings - Fork 507
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
[msbuild] Fix AdditionalAppExtensions entitlement logic #9477
Conversation
chamons
commented
Aug 24, 2020
- Fixes [Xcode 12] AdditionalAppExtensions entitlements path resolution add space and newline characters causing codesign to fail #9458
- Defining the variable over multiple lines changes output to include incorrect newline
- Auto tests did not catch, as they don't sign on bots due to infrastructure issues.
- Was refactored after manual tests and not retested
- Fixes xamarin#9458 - Defining the variable over multiple lines changes output to include incorrect newline - Auto tests did not catch, as they don't sign on bots due to infrastructure issues. - Was refactored _after_ manual tests and not retested
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.
Oh wow this is... well... I'll just 🙊
Build failure Test results1 tests failed, 76 tests passed.Failed tests
|
Failures are all networking:
|
@chamons failures are a known issue: https://github.com/xamarin/maccore/issues/1028 |
@monojenkins backport main |