[m-rev.] diff: Fix problems with check.pl when using remote CVS.

[Paul Bone]

On Wed, Feb 11, 2009 at 02:36:32PM +1100, Peter Wang wrote:
> 2009/2/11 Paul Bone <pbone at csse.unimelb.edu.au>:
> > On Wed, Feb 11, 2009 at 01:54:13PM +1100, Peter Wang wrote:
> >> 2009/2/11 Paul Bone <pbone at csse.unimelb.edu.au>:
> >> > @@ -1,4 +1,4 @@
> >> > -#!/usr/local/bin/perl -w
> >> > +#!/bin/sh -c "perl -w"
> >>
> >> Does this work?  '-c "perl -w"' is likely to be passed to /bin/sh as a single
> >> argument on most systems.
> >>
> >
> > I tested it on taura and it worked correctly.
> 
> Are you sure?
> 
>     wangp at taura ~ % cat hi
>     #!/bin/sh -c "perl -w"
>     print "hi\n";
> 
>     wangp at taura ~ % ./hi
>     /bin/sh: Illegal option -
>     zsh: exit 2     ./hi

Not anymore :-).  I've also tested this for myself, it doesn't work.
And using env won't support passing -w to perl either.  The best case is
to add the statment "use warn;" to the top of the perl scripts, although
this has slightly different semantics but only the kind of semantics
that perl programmers understand :-).

I'll change it to:

    #!/bin/sh perl

    use warn;

Thanks.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20090211/d03c56ed/attachment.sig>