[pkgsrc/trunk]: pkgsrc/mk/bulk Don't test(1) and cd(1), cd(1) directly. As lo...

[rillig <rillig%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/941e1b343e66
branches:  trunk
changeset: 507276:941e1b343e66
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Sun Jan 29 23:07:04 2006 +0000

description:
Don't test(1) and cd(1), cd(1) directly. As long as "set -e" is not
activated this prevents unexpected behavior.

diffstat:

 mk/bulk/pre-build |  5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diffs (20 lines):

diff -r 020ec781492f -r 941e1b343e66 mk/bulk/pre-build
--- a/mk/bulk/pre-build Sun Jan 29 22:29:01 2006 +0000
+++ b/mk/bulk/pre-build Sun Jan 29 23:07:04 2006 +0000
@@ -1,5 +1,5 @@
 #!/bin/sh
-# $NetBSD: pre-build,v 1.56 2005/12/08 21:21:15 rillig Exp $
+# $NetBSD: pre-build,v 1.57 2006/01/29 23:07:04 rillig Exp $
 #
 # Clean up system to be ready for bulk pkg build
 #
@@ -111,8 +111,7 @@
 
 for dbdir in "${PKG_DBDIR}" "${DEPOTBASE}"; do
        echo "pre-build> Removing all installed packages in ${dbdir}"
-       if [ -d "$dbdir" ]; then
-               cd "$dbdir"
+       if cd "$dbdir" 1>/dev/null 2>&1; then
                for pkg in *
                do
                        if ${PKG_TOOLS_BIN}/pkg_info -K "$dbdir" -qe "$pkg"; then