Skip to content

Commit d5f2c75

Browse files
committed
add maybe_sudo logic to install_prereqs scripts
1 parent 71a2702 commit d5f2c75

File tree

2 files changed

+27
-17
lines changed

2 files changed

+27
-17
lines changed

drake_pip/setup/install_prereqs

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,11 @@
3131

3232
set -euxo pipefail
3333

34+
maybe_sudo=
35+
if [[ "${EUID}" -ne 0 ]]; then
36+
maybe_sudo=sudo
37+
fi
38+
3439
case "$OSTYPE" in
3540
darwin*)
3641
# Mac specific installations
@@ -40,10 +45,10 @@ case "$OSTYPE" in
4045
fi
4146

4247
if [[ ! -d /opt/drake ]]; then
43-
sudo mkdir -p /opt/drake
44-
sudo chmod g+rwx /opt/drake
45-
sudo chown "${USER}" /opt/drake
46-
sudo chgrp admin /opt/drake
48+
${maybe_sudo} mkdir -p /opt/drake
49+
${maybe_sudo} chmod g+rwx /opt/drake
50+
${maybe_sudo} chown "${USER}" /opt/drake
51+
${maybe_sudo} chgrp admin /opt/drake
4752
fi
4853

4954
# Install Drake dependencies.
@@ -59,15 +64,15 @@ case "$OSTYPE" in
5964
exit 2
6065
fi
6166

62-
apt-get update
63-
apt-get install --no-install-recommends lsb-release
67+
${maybe_sudo} apt-get update
68+
${maybe_sudo} apt-get install --no-install-recommends lsb-release
6469

6570
if [[ "$(lsb_release -sc)" != 'jammy' ]]; then
6671
echo 'This script requires Ubuntu 22.04 (Jammy)' >&2
6772
exit 3
6873
fi
6974

70-
apt-get install --no-install-recommends $(cat <<EOF
75+
${maybe_sudo} apt-get install --no-install-recommends $(cat <<EOF
7176
ca-certificates
7277
wget
7378
EOF
@@ -78,8 +83,8 @@ EOF
7883
trap 'rm -f drake.tar.gz' EXIT
7984
tar -xf drake.tar.gz -C /opt
8085

81-
apt-get update
82-
apt-get install --no-install-recommends $(cat <<EOF
86+
${maybe_sudo} apt-get update
87+
${maybe_sudo} apt-get install --no-install-recommends $(cat <<EOF
8388
python3-all-dev
8489
python3.10-venv
8590
EOF

drake_poetry/setup/install_prereqs

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,11 @@
3131

3232
set -euxo pipefail
3333

34+
maybe_sudo=
35+
if [[ "${EUID}" -ne 0 ]]; then
36+
maybe_sudo=sudo
37+
fi
38+
3439
case "$OSTYPE" in
3540
darwin*)
3641
# Mac specific installations
@@ -40,10 +45,10 @@ case "$OSTYPE" in
4045
fi
4146

4247
if [[ ! -d /opt/drake ]]; then
43-
sudo mkdir -p /opt/drake
44-
sudo chmod g+rwx /opt/drake
45-
sudo chown "${USER}" /opt/drake
46-
sudo chgrp admin /opt/drake
48+
${maybe_sudo} mkdir -p /opt/drake
49+
${maybe_sudo} chmod g+rwx /opt/drake
50+
${maybe_sudo} chown "${USER}" /opt/drake
51+
${maybe_sudo} chgrp admin /opt/drake
4752
fi
4853

4954
# Install Drake dependencies.
@@ -62,8 +67,8 @@ case "$OSTYPE" in
6267
exit 2
6368
fi
6469

65-
apt-get update
66-
apt-get install --no-install-recommends lsb-release
70+
${maybe_sudo} apt-get update
71+
${maybe_sudo} apt-get install --no-install-recommends lsb-release
6772

6873
if [[ "$(lsb_release -sc)" != 'jammy' ]]; then
6974
echo 'This script requires Ubuntu 22.04 (Jammy)' >&2
@@ -81,8 +86,8 @@ EOF
8186
trap 'rm -f drake.tar.gz' EXIT
8287
tar -xf drake.tar.gz -C /opt
8388

84-
apt-get update
85-
apt-get install --no-install-recommends $(cat <<EOF
89+
${maybe_sudo} apt-get update
90+
${maybe_sudo} apt-get install --no-install-recommends $(cat <<EOF
8691
python3-all-dev
8792
pipx
8893
EOF

0 commit comments

Comments
 (0)