Skip to content

$PATH got overwritten in docker #4758

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

Closed
chunjiw opened this issue Jan 16, 2022 · 7 comments
Closed

$PATH got overwritten in docker #4758

chunjiw opened this issue Jan 16, 2022 · 7 comments
Milestone

Comments

@chunjiw
Copy link

chunjiw commented Jan 16, 2022

OS/Web Information

  • Web Browser: Chrome
  • Local OS: Arch Linux (Endeavour OS)
  • Remote OS: Debian
  • Remote Architecture: x86_64
  • code-server --version: 4.0.1

Steps to Reproduce

  1. Add code-server to julia docker
FROM julia:latest
# RUN curl -fOL https://github.com/cdr/code-server/releases/download/v3.12.0/code-server_3.12.0_amd64.deb; \
#    dpkg -i code-server_3.12.0_amd64.deb;\
#    rm code-server_3.12.0_amd64.deb;
RUN curl -fOL https://github.com/coder/code-server/releases/download/v4.0.1/code-server_4.0.1_amd64.deb; \
    dpkg -i code-server_4.0.1_amd64.deb;\
    rm code-server_4.0.1_amd64.deb;
  1. Start the docker
  2. Log in code server
  3. run echo $PATH

Expected

Output is the same in Julia docker /usr/local/julia/bin:...

Actual

/usr/local/julia/bin is missing from PATH. Instead, there is a code-server related path.

This issue can be reproduced in VS Code: No

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 18, 2022

Did this happen in 3.12.0?

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 18, 2022

cc @code-asher

@jsjoeio jsjoeio added the needs-investigation This issue needs to be further investigated label Jan 18, 2022
@chunjiw
Copy link
Author

chunjiw commented Jan 18, 2022

No, I've been using 3.12.0 with no problem

@jsjoeio jsjoeio added this to the Backlog milestone Jan 18, 2022
@code-asher
Copy link
Member

Perhaps the same as #4699

@chunjiw
Copy link
Author

chunjiw commented Jan 19, 2022

Yes, I think its same as #4699. Somehow I failed to notice #4699 before creating this issue..

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 19, 2022

@chunjiw no worries! I also failed to realize this was a duplicate. Let's consolidate then and track in #4699

@repo-ranger
Copy link
Contributor

repo-ranger bot commented Jan 19, 2022

⚠️ This issue has been marked duplicate and will be closed in 2 days.

@repo-ranger repo-ranger bot closed this as completed Jan 21, 2022
@code-asher code-asher removed the needs-investigation This issue needs to be further investigated label Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants