Simplify test for libpq-dev user/jeff/lp801624_fix_bash_test
authorJeff Godin <jgodin@tadl.org>
Tue, 13 Sep 2011 19:40:10 +0000 (15:40 -0400)
committerJeff Godin <jgodin@tadl.org>
Tue, 13 Sep 2011 19:40:10 +0000 (15:40 -0400)
Simplify libpq-dev test logic per dbs suggestion (single if
instead of nested), also remove leftover debugging echo statement.

Open-ILS/src/extras/Makefile.install

index 26927f9..bd9ac6f 100644 (file)
@@ -388,19 +388,15 @@ install_debs:
 
 install_pgsql_client_debs_90:
        @LIBPQ=`$(APT_TOOL) versions libpq-dev 2>&1 | grep ^i|sed 's/^i .[ \t]*//'|cut -d. -f1` ; \
-       if [ $$LIBPQ ] ; \
+       if [[ -n $$LIBPQ && $$LIBPQ -eq 8 ]] ; \
                then \
-               echo $$LIBPQ; \
-               if [ $$LIBPQ -eq 8 ] ; \
-                       then \
-                       echo -e "libpq5 / libpq-dev are installed as part of OpenSRF (a dependency" \
-                       "of apache2-prefork-dev), but these are the 8.4 versions and they" \
-                       "conflict with the 9.0 versions - so remove them, install the pinned" \
-                       "backports of 9.0, then reinstall the apache2-prefork-dev package.\n\n" \
-                       "**WARNING**: Saying 'Yes' to this step will _remove_ any PostgreSQL" \
-                       "databases that might currently exist on this machine." && \
-                       $(APT_SAFE_TOOL) remove libpq5 libpq-dev ; \
-               fi \
+               echo -e "libpq5 / libpq-dev are installed as part of OpenSRF (a dependency" \
+               "of apache2-prefork-dev), but these are the 8.4 versions and they" \
+               "conflict with the 9.0 versions - so remove them, install the pinned" \
+               "backports of 9.0, then reinstall the apache2-prefork-dev package.\n\n" \
+               "**WARNING**: Saying 'Yes' to this step will _remove_ any PostgreSQL" \
+               "databases that might currently exist on this machine." && \
+               $(APT_SAFE_TOOL) remove libpq5 libpq-dev ; \
        fi
        $(APT_BACKPORT_TOOL) install $(PGSQL_CLIENT_DEBS_90)
        $(APT_TOOL) install apache2-prefork-dev