Skip to content

Commit

Permalink
build: Case-insensitive check for GRASS in prefix (#949)
Browse files Browse the repository at this point in the history
A check for the word grass in the installation directory (prefix) was lowercase only. This was too exclusive and caused a warning and unnecessary interaction for cases where GRASS is written in uppercase which is the case, e.g., on macOS with /Applications/GRASS-7.9.app/...

This checks against a lower-cased install directory path, hence making it case-insensitive.
  • Loading branch information
nilason committed Sep 15, 2020
1 parent ecc150f commit 5e976f7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion include/Make/Install.make
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ install-check-writable:
fi

install-check-prefix:
@ result=`echo "$(INST_DIR)" | awk '{ if ($$1 ~ /grass/) print $$1 }'`; \
@ result=`echo "$(INST_DIR)" | awk '{ if (tolower($$1) ~ /grass/) print $$1 }'`; \
if [ "$$result" = "" ] ; then \
echo "WARNING: Your install directory $(INST_DIR)" >&2 ; \
echo " does not contain the word 'grass'." >&2 ; \
Expand Down

0 comments on commit 5e976f7

Please sign in to comment.