Skip to content

Bump to mono:2018-08 - #4596

Merged
spouliot merged 113 commits into
masterfrom
mono-2018-08
Dec 6, 2018
Merged

Bump to mono:2018-08#4596
spouliot merged 113 commits into
masterfrom
mono-2018-08

Conversation

@directhex

@directhex directhex commented Aug 8, 2018

Copy link
Copy Markdown
Contributor

No description provided.

Jo Shields and others added 25 commits June 15, 2018 12:43
`mono_jit_set_aot_only` is deprecated and accidentally broke with
mono/mono#7887

This should fix device tests with `mono-2018-06`
Commit list for mono/mono:

* mono/mono@7bcda192a06 Bump llvm to release_60/fc854b8ec5873d294b80afa3e6cf6a88c5c48886. (#9786). (#9804)
* mono/mono@23e95ec7ad7 Apply F# portable pdb debug fix for pinvokes & bump (#9797)
* mono/mono@295f6d32afd [2018-06] [MacOS] On Mac, use the copyfile API to copy files (#9696)

Diff: mono/mono@7d5f4b6...7bcda19
Starting with mono 5.16 mcs will now add assembly references when the assembly
is only used in attributes (this was already the case for csc in both 5.14 and
5.16, so it seems to be a compatibility change).

Adjust the MT0137 test accordingly.
…s in the error message.

Fixes this test:

    1) Test Failure : Xamarin.iOS.Tasks.Bug60536.TestACToolTaskCatchesJsonException
         ColumnNumber
      Expected: 2
      But was:  0
@directhex
directhex requested a review from rolfbjarne as a code owner August 8, 2018 16:23
@directhex directhex added enable-device-build Makes our build include device support (which we disable for simple PRs to speed them up) run-all-tests Run all our tests. labels Aug 8, 2018
@xamarin-release-manager

Copy link
Copy Markdown
Collaborator

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Package XM tests' 🔥

Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
ℹ️ Test run skipped: Not running tests here because they're run on public Jenkins.

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

3 tests failed, 0 tests skipped, 281 tests passed.

Failed tests

  • apitest/Mac Classic/NonSystem: Failed (Test run crashed (exit code: 134).)
  • Xtro/Mac: Failed (Test run failed.)
  • monotouch-test/watchOS - simulator/Debug: Crashed

@rolfbjarne

Copy link
Copy Markdown
Member

Test failures on internal jenkins are unrelated:

Test failures on public jenkins are unrelated:

@rolfbjarne

Copy link
Copy Markdown
Member

@marek-safar there's an API break in mscorlib, is this expected?

screenshot 2018-11-21 11 59 05

@marek-safar

marek-safar commented Nov 22, 2018

Copy link
Copy Markdown
Contributor

Yes, this is expected. We had discussed it with XA in previous integration (this is not new in 2018-08). See for more details dotnet/android#1836 (comment) but in short, this is an infrastructure code removed bu CoreFX folks which was not intended to be used by users.

@rolfbjarne

Copy link
Copy Markdown
Member

I ran some of the device tests locally, and found that:

@xamarin-release-manager

Copy link
Copy Markdown
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Running XM tests on '10.9'' 🔥 : hudson.AbortException: Xamarin.Mac tests on macOS 10.9 failed (introspection, xammac_tests)

Build succeeded
API Diff (from stable)
⚠️ API Diff (from PR only) (🔥 breaking changes 🔥)
Generator Diff (no change)
ℹ️ Test run skipped: Not running tests here because they're run on public Jenkins.
🔥 Xamarin.Mac tests on macOS 10.9 failed (introspection, xammac_tests) 🔥

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

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

Failed tests

  • mmptest/macOS/NonSystem: Failed (Execution failed with exit code 2)
  • MMP Regression Tests/Mac: Failed

@rolfbjarne

Copy link
Copy Markdown
Member

Test failures on internal Jenkins are unrelated:

Test failures on public Jenkins are unrelated:

@GouriKumari

Copy link
Copy Markdown
Contributor

Sample building tests for XI and XM are executed successfully with mono2018-08 branch. IDE integration tests are also executed against the mono2018-08 branch. Failures are all known issues and not related mono2018-08.

…alling the AOT compiler. Works around mono/mono#11765.

Visual Studio for Mac may set MONO_THREADS_SUSPEND, and this ends up confusing
the AOT compiler when compiling for watchOS.

So unset the environment variable before calling the AOT compiler.

Works around mono/mono#11765.
@xamarin-release-manager

Copy link
Copy Markdown
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Running XM tests on '10.9'' 🔥 : hudson.AbortException: Xamarin.Mac tests on macOS 10.9 failed (introspection, xammac_tests)

Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
ℹ️ Test run skipped: Not running tests here because they're run on public Jenkins.
🔥 Xamarin.Mac tests on macOS 10.9 failed (introspection, xammac_tests) 🔥

@marek-safar

Copy link
Copy Markdown
Contributor

@spouliot are we ready or do we have to restart again because we now have conflict again?

@spouliot

Copy link
Copy Markdown
Contributor

@marek-safar the conflict does not matter. We're waiting on the watchOS results - which tests restarted now that we have a workaround. @rolfbjarne will comment here with the results once the run is complete.

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

3 tests failed, 0 tests skipped, 281 tests passed.

Failed tests

  • monotouch-test/watchOS - simulator/Debug: Failed
  • mmptest/macOS/NonSystem: Failed (Execution failed with exit code 2)
  • MMP Regression Tests/Mac: Failed

@rolfbjarne

Copy link
Copy Markdown
Member

The watchOS device tests ran fine (there were a few failing tests, but I ran the same tests with master and they failed there too, so they're not regressions. I'll file issues about these later).

The test failures from the latest comment from public Jenkins are unrelated:

So this is good to go.

@xamarin-release-manager

Copy link
Copy Markdown
Collaborator

Jenkins job (on internal Jenkins) succeeded

Build succeeded
API Diff (from stable)
⚠️ API Diff (from PR only) (🔥 breaking changes 🔥)
Generator Diff (no change)
ℹ️ Test run skipped: Not running tests here because they're run on public Jenkins.

@dalexsoto

Copy link
Copy Markdown
Member

✅ Api diff breaking changes are expected and unrelated to this PR, they come from the last merge of master due to acf8357.

Xamarin.Mac.dll

Namespace ObjCRuntime

Type Changed: ObjCRuntime.Constants

Modified fields:

-	public const string CFNetworkLibrary = "/System/Library/Frameworks/CFNetwork.framework/CFNetwork"
+	public const string CFNetworkLibrary = "/System/Library/Frameworks/CoreServices.framework/Frameworks/CFNetwork.framework/CFNetwork";

-	public const string CoreBluetoothLibrary = "/System/Library/Frameworks//CoreBluetooth.framework/CoreBluetooth"
+	public const string CoreBluetoothLibrary = "/System/Library/Frameworks/IOBluetooth.framework/Versions/A/Frameworks/CoreBluetooth.framework/CoreBluetooth";

@monojenkins

Copy link
Copy Markdown
Contributor

Build success
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
Test run succeeded

@dalexsoto

dalexsoto commented Dec 5, 2018

Copy link
Copy Markdown
Member

@spouliot / @marek-safar it seems that https://github.com/mono/mono/commits/2018-08 has newer commits, do we want to bump this branch to the tip of https://github.com/mono/mono/commits/2018-08?

@marek-safar

Copy link
Copy Markdown
Contributor

I'd prefer not to at this point, we are still waiting on XA to merge but once that happens we'll normal bump in master

@spouliot

spouliot commented Dec 5, 2018

Copy link
Copy Markdown
Contributor

@dalexsoto thanks for noticing & pinging. Let's wait, as asked by @marek-safar, to minimize the number of moving parts.

@spouliot
spouliot merged commit 7b83195 into master Dec 6, 2018
@luhenry

luhenry commented Dec 6, 2018

Copy link
Copy Markdown
Contributor

🎉

@mandel-macaque
mandel-macaque deleted the mono-2018-08 branch December 7, 2021 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-package Build (and create package) on internal Jenkins. Apply 'run-internal-tests' to run tests too. do-not-merge Do not merge this pull request enable-device-build Makes our build include device support (which we disable for simple PRs to speed them up) run-all-tests Run all our tests. skip-api-comparison Skips API / generator diffs when testing pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.