-
Notifications
You must be signed in to change notification settings - Fork 532
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
Dev Container: Get 'debian-docker.sh' from github, fix nvm, prune already installed #3524
Conversation
@@ -2,17 +2,17 @@ | |||
# export PATH=$HOME/bin:/usr/local/bin:$PATH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except as noted, all .zshrc changes are due to having used an old version of 'oh-my-zsh'.
if [ "$(stat -c '%U' $NVM_DIR)" != "node" ]; then | ||
sudo chown -R node:root $NVM_DIR | ||
fi | ||
if [ "$(stat -c '%U' /usr/local/share/npm-global)" != "node" ]; then sudo chown -R 1000:root /usr/local/share/npm-global /usr/local/share/nvm; fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This missing bit is required for NVM
FROM mcr.microsoft.com/vscode/devcontainers/javascript-node:${NODE_VERSION} | ||
|
||
# Avoid warnings by switching to noninteractive | ||
ENV DEBIAN_FRONTEND=noninteractive | ||
|
||
COPY library-scripts/*.sh /tmp/library-scripts/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We now get the 'debian-docker.sh' install script from GitHub instead of checking it in.
# https://jilles.me/badassify-your-terminal-and-shell/ | ||
# https://billgrant.io/post/2019-05-05-vsremote/ | ||
&& apt-get install -y zsh \ | ||
wget \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of the above has already been done in the base image. We don't need to do it again.
git \ | ||
curl \ | ||
g++ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also already done in the debian base image. (Uses packages and everything.)
Thanks, @tanviraumi |
A few more tweaks to our dev container...