Skip to content

[7.x] Fix rules_apple device runner crash on Rapid Security Response versions#2902

Merged
keith merged 1 commit intobazel/7.xfrom
ks/7.x-fix-rules_apple-device-runner-crash-on-rapid-security-response-versions-2898
Mar 26, 2026
Merged

[7.x] Fix rules_apple device runner crash on Rapid Security Response versions#2902
keith merged 1 commit intobazel/7.xfrom
ks/7.x-fix-rules_apple-device-runner-crash-on-rapid-security-response-versions-2898

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented Mar 26, 2026

Devices running an Apple RSR update report OS versions like "18.3.1
(a)". The rules_apple device runner fails to parse the "(a)" suffix:

  ValueError: invalid literal for int() with base 10: '1 (a)'

Patch rules_apple 4.5.1 to strip non-numeric suffixes from each version
component before integer conversion.

bazel run an iOS application target on a physical device (builds,
uploads, and runs)

Fixes #2900

(cherry picked from commit 8c999a8)

…versions (#2898)

Devices running an Apple RSR update report OS versions like "18.3.1
(a)". The rules_apple device runner fails to parse the "(a)" suffix:

```
  ValueError: invalid literal for int() with base 10: '1 (a)'
```

Patch rules_apple 4.5.1 to strip non-numeric suffixes from each version
component before integer conversion.

`bazel run` an iOS application target on a physical device (builds,
uploads, and runs)

Fixes #2900

(cherry picked from commit 8c999a8)
@keith keith changed the title [7.x] Fix rules_apple device runner crash on Rapid Security Response versions (#2898) [7.x] Fix rules_apple device runner crash on Rapid Security Response versions Mar 26, 2026
@keith keith enabled auto-merge (squash) March 26, 2026 16:26
@keith keith merged commit 32e0610 into bazel/7.x Mar 26, 2026
10 checks passed
@keith keith deleted the ks/7.x-fix-rules_apple-device-runner-crash-on-rapid-security-response-versions-2898 branch March 26, 2026 17:31
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.

5 participants