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

Paul Bone pbone at csse.unimelb.edu.au
Wed Feb 11 14:19:49 AEDT 2009


On Wed, Feb 11, 2009 at 01:54:13PM +1100, Peter Wang wrote:
> 2009/2/11 Paul Bone <pbone at csse.unimelb.edu.au>:
> >
> > Estimated hours taken: 0.45
> > Modules: CVSROOT
> >
> > Fixed a number of problems in the CVSROOT module.
> >
> > modules:
> >        Commented out deprecated options, these are now implicit in modern
> >        versions of CVS.
> >
> > check.pl:
> > newcheck.pl:
> > log.pl:
> >        Replaced hash-bang lines with portable alternatives.
> >
> > Index: check.pl
> > ===================================================================
> > RCS file: /home/mercury/mercury1/repository/CVSROOT/check.pl,v
> > retrieving revision 1.21
> > diff -u -p -b -r1.21 check.pl
> > --- check.pl    8 Dec 2004 01:30:23 -0000       1.21
> > +++ check.pl    11 Feb 2009 02:22:19 -0000
> > @@ -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.  I would like to use env
rather than sh but I beleive it is less standard.  Provided that the
command line interface of sh is more standard than the existance of env
this is the correct solution.


-------------- 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/f2732f84/attachment.sig>


More information about the reviews mailing list