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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pixi/0.39.4 package update #38302

Merged
merged 2 commits into from
Dec 25, 2024

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Dec 24, 2024

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr labels Dec 24, 2024
@octo-sts octo-sts bot mentioned this pull request Dec 24, 2024
Copy link
Contributor Author

octo-sts bot commented Dec 24, 2024

Gen AI suggestions to solve the build error:

Let me analyze this build error and provide specific recommendations.

• Detected Error:

error: failed to select a version for `quinn-proto`.
    ... required by package `quinn v0.11.6`
    ... which satisfies dependency `reqwest = "^0.12.9"` (locked to 0.12.9) of package `pixi v0.39.4`

• Error Category: Dependency

• Failure Point: During the rust/cargobump step when trying to resolve dependencies

• Root Cause Analysis: There's a version conflict between quinn-proto and quinn packages. Specifically, quinn v0.11.6 requires quinn-proto with the rustls-ring feature, but the available version (0.11.7) doesn't have this feature.

• Suggested Fix:

  1. Update the package.yaml to pin specific versions:
environment:
  contents:
    packages:
      - build-base
      - busybox
      - ca-certificates-bundle
      - cargo-auditable
      - openssl
      - openssl-dev
      - perl
      - rust>=1.83.0
      - wolfi-base

pipeline:
  - uses: git-checkout
    with:
      repository: https://github.com/prefix-dev/pixi
      expected-commit: a343eef1d6b6be5e620a9dc4f156bc3ad837d40a
      tag: v${{package.version}}

  - name: Pre-configure
    runs: |
      cargo update -p quinn-proto --precise 0.11.6
      cargo update -p quinn --precise 0.11.6

  - uses: rust/cargobump

  - name: Configure and build
    runs: |
      cargo auditable build --release
      mkdir -p ${{targets.destdir}}/usr/bin/
      mv target/release/pixi ${{targets.destdir}}/usr/bin/

• Explanation: The error occurs because newer versions of quinn-proto removed the rustls-ring feature. By explicitly pinning the versions of both quinn and quinn-proto to compatible versions, we ensure the build can proceed with matching feature sets.

• Additional Notes:

  • This is a common issue when dealing with Rust ecosystem updates where features get reorganized
  • The fix ensures compatibility while maintaining security by using recent stable versions
  • An alternative approach would be to upgrade quinn to a newer version that doesn't require the rustls-ring feature

• References:

Signed-off-by: Batuhan Apaydin <[email protected]>
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Dec 25, 2024
Copy link
Member

@debasishbsws debasishbsws left a comment

Choose a reason for hiding this comment

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

🤔 Cargobump automation should cleaning itself

@debasishbsws debasishbsws merged commit a35cd74 into main Dec 25, 2024
14 checks passed
@debasishbsws debasishbsws deleted the wolfictl-3a18c891-d1f5-4fab-8cc8-47765840cb9e branch December 25, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. request-version-update request for a newer version of a package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants