Skip to content

fix: processor architecture selector for Linux#355

Merged
buenaflor merged 6 commits intogetsentry:mainfrom
smihica:fix-processor-architecture-linux
Mar 24, 2026
Merged

fix: processor architecture selector for Linux#355
buenaflor merged 6 commits intogetsentry:mainfrom
smihica:fix-processor-architecture-linux

Conversation

@smihica
Copy link
Copy Markdown
Contributor

@smihica smihica commented Sep 2, 2025

📜 Description

The processor architecture in the relevant section was as follows,

switch (SysInfo.kernelArchitecture.name.toLowerCase()) {
  case 'arm':
  case 'armv6':
  case 'armv7':
    return HostPlatform.linuxArmv7;
  case 'aarch64':
    return HostPlatform.linuxAarch64;
  case 'amd64':
  case 'x86_64':
    return HostPlatform.linux64bit;

but the SysInfo.kernelArchitecture.name returns different values.
See here => https://github.com/onepub-dev/system_info/blob/master/lib/src/processor_architecture.dart

My aarch64 linux was failing to run by this bug.

$ dart run sentry_dart_plugin
[❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚] 0% reading config values..        
Searching for pubspec.yaml or sentry.properties config...
Found config from pubspec.yaml
Failed to download Sentry CLI: Exception: Host platform not supported: linux ARM64

↑ The name was ARM64

This fixes the problem.

💡 Motivation and Context

To release my app.

💚 How did you test it?

Run in my environment.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated the docs if needed
  • All tests passing
  • No breaking changes

🔮 Next steps

@smihica smihica changed the title Fixed processor architectures in Linux BugFix: processor architecture selector for Linux Sep 3, 2025
Copy link
Copy Markdown
Contributor

@buenaflor buenaflor left a comment

Choose a reason for hiding this comment

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

thanks! looks good to me

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 27, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • Processor architecture selector for Linux by smihica in #355

Internal Changes 🔧

Deps

  • Bump actions/create-github-app-token from 2.0.6 to 2.1.4 by dependabot in #356
  • Bump actions/upload-artifact from 4 to 5 by dependabot in #365
  • Bump actions/setup-java from 4 to 5 by dependabot in #353
  • Bump actions/checkout from 4 to 5 by dependabot in #350

Release

  • Fix changelog-preview permissions by BYK in #373
  • Switch from action-prepare-release to Craft by BYK in #371

Other

  • Use pull_request_target for changelog preview by BYK in #372

Other

  • internal: Instrument plugin with Sentry tracing & error by buenaflor in #381

🤖 This preview updates automatically when you update the PR.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@buenaflor buenaflor requested review from denrase and removed request for stefanosiano March 24, 2026 08:51
@buenaflor buenaflor changed the title BugFix: processor architecture selector for Linux fix: processor architecture selector for Linux Mar 24, 2026
denrase added 3 commits March 24, 2026 09:56
Removed internal section detailing Sentry tracing instrumentation and its opt-out option.
@buenaflor
Copy link
Copy Markdown
Contributor

warden workflow fails since this is a fork but PR looks good. will be merging and have warden run on the base branch again

@buenaflor buenaflor merged commit 6945016 into getsentry:main Mar 24, 2026
20 of 23 checks passed
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.

3 participants