Skip to content

[main][watchOS] Add x86_64 simulator support#10059

Merged
dalexsoto merged 10 commits intodotnet:mainfrom
dalexsoto:main-watchsim64
Nov 20, 2020
Merged

[main][watchOS] Add x86_64 simulator support#10059
dalexsoto merged 10 commits intodotnet:mainfrom
dalexsoto:main-watchsim64

Conversation

@dalexsoto
Copy link
Member

@dalexsoto dalexsoto commented Nov 5, 2020

  • Bumps mono binaries to include x86_64 watchOS support
  • Build runtime/registrar x86_64 slices
  • Produce a 64 bit version of Xamarin.WatchOS.dll
  • Allow building x86_64 for watch simulators in mtouch
  • Let xharness know about x86_64

New commits in mono/mono:

Diff: https://github.com/mono/mono/compare/ca11fb0fd819a1e32565b0d626506f66fb9fd272..ac596375c762c6b8dbe3c802f0ce626004eab51c

* Bumps mono binaries to include x86_64 watchOS support
* Build runtime/registrar x86_64 slices
* Produce a 64 bit version of Xamarin.WatchOS.dll
* Allow building x86_64 for watch simulators in mtouch
* Let xharness know about x86_64

New commits in mono/mono:

* mono/mono@392fe5b87b2 [2020-02][watchOS] Add simwatch64 support (dotnet#20552)
* mono/mono@a22ed3f094e Fix potential crash for Encoder.Convert (dotnet#20522)
* mono/mono@970783731fc Bump to F# 5.0 (dotnet#20511)
* mono/mono@32ab5066f72 Bump msbuild to fix a build issue
* mono/mono@93a7fe77e8e Ensure special static slots respect alignment. (dotnet#20506)
* mono/mono@3db5b358413 [debugger] Switch to GC Unsafe in signal handler callbacks (dotnet#20495)
* mono/mono@af315f44c40 [2020-02][corlib] ThreadAbortException protection for ArraySortHelper (dotnet#20468)

Diff: https://github.com/mono/mono/compare/ca11fb0fd819a1e32565b0d626506f66fb9fd272..392fe5b87b2ed710202a84588b0f93ad43728cad
@dalexsoto dalexsoto added the note-highlight Worth calling out specifically in release notes label Nov 5, 2020
@dalexsoto dalexsoto added this to the Future milestone Nov 5, 2020
@dalexsoto dalexsoto linked an issue Nov 5, 2020 that may be closed by this pull request
@dalexsoto dalexsoto changed the title [watchOS] Add x86_64 simulator support [main][watchOS] Add x86_64 simulator support Nov 5, 2020
@monojenkins
Copy link
Contributor

monojenkins commented Nov 6, 2020

Build failure
Build failed or was aborted

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

Test results

Test run in progress: BuildQueued: 123, Built: 1, Running: 1, Succeeded: 1, Ignored: 1190

Looks like the bot ran out of space...

@dalexsoto
Copy link
Member Author

build

@monojenkins
Copy link
Contributor

Build failure

!!! Couldn't read commit file !!!

@monojenkins
Copy link
Contributor

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

Test results

Test run in progress: BuildQueued: 123, Built: 1, Running: 1, Succeeded: 1, Ignored: 1190

Copy link
Member

@rolfbjarne rolfbjarne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! It also looks like mtouch didn't need many changes to support fat watch simulator apps!

@monojenkins
Copy link
Contributor

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

Test results

# Test run in progress: Building: 2, BuildQueued: 8, Built: 87, Succeeded: 2, Ignored: 1210, BuildFailure: 7

Failed tests

  • monotouch-test/watchOS 32-bits - simulator/Debug: BuildFailure
  • framework-test/watchOS 32-bits - simulator/Debug: BuildFailure
  • interdependent-binding-projects/watchOS 32-bits - simulator/Debug: BuildFailure
  • link all/watchOS 32-bits - simulator/Debug: BuildFailure
  • link all/watchOS 32-bits - simulator/Release: BuildFailure
  • link sdk/watchOS 32-bits - simulator/Debug: BuildFailure
  • link sdk/watchOS 32-bits - simulator/Release: BuildFailure

@rolfbjarne
Copy link
Member

Native linking failed, undefined Objective-C class: FrameworkTest. The symbol 'OBJC_CLASS$_FrameworkTest' could not be found in any of the libraries or frameworks linked with your application.

This is because you have to add a x86_64 build of our test libraries in tests/test-libraries

@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)

@monojenkins
Copy link
Contributor

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

Test results

# Test run in progress: Running: 1, Succeeded: 95, Failed: 1, Ignored: 1210, Crashed: 9

Failed tests

  • monotouch-test/iOS Unified 64-bits - simulator/Debug: Failed
  • monotouch-test/watchOS 32-bits - simulator/Debug: Crashed
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): Crashed
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): Crashed
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): Crashed
  • [NUnit] Mono SystemNumericsTests/watchOS 32-bits - simulator/Debug: Crashed
  • [NUnit] Mono MonoRuntimeTests/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemNumericsXunit/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono MicrosoftCSharpXunit/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemCoreXunit Part 1/watchOS 32-bits - simulator/Debug: Crashed

@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded
🔥 Build failed 🔥

@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded
🔥 Build failed 🔥

@monojenkins
Copy link
Contributor

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

Test results

4 tests failed, 66 tests passed.

Failed tests

  • monotouch-test/watchOS 32-bits - simulator/Debug: Crashed
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): Crashed
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): Crashed
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): Crashed

@monojenkins
Copy link
Contributor

Build failure

!!! Couldn't read commit file !!!

@rolfbjarne
Copy link
Member

build

@monojenkins
Copy link
Contributor

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

Test results

4 tests failed, 66 tests passed.

Failed tests

  • monotouch-test/watchOS 32-bits - simulator/Debug: Failed
  • monotouch-test/watchOS 32-bits - simulator/Debug (LinkSdk): Failed
  • monotouch-test/watchOS 32-bits - simulator/Debug (static registrar): Failed
  • monotouch-test/watchOS 32-bits - simulator/Release (all optimizations): Failed

@dalexsoto
Copy link
Member Author

build

@monojenkins
Copy link
Contributor

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

@dalexsoto
Copy link
Member Author

dalexsoto commented Nov 20, 2020

Fake green aka gred 🤥

⚠️ Build test/test-libraries failed ⚠️

@monojenkins
Copy link
Contributor

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

@dalexsoto
Copy link
Member Author

Fake green aka gred 🤥

⚠️ Build test/test-libraries failed ⚠️

@rolfbjarne
Copy link
Member

build

@rolfbjarne
Copy link
Member

@dalexsoto it seems that merging main disabled the watch simulator tests again, which is why the PR turned greenish.

@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)

@monojenkins
Copy link
Contributor

Build failure
Build failed or was aborted

Provisioning succeeded

@monojenkins
Copy link
Contributor

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

@dalexsoto dalexsoto merged commit 72c7b1f into dotnet:main Nov 20, 2020
@dalexsoto dalexsoto deleted the main-watchsim64 branch November 20, 2020 14:44
dalexsoto added a commit to dalexsoto/xamarin-macios that referenced this pull request Nov 21, 2020
* [watchOS] Add x86_64 simulator support

* Bumps mono binaries to include x86_64 watchOS support
* Build runtime/registrar x86_64 slices
* Produce a 64 bit version of Xamarin.WatchOS.dll
* Allow building x86_64 for watch simulators in mtouch
* Let xharness know about x86_64

* [tests] Add x86_64 arch to test-libraries

* Make dotnet package aware of x64

* [ObjCRuntime] Fix computing if we're calling a stret function or not in a 64-bit watchOS simulator.

* [xharness] Re-enable some watchOS tests.

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>

New commits in mono/mono:

* mono/mono@ac596375c76 Add support for OP_FCONV_TO_I to mini-arm64.c. (dotnet#20548)
* mono/mono@392fe5b87b2 [2020-02][watchOS] Add simwatch64 support (dotnet#20552)
* mono/mono@a22ed3f094e Fix potential crash for Encoder.Convert (dotnet#20522)
* mono/mono@970783731fc Bump to F# 5.0 (dotnet#20511)
* mono/mono@32ab5066f72 Bump msbuild to fix a build issue
* mono/mono@93a7fe77e8e Ensure special static slots respect alignment. (dotnet#20506)
* mono/mono@3db5b358413 [debugger] Switch to GC Unsafe in signal handler callbacks (dotnet#20495)
* mono/mono@af315f44c40 [2020-02][corlib] ThreadAbortException protection for ArraySortHelper (dotnet#20468)
* mono/mono@ca11fb0fd81 [2020-02] Bump ikvm-fork to include mono/ikvm-fork#20 (dotnet#20452)

Diff: https://github.com/mono/mono/compare/be2226b5a1c57df065efc4c1cf008d581e5cec7d..ac596375c762c6b8dbe3c802f0ce626004eab51c
dalexsoto added a commit to dalexsoto/xamarin-macios that referenced this pull request Nov 21, 2020
* [watchOS] Add x86_64 simulator support

* Bumps mono binaries to include x86_64 watchOS support
* Build runtime/registrar x86_64 slices
* Produce a 64 bit version of Xamarin.WatchOS.dll
* Allow building x86_64 for watch simulators in mtouch
* Let xharness know about x86_64

* [tests] Add x86_64 arch to test-libraries

* Make dotnet package aware of x64

* [ObjCRuntime] Fix computing if we're calling a stret function or not in a 64-bit watchOS simulator.

* [xharness] Re-enable some watchOS tests.

Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com>

New commits in mono/mono:

* mono/mono@ac596375c76 Add support for OP_FCONV_TO_I to mini-arm64.c. (dotnet#20548)
* mono/mono@392fe5b87b2 [2020-02][watchOS] Add simwatch64 support (dotnet#20552)
* mono/mono@a22ed3f094e Fix potential crash for Encoder.Convert (dotnet#20522)
* mono/mono@970783731fc Bump to F# 5.0 (dotnet#20511)
* mono/mono@32ab5066f72 Bump msbuild to fix a build issue
* mono/mono@93a7fe77e8e Ensure special static slots respect alignment. (dotnet#20506)
* mono/mono@3db5b358413 [debugger] Switch to GC Unsafe in signal handler callbacks (dotnet#20495)
* mono/mono@af315f44c40 [2020-02][corlib] ThreadAbortException protection for ArraySortHelper (dotnet#20468)
* mono/mono@ca11fb0fd81 [2020-02] Bump ikvm-fork to include mono/ikvm-fork#20 (dotnet#20452)

Diff: https://github.com/mono/mono/compare/be2226b5a1c57df065efc4c1cf008d581e5cec7d..ac596375c762c6b8dbe3c802f0ce626004eab51c
dalexsoto added a commit that referenced this pull request Dec 8, 2020
[Xcode12.3] [watchOS] Add x86_64 simulator support (#10059)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

note-highlight Worth calling out specifically in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[xcode12.2] watchOS apps don't deploy to watchOS 7.1 simulators

7 participants