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

Typo found: libnss_wrapper.sh #87

Closed
mikegwhit opened this issue Jun 16, 2018 · 2 comments
Closed

Typo found: libnss_wrapper.sh #87

mikegwhit opened this issue Jun 16, 2018 · 2 comments

Comments

@mikegwhit
Copy link

Line 10 contains "souce" instead of "source" in echo statement.

#!/usr/bin/env bash
### every exit != 0 fails the script
set -e

echo "$bitsass Install nss-wrapper to be able to execute image as non-root user"
apt-get update 
apt-get install -y libnss-wrapper gettext
apt-get clean -y

echo "add 'souce generate_container_user' to .bashrc"

# have to be added to hold all env vars correctly
echo 'source $STARTUPDIR/generate_container_user' >> $HOME/.bashrc
@mikegwhit
Copy link
Author

Another interesting item is that generate_container_user is called and yet this seems to be a logic error. The Dockerfile stanza adding generate_container_user is executed AFTER this script is run.

toschneck added a commit that referenced this issue Jul 13, 2018
@toschneck
Copy link
Contributor

@mikegwhit thx for the hint. I fixed the typo. yeah the script generate_contaienr_user will run at the container startup, so you able to specify the USER var/id at run time like docker run -u 1000 ... will do

t-matsuo pushed a commit to t-matsuo/docker-centos-xfce-ja that referenced this issue Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants