Skip to content

apple: make the in-repo Swift iOS fixture build on macOS arm64 (part of #6597) - #7000

Merged
TrapsterDK merged 5 commits into
mainfrom
swift-ios-evidence-6597
Nov 5, 2026
Merged

TrapsterDK merged 5 commits into
mainfrom
swift-ios-evidence-6597

Conversation

@TrapsterDK

@TrapsterDK TrapsterDK commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

Part of #6597. Follows #6900.

macOS evidence found the fixture never analysed. Fixes:

  • apple/ios/app.bzl: the test selection rule's deps attr was a string_list, which the rules_apple framework aspect walked and crashed on. Renamed to dep_labels.
  • Fixture Rust, C and Swift targets compatible with iOS and macOS arm64 (APPLE_HOSTS) so they build under the simulator platform; libraries are manual so the wildcard does not build them for macOS.
  • export_simulator uses the fixture simulator app (the old target failed analysis on macOS).
  • Device app, mobileprovision and build_test removed: the device build needs the aarch64-apple-ios Rust target, not registered.

Evidence runs (macos_arm64, scope //apple/tests/fixtures/swift_app/... //apple/ios/...): run 41989737419 built the app and ran ledger_app_test (XCTest on the simulator) PASSED in 407s; run 41992872663 ledger_ipa_test PASSED (also fixed check_ipa.sh to read the .ipa next to the app executable). Nested-Bazel tests not run locally; Linux scoped test passes.

Remaining: device/arm64 app (aarch64-apple-ios triple, MODULE.bazel.lock), XCTest case names/executor.txt, negative failure_tests for xcode_version 9.9, support matrix cells.

Assisted by Claude.

🤖 Generated with Claude Code

https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK

TrapsterDK and others added 5 commits November 5, 2026 07:55
…_apple aspect skips them (part of #6597)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
… test uses a valid simulator app (part of #6597)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
…atform (part of #6597)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
…orm (part of #6597)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
…OS device Rust target (part of #6597)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
@TrapsterDK
TrapsterDK merged commit 6f5b9b3 into main Nov 5, 2026
1 of 2 checks passed
@TrapsterDK
TrapsterDK deleted the swift-ios-evidence-6597 branch November 5, 2026 07:51
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.

1 participant