diff options
author | Zach White <skullydazed@gmail.com> | 2021-05-19 15:24:46 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-05-19 15:24:46 -0700 |
commit | db1eacdaacb9c8f6889f46bc1c6af155b81ad72a (patch) | |
tree | cd32a69a04c7ff93a11941d82aef3ce31c2b7523 /util/install | |
parent | a9aec546c873fa5a2cb1d9a10878aca71818b609 (diff) | |
download | qmk_firmware-db1eacdaacb9c8f6889f46bc1c6af155b81ad72a.tar.gz qmk_firmware-db1eacdaacb9c8f6889f46bc1c6af155b81ad72a.zip |
Align our subprocess usage with current best practices. (#12940)
* Align our subprocess usage with current best practices.
* remove unused import
* Apply suggestions from code review
Co-authored-by: Ryan <fauxpark@gmail.com>
* fix the cpp invocation for older python
* allow for unprompted installation
* make sure qmk new-keyboard works on windows
Co-authored-by: Ryan <fauxpark@gmail.com>
Diffstat (limited to 'util/install')
-rwxr-xr-x | util/install/debian.sh | 2 | ||||
-rwxr-xr-x | util/install/fedora.sh | 2 | ||||
-rwxr-xr-x | util/install/freebsd.sh | 2 | ||||
-rwxr-xr-x | util/install/msys2.sh | 2 | ||||
-rwxr-xr-x | util/install/void.sh | 2 |
5 files changed, 5 insertions, 5 deletions
diff --git a/util/install/debian.sh b/util/install/debian.sh index 0ae9764a3..885df723d 100755 --- a/util/install/debian.sh +++ b/util/install/debian.sh | |||
@@ -5,7 +5,7 @@ DEBCONF_NONINTERACTIVE_SEEN=true | |||
5 | export DEBIAN_FRONTEND DEBCONF_NONINTERACTIVE_SEEN | 5 | export DEBIAN_FRONTEND DEBCONF_NONINTERACTIVE_SEEN |
6 | 6 | ||
7 | _qmk_install_prepare() { | 7 | _qmk_install_prepare() { |
8 | sudo apt-get update | 8 | sudo apt-get update $SKIP_PROMPT |
9 | } | 9 | } |
10 | 10 | ||
11 | _qmk_install() { | 11 | _qmk_install() { |
diff --git a/util/install/fedora.sh b/util/install/fedora.sh index 44b71b98b..d9452f68d 100755 --- a/util/install/fedora.sh +++ b/util/install/fedora.sh | |||
@@ -4,7 +4,7 @@ _qmk_install() { | |||
4 | echo "Installing dependencies" | 4 | echo "Installing dependencies" |
5 | 5 | ||
6 | # TODO: Check whether devel/headers packages are really needed | 6 | # TODO: Check whether devel/headers packages are really needed |
7 | sudo dnf -y install \ | 7 | sudo dnf $SKIP_PROMPT install \ |
8 | clang diffutils git gcc glibc-headers kernel-devel kernel-headers make unzip wget zip \ | 8 | clang diffutils git gcc glibc-headers kernel-devel kernel-headers make unzip wget zip \ |
9 | python3 \ | 9 | python3 \ |
10 | avr-binutils avr-gcc avr-libc \ | 10 | avr-binutils avr-gcc avr-libc \ |
diff --git a/util/install/freebsd.sh b/util/install/freebsd.sh index 353c52d64..d9399a5de 100755 --- a/util/install/freebsd.sh +++ b/util/install/freebsd.sh | |||
@@ -1,7 +1,7 @@ | |||
1 | #!/bin/bash | 1 | #!/bin/bash |
2 | 2 | ||
3 | _qmk_install_prepare() { | 3 | _qmk_install_prepare() { |
4 | sudo pkg update | 4 | sudo pkg update $SKIP_PROMPT |
5 | } | 5 | } |
6 | 6 | ||
7 | _qmk_install() { | 7 | _qmk_install() { |
diff --git a/util/install/msys2.sh b/util/install/msys2.sh index c8598a60f..b1300a34d 100755 --- a/util/install/msys2.sh +++ b/util/install/msys2.sh | |||
@@ -1,7 +1,7 @@ | |||
1 | #!/bin/bash | 1 | #!/bin/bash |
2 | 2 | ||
3 | _qmk_install_prepare() { | 3 | _qmk_install_prepare() { |
4 | pacman -Syu | 4 | pacman -Syu $MSYS2_CONFIRM |
5 | } | 5 | } |
6 | 6 | ||
7 | _qmk_install() { | 7 | _qmk_install() { |
diff --git a/util/install/void.sh b/util/install/void.sh index 9ec7019e5..c49c3ef63 100755 --- a/util/install/void.sh +++ b/util/install/void.sh | |||
@@ -3,7 +3,7 @@ | |||
3 | _qmk_install() { | 3 | _qmk_install() { |
4 | echo "Installing dependencies" | 4 | echo "Installing dependencies" |
5 | 5 | ||
6 | sudo xbps-install \ | 6 | sudo xbps-install $SKIP_PROMPT \ |
7 | gcc git make wget unzip zip \ | 7 | gcc git make wget unzip zip \ |
8 | python3-pip \ | 8 | python3-pip \ |
9 | avr-binutils avr-gcc avr-libc \ | 9 | avr-binutils avr-gcc avr-libc \ |