Commit b5b1d1f
build: replace non-POSIX test -a|o
test/[ from sbase unix tools[1] throws "too many arguments" if -a or
-o is provided. The syntax has been marked obsolescent as per the
manual[2].
[1] http://core.suckless.org/sbase/
[2] https://pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html#tag_20_128_16
PR-URL: #38731
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>1 parent aeea252 commit b5b1d1f
2 files changed
Lines changed: 7 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -102,7 +102,7 @@ $(NODE_EXE): build_type:=Release | |||
| 102 | 102 | $(NODE_G_EXE): build_type:=Debug | |
| 103 | 103 | $(NODE_EXE) $(NODE_G_EXE): config.gypi out/Makefile | |
| 104 | 104 | $(MAKE) -C out BUILDTYPE=${build_type} V=$(V) | |
| 105 | - if [ ! -r $@ -o ! -L $@ ]; then \ | ||
| 105 | + if [ ! -r $@ ] || [ ! -L $@ ]; then \ | ||
| 106 | 106 | ln -fs out/${build_type}/$(NODE_EXE) $@; fi | |
| 107 | 107 | else | |
| 108 | 108 | ifeq ($(BUILD_WITH), ninja) | |
@@ -116,11 +116,11 @@ else | |||
| 116 | 116 | endif | |
| 117 | 117 | $(NODE_EXE): config.gypi out/Release/build.ninja | |
| 118 | 118 | ninja -C out/Release $(NINJA_ARGS) | |
| 119 | - if [ ! -r $@ -o ! -L $@ ]; then ln -fs out/Release/$(NODE_EXE) $@; fi | ||
| 119 | + if [ ! -r $@ ] || [ ! -L $@ ]; then ln -fs out/Release/$(NODE_EXE) $@; fi | ||
| 120 | 120 | ||
| 121 | 121 | $(NODE_G_EXE): config.gypi out/Debug/build.ninja | |
| 122 | 122 | ninja -C out/Debug $(NINJA_ARGS) | |
| 123 | - if [ ! -r $@ -o ! -L $@ ]; then ln -fs out/Debug/$(NODE_EXE) $@; fi | ||
| 123 | + if [ ! -r $@ ] || [ ! -L $@ ]; then ln -fs out/Debug/$(NODE_EXE) $@; fi | ||
| 124 | 124 | else | |
| 125 | 125 | $(NODE_EXE) $(NODE_G_EXE): | |
| 126 | 126 | $(warning This Makefile currently only supports building with 'make' or 'ninja') | |
@@ -906,7 +906,7 @@ BINARYTAR=$(BINARYNAME).tar | |||
| 906 | 906 | HAS_XZ ?= $(shell command -v xz > /dev/null 2>&1; [ $$? -eq 0 ] && echo 1 || echo 0) | |
| 907 | 907 | # Supply SKIP_XZ=1 to explicitly skip .tar.xz creation | |
| 908 | 908 | SKIP_XZ ?= 0 | |
| 909 | - XZ = $(shell [ $(HAS_XZ) -eq 1 -a $(SKIP_XZ) -eq 0 ] && echo 1 || echo 0) | ||
| 909 | + XZ = $(shell [ $(HAS_XZ) -eq 1 ] && [ $(SKIP_XZ) -eq 0 ] && echo 1 || echo 0) | ||
| 910 | 910 | XZ_COMPRESSION ?= 9e | |
| 911 | 911 | PKG=$(TARNAME).pkg | |
| 912 | 912 | MACOSOUTDIR=out/macos | |
@@ -947,7 +947,7 @@ release-only: check-xz | |||
| 947 | 947 | echo "" >&2 ; \ | |
| 948 | 948 | exit 1 ; \ | |
| 949 | 949 | fi | |
| 950 | - @if [ "$(DISTTYPE)" != "release" -o "$(RELEASE)" = "1" ]; then \ | ||
| 950 | + @if [ "$(DISTTYPE)" != "release" ] || [ "$(RELEASE)" = "1" ]; then \ | ||
| 951 | 951 | exit 0; \ | |
| 952 | 952 | else \ | |
| 953 | 953 | echo "" >&2 ; \ | |
@@ -956,7 +956,7 @@ release-only: check-xz | |||
| 956 | 956 | echo "" >&2 ; \ | |
| 957 | 957 | exit 1 ; \ | |
| 958 | 958 | fi | |
| 959 | - @if [ "$(RELEASE)" = "0" -o -f "$(CHANGELOG)" ]; then \ | ||
| 959 | + @if [ "$(RELEASE)" = "0" ] || [ -f "$(CHANGELOG)" ]; then \ | ||
| 960 | 960 | exit 0; \ | |
| 961 | 961 | else \ | |
| 962 | 962 | echo "" >&2 ; \ | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -56,7 +56,7 @@ export CXX_host=$(command -v g++) | |||
| 56 | 56 | host_gcc_version=$($CC_host --version | grep gcc | awk '{print $NF}') | |
| 57 | 57 | major=$(echo $host_gcc_version | awk -F . '{print $1}') | |
| 58 | 58 | minor=$(echo $host_gcc_version | awk -F . '{print $2}') | |
| 59 | - if [ -z $major ] || [ -z $minor ] || [ $major -lt 6 ] || [ $major -eq 6 -a $minor -lt 3 ]; then | ||
| 59 | + if [ -z $major ] || [ -z $minor ] || [ $major -lt 6 ] || ( [ $major -eq 6 ] && [ $minor -lt 3 ] ); then | ||
| 60 | 60 | echo "host gcc $host_gcc_version is too old, need gcc 6.3.0" | |
| 61 | 61 | return 1 | |
| 62 | 62 | fi | |
0 commit comments