Skip to content

Conversation

@rolfbjarne
Copy link
Member

This PR is best reviewed commit-by-commit.

Fixes #4864.

Backport of #6040.

rolfbjarne and others added 22 commits May 22, 2019 08:04
ARM64_32 only supports the unified native mode.
…s enabled.

We don't want to try to run arm64_32 tests on bots that don't have devices
attached, so make sure they follow the logic of other device tests.
* Choose the first hostname for the HttpTextWriter if there are multiple hosts.
* Open the HttpTextWriter before writing to it.
* Don't overwrite the http writer with another writer immediately after creating it.
* Close the HttpTextWriter when done writing.
* Wait for the HttpTextWriter to complete the final http request before exiting.
New commits in mono/mono:

* mono/mono@16c6ac81b39 [interp] disable inlining for built-in types (dotnet#14531)
* mono/mono@1ce13bb09d4 [arm] one more attempt to fix slotsize issue on llvmonly (dotnet#14444) (dotnet#14535)
* mono/mono@bb3c9db4976 [2019-02] [monodroid] Introduce 'win32 host' BCL build (dotnet#14504)
* mono/mono@ca17767758c [sdks] Set PATH in Android Windows SDK pipeline job (dotnet#14519)
* mono/mono@1cb80e7ecfc [sdks] Set win-amd64 CI_TAGS for windows pipeline step (dotnet#14516)
* mono/mono@c857c27d2e5 [2019-02][sdks] Build Android SDK on Windows (dotnet#14472)
* mono/mono@ab378434b96 [2019-02][ci] Add Android Windows (Release) archive job to pipeline (dotnet#14503)

Diff: https://github.com/mono/mono/compare/06c3f553122b17457147695475957ead3ec90b6a..16c6ac81b39bdb26786c2fda155a0a3500961798
@spouliot spouliot added this to the d16-2 milestone May 22, 2019
@spouliot
Copy link
Contributor

@rolfbjarne have you run the submission tests on the original PR/packages ?

@rolfbjarne
Copy link
Member Author

@rolfbjarne have you run the submission tests on the original PR/packages ?

No, I'll do that on the master packages and post the results here.

@rolfbjarne rolfbjarne changed the title Implement support for ARM64_32. Fixes #4864. [d16-2] Implement support for ARM64_32. Fixes #4864. May 22, 2019
@monojenkins
Copy link
Collaborator

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

Test results

2 tests failed, 0 tests' device not found, 229 tests passed.

Failed tests

  • [NUnit] Mono SystemTransactionsTests/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemLinqXunit/watchOS 32-bits - simulator/Debug: Crashed

@rolfbjarne
Copy link
Member Author

Test failures are unrelated, both are due to https://github.com/xamarin/maccore/issues/581.

@spouliot
Copy link
Contributor

While looking at XM/QTKit (#6039) I executed the submission tests (for master) and that also confirmed the original PR (on master) is fine.
https://gist.github.com/spouliot/600b49a196f348425649da231114ec45

@rolfbjarne rolfbjarne merged commit 48430a7 into dotnet:d16-2 May 23, 2019
@rolfbjarne
Copy link
Member Author

While looking at XM/QTKit (#6039) I executed the submission tests (for master) and that also confirmed the original PR (on master) is fine.

Thanks!

@rolfbjarne rolfbjarne mentioned this pull request May 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants