Skip to content

build on windows-11-arm#4187

Merged
bruno-garcia merged 35 commits intomainfrom
bruno-garcia-patch-1
May 22, 2025
Merged

build on windows-11-arm#4187
bruno-garcia merged 35 commits intomainfrom
bruno-garcia-patch-1

Conversation

@bruno-garcia
Copy link
Copy Markdown
Member

@bruno-garcia bruno-garcia commented May 13, 2025

Since I tried my luck with:
* getsentry/sentry-native#880

Blocked by:
* getsentry/sentry-native#880
Not blocked because sentry-native itself compiles on arm64 on Windows, the issue there is more about setting up the repo and integration tests.

A lot was figured out by @jpnurmi when adding musl Linux, so I just copied over

TODO:

  • Test with env.CI_PUBLISHING_BUILD set to true
  • Plugin the actual sentry-native arm64 build when available, for Native AOT support

@bruno-garcia bruno-garcia marked this pull request as draft May 17, 2025 21:17
@bruno-garcia bruno-garcia mentioned this pull request May 18, 2025
8 tasks
@bruno-garcia
Copy link
Copy Markdown
Member Author

/Users/runner/.dotnet/sdk/9.0.203/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(221,5): error : Could not find a part of the path '/Users/runner/work/sentry-dotnet/sentry-dotnet/src/Sentry/Platforms/Native/sentry-native/win-arm64'. [/Users/runner/work/sentry-dotnet/sentry-dotnet/src/Sentry/Sentry.csproj]

I think now the only thing left is building sentry-native for Windows arm64 which is being done on:

jpnurmi added 2 commits May 19, 2025 20:16
Downloads are separate steps because
macOS (CI_PUBLISHING_BUILD) needs to download both.
@jpnurmi
Copy link
Copy Markdown
Collaborator

jpnurmi commented May 19, 2025

Does actions/cache with enableCrossOsArchive not work on windows-11-arm?

  1. sentry-native build succeeds on windows-11-arm and it saves the cache.
  2. The cache artifact is visible at https://github.com/getsentry/sentry-dotnet/actions/caches
  3. .NET build for windows-11-arm manages to restore the cache.
  4. .NET build for macOS (CI_PUBLISHING_BUILD=true) does not find the same cache entry.

@bruno-garcia bruno-garcia merged commit d622ff2 into main May 22, 2025
31 checks passed
@bruno-garcia bruno-garcia deleted the bruno-garcia-patch-1 branch May 22, 2025 01:06
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.

4 participants