Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requirement: update orjson #8487

Merged
merged 1 commit into from Sep 14, 2021
Merged

Requirement: update orjson #8487

merged 1 commit into from Sep 14, 2021

Conversation

humitos
Copy link
Member

@humitos humitos commented Sep 14, 2021

We had it pinned because it made Travis to fail, but we are on CircleCI now.
Let's see if we can make it work with the latest version of it.

We need to update it because cargo is failing to install this package due that
it doesn't find some dependencies.

Click for detailed output
Collecting orjson==2.0.7
  Using cached orjson-2.0.7.tar.gz (480 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... error
    ERROR: Command errored out with exit status 1:
     command: /tmp/env/bin/python3 /tmp/env/lib/python3.9/site-packages/pip/_vendor/pep517/_in_process.py prepare_metadata_for_build_wheel /tmp/tmpfqrzpjwz
         cwd: /tmp/pip-install-g2bmyvoy/orjson
    Complete output (43 lines):
    info: syncing channel updates for 'nightly-x86_64-unknown-linux-gnu'
    info: latest update on 2021-09-14, rust version 1.57.0-nightly (9bb77da74 2021-09-13)
    info: downloading component 'cargo'
    info: downloading component 'clippy'
    info: downloading component 'rust-docs'
    info: downloading component 'rust-std'
    info: downloading component 'rustc'
    info: downloading component 'rustfmt'
    info: installing component 'cargo'
    info: installing component 'clippy'
    info: installing component 'rust-docs'
    info: installing component 'rust-std'
    info: installing component 'rustc'
    info: installing component 'rustfmt'
    馃挜 pyo3-pack failed
      Caused by: Cargo metadata failed: Error during execution of `cargo metadata`:     Updating crates.io index
        Updating git repository `https://github.com/PyO3/pyo3.git`
        Updating git repository `https://github.com/ijl/json.git`
    error: failed to get `serde_json` as a dependency of package `orjson v2.0.7 (/tmp/pip-install-g2bmyvoy/orjson)`
    
    Caused by:
      failed to load source for dependency `serde_json`
    
    Caused by:
      Unable to update https://github.com/ijl/json.git?rev=2c0c55b628c9177543283a1535549e1dcb6a870a
    
    Caused by:
      failed to clone into: /home/humitos/.asdf/installs/rust/1.54.0/git/db/json-240b451de80eb693
    
    Caused by:
      failed to authenticate when downloading repository
    
      * attempted to find username/password via git's `credential.helper` support, but failed
    
      if the git CLI succeeds then `net.git-fetch-with-cli` may help here
      https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli
    
    Caused by:
      failed to acquire username/password from local configuration
    
    Checking for Rust toolchain....
    Running `pyo3-pack pep517 write-dist-info --metadata-directory /tmp/pip-modern-metadata-ncn2r3wa --manylinux=off --rustc-extra-args=-C target-feature=+sse2 --strip=on`
    Error: Command '['pyo3-pack', 'pep517', 'write-dist-info', '--metadata-directory', '/tmp/pip-modern-metadata-ncn2r3wa', '--manylinux=off', '--rustc-extra-args=-C target-feature=+sse2', '--strip=on']' returned non-zero exit status 1.
    ----------------------------------------
ERROR: Command errored out with exit status 1: /tmp/env/bin/python3 /tmp/env/lib/python3.9/site-packages/pip/_vendor/pep517/_in_process.py prepare_metadata_for_build_wheel /tmp/tmpfqrzpjwz Check the logs for full command output.

@humitos humitos requested a review from a team September 14, 2021 10:00
We had it pinned because it made Travis to fail, but we are on CircleCI now.
Let's see if we can make it work with the latest version of it.

We need to update it because `cargo` is failing to install this package due that
it doesn't find some dependencies.
@humitos
Copy link
Member Author

humitos commented Sep 14, 2021

Yay! Tests are passing 馃コ

@humitos humitos merged commit 4b627c5 into master Sep 14, 2021
@humitos humitos deleted the humitos/update-orjson branch September 14, 2021 15:37
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.

None yet

2 participants