[m-rev.] New tools command: cvsdiff

Ralph Becket rafe at cs.mu.OZ.AU
Thu Oct 24 15:36:30 AEST 2002


Fergus Henderson, Thursday, 24 October 2002:
> On 24-Oct-2002, Ralph Becket <rafe at cs.mu.OZ.AU> wrote:
> > Index: tools/cvsdiff:
> ...
> > +lastcvsdiff=$(ls -rt cvsdiff-* 2> /dev/null | tail -1)
> 
> Is $(...) portable enough?

I've never had trouble with it in the past, but then I've stuck to bash
since the Dark Ages.

> I'd use `...` rather than $(...).

No problem.

> 
> > +cvs diff > $newcvsdiff
> 
> I suggest using `cvs -n diff' here.

Looking at my .cvsrc, I have `cvs diff -u -N' - would hardcoding that be
acceptable?

- Ralph
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list