FIX #1977: Check to see if $(bindir) is in the path
authorIan Lynagh <igloo@earth.li>
Sun, 24 Feb 2008 13:43:34 +0000 (13:43 +0000)
committerIan Lynagh <igloo@earth.li>
Sun, 24 Feb 2008 13:43:34 +0000 (13:43 +0000)
Before telling the user to add it, when installing a bindist, check to
see if $(bindir) is already in the path.

distrib/Makefile

index a157672..ee8fc28 100644 (file)
@@ -34,13 +34,19 @@ install::
 
 install :: postinstall denounce
 
+# Look to see if $(bindir) is in $(PATH). Assumes there are no funky
+# characters.
+GREPPED_PATH=$(shell echo "$(PATH)" | grep "\(^\|:\)$(bindir)\(:\|$$\)")
+
 denounce:
        @echo
        @echo =======================================================================
        @echo Installation of $(package)-$(version) was successful.
        @echo
+ifneq "$(GREPPED_PATH)" ""
        @echo To use, add $(bindir) to your PATH.
        @echo
+endif
        @if test -f share/html/index.html; then \
           echo For documentation, see $(htmldir)/index.html ; \
        elif test -d share/html ; then \