Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: PATCH: installing executable functions



On Sat, 30 Jan 2010 08:35:03 +0300
Andrey Borzenkov <arvidjaar@xxxxxxxxx> wrote:
> read -r line < $sdir_top/$file
> case "$line" in
>     '#!'* )
> 		chmod +x $instdir/`echo $file | sed -e 's%^.*/%%'`
> 	;;
> esac

This looks pretty good: we already rely on "case" heavily in
configure...  I'm a trifle worried about "read -r" in case some perverse
Bourne shell doesn't handle the "-r", and since we don't want spaces or
backslashes etc.  I don't think it's necessary... any theories / further
suggestions?

-- 
Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/



Messages sorted by: Reverse Date, Date, Thread, Author