RoboStack / ros-jazzy

Recipes and patches for ros-jazzy conda packages
4 stars 2 forks source link

testpr: use rattler-build. #8

Open oursland opened 4 days ago

Tobias-Fischer commented 3 days ago

Hi @wolfv @baszalmstra - some exciting stuff happening here :).

There are some issues in switching from boa to pixi/rattler-build. It seems like the created package for some reason contains the whole environment, and .git folder etc. - how do we avoid this? The CI runner is also running out of space super quickly, any idea why?

oursland commented 3 days ago

I believe this is because the current uploaded packages are non-existent. linux_64 correctly identifies the packages to be built are already built and skips them, completing successfully. macos_arm64 finds many packages, but tries to build the others, and the other platforms have no uploaded artifacts.

Tobias-Fischer commented 3 days ago

I am happy with the list of package that need to be built, but not happy with the contents of the packages :)

traversaro commented 1 day ago

There are some issues in switching from boa to pixi/rattler-build. It seems like the created package for some reason contains the whole environment, and .git folder etc. - how do we avoid this? The CI runner is also running out of space super quickly, any idea why?

I experienced this as well! The issue is https://github.com/prefix-dev/rattler-build/issues/981 . Quoting @wolfv :

You can easily work around this by building outside the recipe folder or by setting a different output directory explicitly.

traversaro commented 1 day ago

I attempted a fix in https://github.com/oursland/ros-jazzy/pull/2, not sure if it works.