PR/34036 CVS commit: pkgsrc/mk/fetch

[Joerg Sonnenberger <joerg%netbsd.org@localhost>]

The following reply was made to PR pkg/34036; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg%netbsd.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: PR/34036 CVS commit: pkgsrc/mk/fetch
Date: Thu, 20 Jul 2006 13:31:08 +0000 (UTC)

 Module Name:   pkgsrc
 Committed By:  joerg
 Date:          Thu Jul 20 13:31:08 UTC 2006
 
 Modified Files:
        pkgsrc/mk/fetch: fetch
 
 Log Message:
 verify_file() returns a non-zero result whenever the distfile does not
 currently exist. This triggers a bug in the Bourne shell of at least
 DragonFly and Irix, when set -e is also active. Remove set -e to avoid
 this. Discussed with jlam@, shown by DragonFly bulk builds, complains on
 DragonFly's user list and PR 34036.
 
 A test case for this shell bug is, which should return 0:
 
 set -e
 
 testf () {
        test -f /nonexistent || return 1
        return 0
 }
 
 if testf; then
        exit 1
 fi
 exit 0
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.4 -r1.5 pkgsrc/mk/fetch/fetch
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.