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

Bump Xamarin.MacDev. #8059

Merged
merged 2 commits into from
Mar 9, 2020
Merged

Conversation

rolfbjarne
Copy link
Member

New commits in xamarin/Xamarin.MacDev:

* xamarin/Xamarin.MacDev@a0a11af Adjust SDK validation to allow tools/bin and not allow usr/bin. (xamarin#71)
* xamarin/Xamarin.MacDev@e21e1aa Accept 'tools/buildinfo' as an alternative path to 'buildinfo' due to an nuget bug. (xamarin#70)
* xamarin/Xamarin.MacDev@ce24236 Don't look in /Developer/MonoTouch anymore, there's nothing there. (xamarin#69)

Diff: https://github.com/xamarin/Xamarin.MacDev/compare/210c664e56117d3a1a7f6dd002109eb444dbdc17..a0a11aff271565bbaba029e5e428a0b9c2ca0e37
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Mar 6, 2020
@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

2 tests failed, 85 tests passed.

Failed tests

  • MSBuild tests/iOS (net461): Failed (Execution failed with exit code 3)
  • MSBuild tests/iOS (netstandard2.0): Failed (Execution failed with exit code 3)

MonoTouchSdk ends up a bit confused when giving it an Sdk directory that isn't
a real Sdk directory, and then assuming that properties that return Sdk paths
won't throw exceptions.

Fix this by giving MonoTouchSdk a better fake Sdk directory (which
MonoTouchSdk detects as a real Sdk directory), and thus those previously
mentioned properties will return true fake values instead of throwing
exceptions.
@monojenkins
Copy link
Collaborator

Build success
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
Test run succeeded

@rolfbjarne rolfbjarne merged commit c68700a into xamarin:master Mar 9, 2020
@rolfbjarne rolfbjarne deleted the bump-xamarin-macdev branch October 21, 2022 07:08
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.

4 participants