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.

Sign up for GitHub

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

Jump to bottom

[d16-0] Bump Xamarin.MacDev to get fix for #5277. #5284

Merged
merged 1 commit into from
Dec 19, 2018
Merged

[d16-0] Bump Xamarin.MacDev to get fix for #5277. #5284

merged 1 commit into from
Dec 19, 2018

Conversation

rolfbjarne
Copy link
Member

@rolfbjarne
Bump Xamarin.MacDev to get fix for #5277.
3d97aa5 
Commit list for xamarin/Xamarin.MacDev:

* xamarin/Xamarin.MacDev@44e90a2 Keys in property list dictionaries aren't necessarily unique. Fixes xamarin-macios#5277. (#39) (#41)
* xamarin/Xamarin.MacDev@c81c625 Get rid of spurious <ItemGroup>
* xamarin/Xamarin.MacDev@d1d3fa4 Remove packagerefs. Leads to unintended problems (see discussion xamarin/md-addins#3946) (#38)

Diff: xamarin/Xamarin.MacDev@179f720...44e90a2

Fixes #5277.
@rolfbjarne rolfbjarne added run-internal-tests If tests should be executed on the internal Jenkins instance. skip-public-jenkins Completely skip execution in the public Jenkins instance labels Dec 17, 2018
@rolfbjarne rolfbjarne changed the title Bump Xamarin.MacDev to get fix for #5277. [d16-0] Bump Xamarin.MacDev to get fix for #5277. Dec 17, 2018
@rolfbjarne rolfbjarne added this to the d16-0 milestone Dec 17, 2018
@rolfbjarne rolfbjarne added the build-package Build (and create package) on internal Jenkins. Apply 'run-internal-tests' to run tests too. label Dec 17, 2018
@xamarin-release-manager
Copy link
Collaborator

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run' 🔥

✅ Build succeeded
✅ API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
✅ Generator Diff (no change)
🔥 Test run failed 🔥

Test results

2 tests failed, 0 tests skipped, 282 tests passed.

Failed tests

  • MSBuild tests/iOS: Failed (Execution failed with exit code 24)
  • MTouch tests/NUnit: Failed (Execution failed with exit code 6)

@rolfbjarne
Copy link
Member Author

Test failures were due to #5277 (comment). I've fixed the bots according to the workaround, and re-scheduled a build.

@xamarin-release-manager
Copy link
Collaborator

❌ Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run' 🔥 : org.jenkinsci.plugins.workflow.steps.FlowInterruptedException

✅ Build succeeded
✅ API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
✅ Generator Diff (no change)

@xamarin-release-manager
Copy link
Collaborator

❌ Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Build' 🔥 : hudson.AbortException: script returned exit code 2

🔥 Build failed 🔥

@xamarin-release-manager
Copy link
Collaborator

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run' 🔥

✅ Build succeeded
✅ API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
✅ Generator Diff (no change)
🔥 Test run failed 🔥

Test results

2 tests failed, 0 tests skipped, 282 tests passed.

Failed tests

  • [NUnit] Mono MonoParallelTests/watchOS - simulator/Debug: Crashed
  • MSBuild tests/iOS: Failed (Execution failed with exit code 2)

@rolfbjarne
Copy link
Member Author

Test failures are known issues:

@rolfbjarne rolfbjarne merged commit 12adbf6 into d16-0 Dec 19, 2018
@rolfbjarne rolfbjarne deleted the issue-5277-bump-d16-0 branch December 19, 2018 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spouliot spouliot spouliot approved these changes

Assignees
No one assigned
Labels
build-package Build (and create package) on internal Jenkins. Apply 'run-internal-tests' to run tests too. run-internal-tests If tests should be executed on the internal Jenkins instance. skip-public-jenkins Completely skip execution in the public Jenkins instance
Projects
None yet
Milestone
d16-0
Development

Successfully merging this pull request may close these issues.
3 participants
@rolfbjarne @xamarin-release-manager @spouliot