airship/treasuremap
Code Issues Proposed changes

Prefix package installation commands with sudo

Browse Source 
This file calls either rpm, dpkg or apt to install a YAML module for
python. This change removes the need for this file to be run as root.

Change-Id: I77590d24e82f0ced69db97ef946fa7202d1f0a7a
This commit is contained in:
Steven Fitzpatrick 2019-08-09 13:20:02 -05:00
parent 3f5af67768
commit 19b9cecb77
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools/airship
View File

@ -21,9 +21,9 @@ test -t 1 && test -t 0 && DEFAULT_TERM_OPTS='-it'
# Python YAML module required to read versions.yaml # Python YAML module required to read versions.yaml
if grep -iq suse /etc/os-release; then if grep -iq suse /etc/os-release; then
rpm -q python3-pyaml --quiet || zypper --non-interactive install python3-pyaml sudo rpm -q python3-pyaml --quiet || sudo zypper --non-interactive install python3-pyaml
else else
dpkg -s python3-yaml &> /dev/null || apt -y install python3-yaml sudo dpkg -s python3-yaml &> /dev/null || sudo apt -y install python3-yaml
fi fi
ENV_FILE=$(mktemp) ENV_FILE=$(mktemp)