Skip to content

Commit 5b545b5

Browse files
Aaron FioreSomberNight
authored andcommitted
contrib: build-{linux,wine}: continue fixing build user env
A continuation of 99b0df0, as requested in spesmilo#10486.
1 parent bee773d commit 5b545b5

3 files changed

Lines changed: 24 additions & 25 deletions

File tree

contrib/build-linux/appimage/Dockerfile

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,13 @@ RUN apt-get update -q && \
7575
apt-get clean
7676

7777
# create new user to avoid using root; but with sudo access and no password for convenience.
78-
ARG UID=1000
79-
RUN if [ "$UID" != "0" ] ; then useradd --uid $UID --create-home --shell /bin/bash "user" ; fi
80-
RUN usermod -append --groups sudo $(id -nu $UID || echo "user")
78+
# NOTE: UID *MUST* align with buildozer's Dockerfile "user"
79+
RUN useradd -u 1000 -m -s /usr/bin/bash -d /home/user user
80+
RUN usermod -aG sudo user
8181
RUN echo "%sudo ALL=(ALL) NOPASSWD: ALL" >> /etc/sudoers
82-
RUN HOME_DIR=$(getent passwd $UID | cut -d: -f6)
83-
ENV WORK_DIR="${HOME_DIR}/wspace" \
84-
PATH="${HOME_DIR}/.local/bin:${PATH}"
82+
ENV HOME_DIR=/home/user
83+
ENV WORK_DIR="${HOME_DIR}/wspace"
84+
ENV PATH="${HOME_DIR}/.local/bin:${PATH}"
8585
WORKDIR ${WORK_DIR}
86-
RUN chown --recursive ${UID} ${WORK_DIR}
87-
USER ${UID}
86+
RUN chown -R user ${WORK_DIR}
87+
USER user

contrib/build-linux/sdist/Dockerfile

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,13 @@ RUN apt-get update -q && \
1717
apt-get clean
1818

1919
# create new user to avoid using root; but with sudo access and no password for convenience.
20-
ARG UID=1000
21-
RUN if [ "$UID" != "0" ] ; then useradd --uid $UID --create-home --shell /bin/bash "user" ; fi
22-
RUN usermod -append --groups sudo $(id -nu $UID || echo "user")
20+
# NOTE: UID *MUST* align with buildozer's Dockerfile "user"
21+
RUN useradd -u 1000 -m -s /usr/bin/bash -d /home/user user
22+
RUN usermod -aG sudo user
2323
RUN echo "%sudo ALL=(ALL) NOPASSWD: ALL" >> /etc/sudoers
24-
RUN HOME_DIR=$(getent passwd $UID | cut -d: -f6)
25-
ENV WORK_DIR="${HOME_DIR}/wspace" \
26-
PATH="${HOME_DIR}/.local/bin:${PATH}"
24+
ENV HOME_DIR=/home/user
25+
ENV WORK_DIR="${HOME_DIR}/wspace"
26+
ENV PATH="${HOME_DIR}/.local/bin:${PATH}"
2727
WORKDIR ${WORK_DIR}
28-
RUN chown --recursive ${UID} ${WORK_DIR}
29-
USER ${UID}
28+
RUN chown -R user ${WORK_DIR}
29+
USER user

contrib/build-wine/Dockerfile

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -57,16 +57,15 @@ RUN DEBIAN_CODENAME=$(lsb_release --codename --short) && \
5757
apt-get clean
5858

5959
# create new user to avoid using root; but with sudo access and no password for convenience.
60-
ARG UID=1000
61-
RUN if [ "$UID" != "0" ] ; then useradd --uid $UID --create-home --shell /bin/bash "user" ; fi
62-
RUN usermod -append --groups sudo $(id -nu $UID || echo "user")
60+
# NOTE: UID *MUST* align with buildozer's Dockerfile "user"
61+
RUN useradd -u 1000 -m -s /usr/bin/bash -d /home/user user
62+
RUN usermod -aG sudo user
6363
RUN echo "%sudo ALL=(ALL) NOPASSWD: ALL" >> /etc/sudoers
64-
RUN HOME_DIR=$(getent passwd $UID | cut -d: -f6)
65-
ENV WORK_DIR="${HOME_DIR}/wspace" \
66-
PATH="${HOME_DIR}/.local/bin:${PATH}"
64+
ENV HOME_DIR=/home/user
65+
ENV WORK_DIR="${HOME_DIR}/wspace"
66+
ENV PATH="${HOME_DIR}/.local/bin:${PATH}"
6767
WORKDIR ${WORK_DIR}
68-
RUN chown --recursive ${UID} ${WORK_DIR}
69-
RUN chown ${UID} /opt
70-
USER ${UID}
68+
RUN chown -R user ${WORK_DIR} /opt
69+
USER user
7170

7271
RUN mkdir --parents "/opt/wine64/drive_c/electrum"

0 commit comments

Comments
 (0)