Added sudo. #33

Merged
merged 1 commit into from Jan 4, 2017

Projects

None yet

3 participants

@bmustiata
Contributor

Just sudo so people can instantiate asciidoctor in the container as the user they are running their stuff.

E.g. a script such as asciidoctor.sh is impossible to write:

PROJECT_FOLDER=$(pwd)

docker run --rm \
    -v $PROJECT_FOLDER:/documents \
    -v /etc/passwd:/etc/passwd:ro \
    -v /etc/group:/etc/group:ro \
    asciidoctor/docker-asciidoctor \
    /usr/bin/sudo -E -u "#$(id -u)" "/usr/local/bin/asciidoctor" "$@"
@mojavelinux
Member

Seems reasonable to me. @gscheibel, any comments?

@gscheibel gscheibel merged commit 81918cd into asciidoctor:master Jan 4, 2017
@gscheibel
Contributor

Thanks @bmustiata for the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment