Skip site navigation (1)Skip section navigation (2)
Date:      28 Aug 1997 23:09:53 +0200
From:      Wolfram Schneider <wosch@apfel.de>
To:        Jun-ichiro Itoh <itojun@FreeBSD.ORG>
Cc:        cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-usrbin@FreeBSD.ORG, Ulrich Drepper <drepper@ipd.info.uni-karlsruhe.de>, joerg_wunsch@uriah.heep.sax.de
Subject:   Re: cvs commit: src/usr.bin/shar shar.sh
Message-ID:  <p1ik9h6m2u6.fsf@panke.panke.de>
In-Reply-To: Jun-ichiro Itoh's message of Thu, 28 Aug 1997 07:35:10 -0700 (PDT)
References:  <199708281435.HAA14141@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Jun-ichiro Itoh <itojun@FreeBSD.ORG> writes:
> itojun      1997/08/28 07:35:09 PDT
>   Modified files:
>     usr.bin/shar         shar.sh 
>   Log:
>   To cope with shell archive file generated by GNU sharutils 4.2.
>   (it assumes that shar will bark if there's invalid argument specified)
>   
>   Patch suggested by Ulrich Drepper <drepper@ipd.info.uni-karlsruhe.de>.
>   
>   Reviewed by:	joerg_wunsch@uriah.heep.sax.de
>   
>   Revision  Changes    Path
>   1.2       +10 -2     src/usr.bin/shar/shar.sh

The patch is not complete. shar(1) still exit with value
zero even a file is not readable.

$ touch /tmp/shar
$ chmod 0000 /tmp/shar 
$ sh shar.sh /tmp/shar > /dev/null 
sed: /tmp/shar: Permission denied
$ echo $?
0


Alternative patch:

Index: shar.sh
===================================================================
RCS file: /usr/cvs/src/usr.bin/shar/shar.sh,v
retrieving revision 1.1
diff -u -r1.1 shar.sh
--- shar.sh	1994/05/27 12:32:39	1.1
+++ shar.sh	1997/08/28 20:50:08
@@ -64,7 +64,7 @@
 	else
 		echo "echo x - $i"
 		echo "sed 's/^X//' >$i << 'END-of-$i'"
-		sed 's/^/X/' $i
+		sed 's/^/X/' $i || exit
 		echo "END-of-$i"
 	fi
 done

-- 
Wolfram Schneider   <wosch@apfel.de>   http://www.apfel.de/~wosch/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p1ik9h6m2u6.fsf>