Subject: Re: review wanted: su/cvs return code checking (resend)
To: Alistair Crooks <agc@pkgsrc.org>
From: Hubert Feyrer <hubert.feyrer@informatik.fh-regensburg.de>
List: tech-pkg
Date: 11/28/2003 23:56:25
On Thu, 27 Nov 2003, Alistair Crooks wrote:
> How about (not tried):
>
> 	if ${ID} ${CVS_USER} > /dev/null 2>&1; then			\
> 		su - ${CVS_USER} -c 'stty sane ; setenv CVS_RSH ssh ; cd '${USR_PKGSRC}' ; cvs -q update -Pd '"${CVS_FLAGS}"' || exit 0'; \
> 		if [ $? != 0 ]; then					\
> 			...						\
> 		fi;							\
> 	else								\
> 		echo "*** $${CVS_USER} (${CVS_USER}) does not exist ***" 2>&1; \
> 		exit 1;							\
> 	fi

Um, pre-build is a /bin/sh script, not a Makefile. ;)
I've committed the change as proposed for a start, we can still discuss
details if needed.


 - Hubert

-- 
  ___ _ _  _   _        * Harddisk Image Cloning *
 / __| | || | | |           www.feyrer.de/g4u/
| (_ |_  _| |_| |
 \___| |_| \___/          v1.12 out now, including partition support!