Skip to content

Conversation

@tdejager
Copy link
Contributor

@tdejager tdejager commented Oct 23, 2025

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

Rattler changes currently live at:

conda/rattler#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

@tdejager tdejager marked this pull request as draft October 23, 2025 13:02
@baszalmstra baszalmstra changed the title wip: upgrade to rattler lock v7 refactor: upgrade to rattler lock v7 Oct 24, 2025
@baszalmstra
Copy link
Contributor

baszalmstra commented Oct 24, 2025

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 ?

@tdejager
Copy link
Contributor Author

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!

@ruben-arts
Copy link
Contributor

@baszalmstra Bumping the version makes sense!

@lucascolley lucascolley added refactor Specifies PR or Issue is related to a refactor lock file labels Oct 29, 2025
@Hofer-Julian Hofer-Julian changed the base branch from main to bas/wip/develop-dependencies October 30, 2025 15:14
@Hofer-Julian Hofer-Julian changed the base branch from bas/wip/develop-dependencies to main October 31, 2025 09:41
@Hofer-Julian
Copy link
Contributor

Hofer-Julian commented Oct 31, 2025

I've updated the PR with current main. We need to properly review and user test this and fix this bug before merging it: prefix-dev/pixi-build-testsuite#88 (comment)

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 Hofer-Julian force-pushed the feat/upgrade-to-lock-v7 branch from f969ba9 to dbf92e8 Compare October 31, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lock file refactor Specifies PR or Issue is related to a refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants