Subject: PR/37489 CVS commit: pkgsrc/mk/pkginstall
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: Roland Illig <rillig@netbsd.org>
List: pkgsrc-bugs
Date: 12/06/2007 22:05:04
The following reply was made to PR pkg/37489; it has been noted by GNATS.

From: Roland Illig <rillig@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/37489 CVS commit: pkgsrc/mk/pkginstall
Date: Thu,  6 Dec 2007 22:03:22 +0000 (UTC)

 Module Name:	pkgsrc
 Committed By:	rillig
 Date:		Thu Dec  6 22:03:22 UTC 2007
 
 Modified Files:
 	pkgsrc/mk/pkginstall: bsd.pkginstall.mk
 
 Log Message:
 Print an understandable error message if the number of arguments for
 CONF_FILES, CONF_FILES_PERMS, REQD_FILES, REQD_FILES_PERMS is wrong.
 
 NB: The code doesn't read like "shift 5 || error_out" since NetBSD's
 shell exits if a shift fails in this case, instead of just reporting an
 error.
 
 Fixes PR 37489.
 
 I didn't fix the code in pkglint (which was suggested in the PR) since
 it seems too complicated to me. There is no support for a
 "MultipleShellWords" data type by now, and pkglint would have to know
 that SETUID_ROOT_PERMS is of type "ThreeShellWords: Username, Groupname,
 Filemode". That's too much work and doesn't look nicely.
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.36 -r1.37 pkgsrc/mk/pkginstall/bsd.pkginstall.mk
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.