pixi icon indicating copy to clipboard operation
pixi copied to clipboard

Published 20 hours ago • verified publisher icon prefix-dev

refactor: upgrade to rattler lock v7

Open tdejager opened this issue 1 month ago • 4 comments

Currently in progress of updating to rattler_lock v7 that includes the variants.

Rattler changes currently live at:

https://github.com/conda/rattler/pull/1775

Some notable changes:

  • Using purls is slightly broken for source packages. This is currently not used but is something we should fix down the line.
  • The python interpreter must be installed as a binary package atm.

Depends on

  • https://github.com/prefix-dev/pixi-build-backends/pull/438
  • https://github.com/prefix-dev/pixi-build-testsuite/pull/88

tdejager avatar Oct 23 '25 13:10 tdejager

To function properly, this requires: https://github.com/prefix-dev/pixi-build-backends/pull/438. Without that PR not all used variants are returned which then causes an error during serialization of the lock-file. We are also currently still using the pixi-build-api-version 2 which makes it backwards compatible but introduces some inefficiencies. Im considering bumping the pixi-build-api-version, removing the inefficiencies and also ensuring that the backwards work properly with the above change in pixi-build-backends integrated.

WDYT @ruben-arts @tdejager ?

baszalmstra avatar Oct 24 '25 12:10 baszalmstra

To function properly, this requires: prefix-dev/pixi-build-backends#438. Without that PR not all used variants are returned which then causes an error during serialization of the lock-file. We are also currently still using the pixi-build-api-version 2 which makes it backwards compatible but introduces some inefficiencies. Im considering bumping the pixi-build-api-version, removing the inefficiencies and also ensuring that the backwards work properly with the above change in pixi-build-backends integrated.

WDYT @ruben-arts @tdejager ?

Yes!

tdejager avatar Oct 24 '25 12:10 tdejager

@baszalmstra Bumping the version makes sense!

ruben-arts avatar Oct 24 '25 13:10 ruben-arts

I've updated the PR with current main. We need to properly review and user test this and fix this bug before merging it: https://github.com/prefix-dev/pixi-build-testsuite/pull/88#issuecomment-3473416720

On this branch this example shows the same problem:

pixi run test-slow lock_file::satisfiability::tests::test_example_satisfiability::manifest_path_16__UP__UP_examples_pixi_build_cpp_git_source_pixi_toml

Hofer-Julian avatar Oct 31 '25 14:10 Hofer-Julian