Skip to content

Fix iOS detection#17973

Merged
konstin merged 2 commits intomainfrom
konsti/iphoneos
Feb 18, 2026
Merged

Fix iOS detection#17973
konstin merged 2 commits intomainfrom
konsti/iphoneos

Conversation

@konstin
Copy link
Member

@konstin konstin commented Feb 11, 2026

The fix is [_version, architecture, _platform] = version_arch.split("-"), but I also improved the general structure a bunch.

Fixes #16724 (comment)

@konstin
Fix iOS detection
a4ba570 
The fix is `[_version, architecture, _platform] = version_arch.split("-")`, but I also improved the general structure a bunch.
@konstin konstin added the bug Something isn't working label Feb 11, 2026
@konstin konstin mentioned this pull request Feb 11, 2026
Closed
zanieb
zanieb reviewed Feb 13, 2026
View reviewed changes
crates/uv-platform-tags/src/platform.rs
Comment on lines +14 to +18
#[error("Invalid Android architecture: {0}")]
InvalidAndroidArch(Arch),
#[error("Invalid iOS simulator architecture: {0}")]
InvalidIosSimulatorArch(Arch),
#[error("Invalid iOS device architecture: {0}")]
Copy link
Member

@zanieb zanieb Feb 13, 2026

Choose a reason for hiding this comment

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

I think I might expect InvalidArch(Platform, Arch) -> "Invalid {platform} architecture: ...

Copy link
Member

@zanieb zanieb Feb 13, 2026

Choose a reason for hiding this comment

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

I don't feel strongly though, it looks like there's not a convenient enum for "Platform" as they're each separate structs?

timrid
timrid reviewed Feb 14, 2026
View reviewed changes
crates/uv-platform-tags/src/tags.rs Outdated
)]
#[rkyv(derive(Debug))]
pub enum IosMultiarch {
pub enum IosPlatform {
Copy link
Contributor

@timrid timrid Feb 14, 2026

Choose a reason for hiding this comment

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

Why is the enum being renamed in IosPlatform?

In the official spec, only the terms arch_sdk or multiarch are used for the values defined in this enum. This comment describes in more detail why I used multiarch instead of arch_sdk. Therefore, in my opinion, it does not make sense to introduce the new term platform for this combination.

Copy link
Member Author

@konstin konstin Feb 18, 2026

Choose a reason for hiding this comment

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

This is a very confusing naming :/ Multiarch is a widely used term for having multiple architectures with or on the same configuration (https://help.ubuntu.com/community/MultiArch, https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/), not for arch + ABI.

@konstin konstin marked this pull request as ready for review February 18, 2026 10:22
@konstin
Copy link
Member Author

konstin commented Feb 18, 2026

Let's try this version, if there's something we missed we can do another fix.

@konstin konstin enabled auto-merge (squash) February 18, 2026 10:28
@konstin konstin merged commit baa977a into main Feb 18, 2026
128 checks passed
@konstin konstin deleted the konsti/iphoneos branch February 18, 2026 10:32
@BrewTestBot BrewTestBot mentioned this pull request Feb 24, 2026
Merged
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Feb 25, 2026
@tmeijn
build(deps): update uv to v0.10.6
8172ed9 
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [uv](https://github.com/astral-sh/uv) | patch | `0.10.4` → `0.10.6` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>astral-sh/uv (uv)</summary>

### [`v0.10.6`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0106)

[Compare Source](astral-sh/uv@0.10.5...0.10.6)

Released on 2026-02-24.

##### Bug fixes

- Apply lockfile marker normalization for fork markers ([#&#8203;18116](astral-sh/uv#18116))
- Fix Python version selection for scripts with a `requires-python` conflicting with `.python-version` ([#&#8203;18097](astral-sh/uv#18097))
- Preserve file permissions when using reflinks on Linux ([#&#8203;18187](astral-sh/uv#18187))

##### Documentation

- Remove verbose documentation from optional dependencies help text ([#&#8203;18180](astral-sh/uv#18180))

### [`v0.10.5`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#0105)

[Compare Source](astral-sh/uv@0.10.4...0.10.5)

Released on 2026-02-23.

##### Enhancements

- Add hint when named index is found in a parent config file ([#&#8203;18087](astral-sh/uv#18087))
- Add warning for `uv lock --frozen` ([#&#8203;17859](astral-sh/uv#17859))
- Attempt to use reflinks by default on Linux ([#&#8203;18117](astral-sh/uv#18117))
- Fallback to hardlinks after reflink failure before copying ([#&#8203;18104](astral-sh/uv#18104))
- Filter `pylock.toml` wheels by tags and `requires-python` ([#&#8203;18081](astral-sh/uv#18081))
- Validate wheel filenames are normalized during `uv publish` ([#&#8203;17783](astral-sh/uv#17783))
- Fix message when `exclude-newer` invalidates the lock file ([#&#8203;18100](astral-sh/uv#18100))
- Change the missing files log level to debug ([#&#8203;18075](astral-sh/uv#18075))

##### Performance

- Improve performance of repeated conflicts with an extra ([#&#8203;18094](astral-sh/uv#18094))

##### Bug fixes

- Fix `--no-emit-workspace` with `--all-packages` on single-member workspaces ([#&#8203;18098](astral-sh/uv#18098))
- Fix `UV_NO_DEFAULT_GROUPS` rejecting truthy values like `1` ([#&#8203;18057](astral-sh/uv#18057))
- Fix iOS detection ([#&#8203;17973](astral-sh/uv#17973))
- Propagate project-level conflicts to package extras ([#&#8203;18096](astral-sh/uv#18096))
- Use a global build concurrency semaphore ([#&#8203;18054](astral-sh/uv#18054))

##### Documentation

- Update documentation heading for environment variable files ([#&#8203;18122](astral-sh/uv#18122))
- Fix comment about `uv export` formats ([#&#8203;17900](astral-sh/uv#17900))
- Make it clear that Windows is supported in user- and system- level configuration docs ([#&#8203;18106](astral-sh/uv#18106))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4zMS4xIiwidXBkYXRlZEluVmVyIjoiNDMuMzEuOSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90IiwiYXV0b21hdGlvbjpib3QtYXV0aG9yZWQiLCJkZXBlbmRlbmN5LXR5cGU6OnBhdGNoIl19-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zanieb zanieb zanieb approved these changes

+1 more reviewer

@timrid timrid timrid left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

iOS support isn't working on 0.9.9

3 participants

@konstin @zanieb @timrid