Skip to content

Conversation

@hadim
Copy link
Contributor

@hadim hadim commented Jun 6, 2025

This is a simple POC adding support for pixi build. The motivation is to be able to simply and quickly test a custom rattler-build binary within a conda-forge feedstock.

With this PR you can now tweak the pixi.toml of any feedstock that way:

@@ -3,20 +3,22 @@
 # -*- mode: toml -*-
 
 #                             VVVVVV  minimum `pixi` version
-"$schema" = "https://pixi.sh/v0.36.0/schema/manifest/schema.json"
+# "$schema" = "https://pixi.sh/v0.36.0/schema/manifest/schema.json"
 
-[project]
+[workspace]
 name = "katago-feedstock"
 version = "3.50.0"  # conda-smithy version used to generate this file
 description = "Pixi configuration for conda-forge/katago-feedstock"
 authors = ["@conda-forge/katago"]
 channels = ["conda-forge"]
-platforms = ["linux-64", "osx-64", "win-64"]
+platforms = ["linux-64", "osx-64", "win-64", "osx-arm64"]
+preview = ["pixi-build"]
 
 [dependencies]
 conda-build = ">=24.1"
 conda-forge-ci-setup = "4.*"
-rattler-build = "*"
+# rattler-build = "*"
+rattler-build = { git = "https://github.com/hadim/rattler-build.git", branch = "pixi_build" }
 
 [tasks]
 [tasks.inspect-all]

Simply set the location of rattler-build repo or URL you want to test with rattler-build = { git = "https://github.com/hadim/rattler-build.git", branch = "pixi_build" } and then the c-f feedstock will build it and use it during the c-f builds on the c-f CIs.


This POC is leveraging the pixi-build-rattler-build backend but moving forward it might make sense to use pixi-build-rust instead.

@hadim hadim changed the title Add support forpixi build Add support for pixi build Jun 6, 2025
@hadim hadim mentioned this pull request Jun 6, 2025
@wolfv
Copy link
Member

wolfv commented Jun 6, 2025

Cool! I also wanted to try this out :)

@hadim
Copy link
Contributor Author

hadim commented Jun 6, 2025

I am done here as it is a simple POC. Please let me know if you would like to review and merge, or I can also close if you prefer to take care of it later.

@hadim hadim marked this pull request as ready for review June 21, 2025 18:07
@wolfv wolfv merged commit 770f4b4 into prefix-dev:main Jun 23, 2025
6 checks passed
@wolfv
Copy link
Member

wolfv commented Jun 23, 2025

Awesome! Sorry for letting it linger :)

@lucascolley
Copy link
Contributor

it seems like things still work for me switching to pixi-build-rust instead — should we do that? Maybe there are edge-cases here that wouldn't be covered?

@wolfv
Copy link
Member

wolfv commented Jun 30, 2025

Would be great to move to the Rust backend, indeed.

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.

3 participants