Skip to content

Conversation

@ewdurbin
Copy link
Member

a release of orjson caused make deps to start failing unexpectedly with:

+ pip-compile --upgrade -o /tmp/tmp.VM3uXJx64Y/main.txt requirements/main.in
    error: subprocess-exited-with-error

    × Preparing metadata (pyproject.toml) did not run successfully.
    │ exit code: 1
    ╰─> [6 lines of output]
        Checking for Rust toolchain....

        Cargo, the Rust package manager, is not installed or is not on PATH.
        This package requires Rust and Cargo to compile extensions. Install it through
        the system's package manager or via https://rustup.rs/

        [end of output]

    note: This error originates from a subprocess, and is likely not a problem with pip.

This adds the rust toolchain to our build intermediate container, and development container so that builds and dependency resolution can utilize it.

a release of orjson caused `make deps` to start [failing unexpectedly](https://github.com/pypi/warehouse/actions/runs/12845424159/job/35819512078?pr=17451) with:

```
+ pip-compile --upgrade -o /tmp/tmp.VM3uXJx64Y/main.txt requirements/main.in
    error: subprocess-exited-with-error

    × Preparing metadata (pyproject.toml) did not run successfully.
    │ exit code: 1
    ╰─> [6 lines of output]
        Checking for Rust toolchain....

        Cargo, the Rust package manager, is not installed or is not on PATH.
        This package requires Rust and Cargo to compile extensions. Install it through
        the system's package manager or via https://rustup.rs/

        [end of output]

    note: This error originates from a subprocess, and is likely not a problem with pip.
```

This adds the rust toolchain to our build intermediate container, and development container so that builds and dependency resolution can utilize it.
@ewdurbin ewdurbin requested a review from a team as a code owner January 18, 2025 16:16
@ewdurbin ewdurbin merged commit 2c64e63 into main Jan 18, 2025
20 checks passed
@ewdurbin ewdurbin deleted the rust_toolchain branch January 18, 2025 16:29
@miketheman
Copy link
Member

See also ijl/orjson#548

@di
Copy link
Member

di commented Jan 21, 2025

Per ijl/orjson#548, wheels are now be available for this, should we revert?

di added a commit to di/warehouse that referenced this pull request Jan 21, 2025
di added a commit that referenced this pull request Jan 21, 2025
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.

4 participants