Skip to content

Fetch pixi.toml directly from a git repository clone. #806

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.

Sign up for GitHub

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

Open
nuclearsandwich wants to merge 13 commits into
base: master
Choose a base branch
from
+1 −1
Open

Fetch pixi.toml directly from a git repository clone. #806

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
181e412
Fetch pixi.toml directly from a git repository clone.
nuclearsandwich Mar 21, 2025
4a89d02
What OS are we on?
nuclearsandwich Mar 21, 2025
ef6e301
Don't use /b as it makes it mad.
nuclearsandwich Mar 21, 2025
24c567e
Why can't Windows just be normal*
nuclearsandwich Mar 21, 2025
2843b30
Add curly braces in the suspicion that they affect variable expansion.
nuclearsandwich Mar 22, 2025
f85ff70
.git suffix is required to trigger git repository behavior.
nuclearsandwich Mar 22, 2025
17c13e6
findoutwtf
nuclearsandwich Mar 22, 2025
19b1e98
Could it be the variable expansion?
nuclearsandwich Mar 22, 2025
d574ef7
Fall back to githubusercontent for sanity's sake.
nuclearsandwich Mar 22, 2025
d1e1e0a
Try git repos again.
nuclearsandwich Mar 22, 2025
f4e62f8
Forgot to copy the file.
nuclearsandwich Mar 22, 2025
3841137
Revert "Forgot to copy the file."
nuclearsandwich Mar 22, 2025
2d98d8d
Revert "Try git repos again."
nuclearsandwich Mar 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion windows_docker_resources/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ RUN powershell -noexit "Set-ExecutionPolicy Bypass -Scope Process -Force; iex ((
# Install dependencies via pixi
ARG ROS_DISTRO=rolling
WORKDIR C:\pixi_ws
RUN powershell -noexit irm https://raw.githubusercontent.com/ros2/ros2/refs/heads/%ROS_DISTRO%/pixi.toml -OutFile pixi.toml
ADD https://raw.githubusercontent.com/ros2/ros2/${ROS_DISTRO}/pixi.toml pixi.toml
Copy link
Member Author

@nuclearsandwich nuclearsandwich Mar 22, 2025

Choose a reason for hiding this comment

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

Suggested change
ADD https://raw.githubusercontent.com/ros2/ros2/${ROS_DISTRO}/pixi.toml pixi.toml
ADD https://raw.githubusercontent.com/ros2/ros2/${ROS_DISTRO}/pixi.toml pixi.toml

Since this command is expanded by docker not CMD or powershell the sh-style variable expansion is what works.

RUN pixi --color never --no-progress -q install
RUN pixi --color never --no-progress -q list

Expand Down