Repository navigation
build(deps): upgrade libc and fix iOS compatibility - #9359
youknowdot wants to merge 5 commits into
Conversation
Bumps [libc](https://github.com/rust-lang/libc) from 0.2.189 to 0.2.191. - [Release notes](https://github.com/rust-lang/libc/releases) - [Changelog](https://github.com/rust-lang/libc/blob/0.2.191/CHANGELOG.md) - [Commits](rust-lang/libc@0.2.189...0.2.191) --- updated-dependencies: - dependency-name: libc dependency-version: 0.2.191 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Pin nix to the exact upstream macOS-only API fix until it is released. Update the cfg_aliases minimum required by that revision. Assisted-by: OpenAI Codex:model version unavailable
Preserve the Dependabot upgrade and iOS fix while incorporating upstream actionlint and mach2 dependency updates. Assisted-by: OpenAI Codex:model version unavailable
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
System API availability Cargo.toml, crates/host_env/src/os.rs, crates/vm/src/stdlib/os.rs |
The nix dependency now uses a specified Git revision. The system APIs are excluded on iOS, tvOS, watchOS, and visionOS. The ToCString import is local to the function. |
Socket constant availability crates/host_env/src/socket.rs, crates/stdlib/src/socket.rs |
SYSPROTO_CONTROL is now restricted to macOS. The other listed Apple socket constants retain their Apple-target availability. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Bug fix
Suggested reviewers: youknowone, shaharnaveh
Merge Risk: 
Minimal · up to 3d628
-
No concrete current-head compatibility or dependency-resolution risk was established.
Pre-merge checks: 
4 · 
1
-
Failed checks: 1 warning Passed checks: 4 passedFull details: Docstring Coverage
-
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 4 files. (1 skipped: 1 unsupported.)
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Expose SYSPROTO_CONTROL only on macOS and omit os.system on Apple mobile targets where libc does not provide it. Keep host and Python exports consistent with the platform declarations and CPython availability. Assisted-by: OpenAI Codex:model version unavailable
Migrate Lib/_ios_support.py through scripts/update_lib to CPython 3.15 commit d220409a13b77de7a6c30abe16ea90db01528edc. Fall back to the bare libobjc.dylib name when find_library cannot resolve the shared-cache entry, and normalize loader OSError to ImportError. Linux default and frozen-stdlib cargo check gates are compile-only. They do not validate iOS dynamic loading, Objective-C symbols, or native runtime. Assisted-by: OpenAI Codex:model version unavailable
📦 Library DependenciesThe following Lib/ modules were modified. Here are their dependencies: [x] lib: cpython/Lib/_ios_support.py dependencies:
dependent tests: (30 tests) Legend:
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
|
||
| [patch.crates-io] | ||
| # Keep libc 0.2.191 compatible with iOS until nix releases https://github.com/nix-rust/nix/pull/2821. | ||
| nix = { git = "https://github.com/nix-rust/nix", rev = "f9ba3ed0698255d93240b07ecbf39ecb05a8a5cd" } |
There was a problem hiding this comment.
not a good idea. wait new release
Summary
Continue #9240's libc 0.2.189 → 0.2.191 upgrade, preserving the original Dependabot commit and attribution.
No fake platform APIs or new test skips are introduced.
Validation
At head 3d6287b:
The hosted iOS result is a cross-compilation check, not an iOS device or simulator runtime test.
AI assistance: OpenAI Codex, disclosed in the commit trailers.
Summary by CodeRabbit
SYSPROTO_CONTROLsocket attribute to macOS; other supported Apple socket attributes remain available on Apple platforms.