Skip to content

Commit

Permalink
fix Makefile for legacy shell
Browse files Browse the repository at this point in the history
The top level Makefile needs to use multiple []'s between  ||'s
to not cause problems for legacy shells such as the Bourne shell.

For example:

```make
	-${Q} if [ -z "${MANDIR}" ] || [ ! -s calc.1 ]; then \
...
	-${Q} if [ -z "${CATDIR}" ] || [ ! -s calc.cat1 ]; then \
```
  • Loading branch information
lcn2 committed Dec 21, 2023
1 parent 3e7ccfd commit 60698d2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -3774,7 +3774,7 @@ install: ${LIB_H_SRC} ${BUILD_H_SRC} calc.1 all custom/Makefile
echo "removed old ${T}${CALC_INCDIR}/string.h"; \
fi; \
done
-${Q} if [ -z "${MANDIR}" || ! -s calc.1 ]; then \
-${Q} if [ -z "${MANDIR}" ] || [ ! -s calc.1 ]; then \
${TRUE}; \
else \
if ${CMP} -s calc.1 ${T}${MANDIR}/calc.${MANEXT}; then \
Expand All @@ -3788,7 +3788,7 @@ install: ${LIB_H_SRC} ${BUILD_H_SRC} calc.1 all custom/Makefile
echo "installed ${T}${MANDIR}/calc.${MANEXT}"; \
fi; \
fi
-${Q} if [ -z "${CATDIR}" || ! -s calc.cat1 ]; then \
-${Q} if [ -z "${CATDIR}" ] || [ ! -s calc.cat1 ]; then \
${TRUE}; \
else \
if ${CMP} -s calc.cat1 ${T}${CATDIR}/calc.${MANEXT}; then \
Expand Down

0 comments on commit 60698d2

Please sign in to comment.