Skip to content
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.

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

Global binaries not available by default #63

Closed
hadim opened this issue Dec 12, 2024 · 3 comments
Closed

Global binaries not available by default #63

hadim opened this issue Dec 12, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@hadim
Copy link
Contributor

hadim commented Dec 12, 2024

$ docker run --rm -ti ghcr.io/prefix-dev/pixi bash -c "pixi global install rsync && rsync --version"
Global environments as specified in '/root/.pixi/manifests/pixi-global.toml'
└── rsync: 3.3.0 (installed)
    └─ exposes: rsync, rsync-ssl
bash: line 1: rsync: command not found

This is because /root/.pixi/bin/ is not in the PATH by default.

What are your thoughts on making the global binaries available by default in the docker image?

A legit use case is to quickly install tools and use them in the docker image. For example, the above command is actually much faster than using the ubuntu docker image with apt update && apt install rsync.

@hadim
Copy link
Contributor Author

hadim commented Dec 12, 2024

Eventually we could set export PIXI_HOME=/usr/local but this might have unwanted side effects.

@pavelzw
Copy link
Collaborator

pavelzw commented Dec 13, 2024

I think this makes sense. I don't think there is a way to append to env variables (i.e. export PATH="/root/.pixi/bin:$PATH") in Docker syntax, the only way is to overwrite it using ENV PATH=....
Before doing this, we should make sure we don't accidentally delete parts of the PATH of the base images (or add too much).

- debian:bookworm-slim # 12
- debian:bookworm # 12
- debian:bullseye-slim # 11
- debian:bullseye # 11
- ubuntu:noble # 24.04
- ubuntu:mantic # 23.10
- ubuntu:jammy # 22.04
- ubuntu:focal # 20.04
- nvidia/cuda:12.3.1-base-ubuntu22.04
- nvidia/cuda:12.3.1-base-ubuntu20.04
- nvidia/cuda:12.2.2-base-ubuntu22.04
- nvidia/cuda:12.2.2-base-ubuntu20.04
- nvidia/cuda:12.1.1-base-ubuntu22.04
- nvidia/cuda:12.1.1-base-ubuntu20.04
- nvidia/cuda:11.8.0-base-ubuntu22.04
- nvidia/cuda:11.8.0-base-ubuntu20.04
- nvidia/cuda:11.7.1-base-ubuntu22.04
- nvidia/cuda:11.7.1-base-ubuntu20.04
- nvidia/cuda:11.6.2-base-ubuntu20.04
- nvidia/cuda:11.4.3-base-ubuntu20.04
- nvidia/cuda:11.3.1-base-ubuntu20.04
- nvidia/cuda:11.2.2-base-ubuntu20.04

PRs are welcome

@pavelzw pavelzw added the enhancement New feature or request label Dec 13, 2024
@pavelzw
Copy link
Collaborator

pavelzw commented Dec 13, 2024

just saw this is a duplicate of #40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants