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 to mono:2018-08 #4596

Merged
merged 113 commits into from
Dec 6, 2018
Merged

Bump to mono:2018-08 #4596

merged 113 commits into from
Dec 6, 2018

Conversation

directhex
Copy link
Contributor

@directhex directhex commented Aug 8, 2018

No description provided.

directhex 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
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
Collaborator

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
Member

@rolfbjarne
Copy link
Member

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

screenshot 2018-11-21 11 59 05

@marek-safar
Copy link
Contributor

marek-safar commented Nov 22, 2018

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
Member

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

@xamarin-release-manager
Copy link
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
Collaborator

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
Member

Test failures on internal Jenkins are unrelated:

Test failures on public Jenkins are unrelated:

@GouriKumari
Copy link
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
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
Contributor

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

@spouliot
Copy link
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
Collaborator

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
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
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
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
Collaborator

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

@dalexsoto
Copy link
Member

dalexsoto commented Dec 5, 2018

@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
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
Copy link
Contributor

spouliot commented Dec 5, 2018

@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
Copy link
Contributor

luhenry commented Dec 6, 2018

🎉

@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.