[m-rev.] for review: set perl path using configure

Ian MacLarty maclarty at csse.unimelb.edu.au
Tue Dec 18 17:51:43 AEDT 2007


On Tue, Dec 18, 2007 at 04:35:58PM +1100, Zoltan Somogyi wrote:
> On 18-Dec-2007, Ian MacLarty <maclarty at csse.unimelb.edu.au> wrote:
> > Index: scripts/mtags.in
> > ===================================================================
> > RCS file: scripts/mtags.in
> > diff -N scripts/mtags.in
> > --- /dev/null	1 Jan 1970 00:00:00 -0000
> > +++ scripts/mtags.in	18 Dec 2007 04:42:43 -0000
> > @@ -0,0 +1,560 @@
> > +#!@PERL@
> 
> The raw diff isn't as useful as a diff between the old mtags and the new
> mtags.in would have been. Am I correct in assuming that this diff would show
> no changes except the top few lines of the file?

Yes.

> 
> If yes, and if perl doesn't require any arguments for a script (the way
> awk requires -f), then the diff is fine.
> 

The old version used to pass the -w option (for warnings).  I've now
added -w to mtags.in.  The old version also used to pass -S which told
perl to look in the PATH for mtags.  However we don't pass mtags as an
argument to perl anymore, so I don't think this option is necessary.

I tested these changes by running mtags on compiler/*.m and jumping to
a few tags from vim.

Here is the interdiff:

diff -u scripts/mtags.in scripts/mtags.in
--- scripts/mtags.in	18 Dec 2007 04:42:43 -0000
+++ scripts/mtags.in	18 Dec 2007 06:41:53 -0000
@@ -1,4 +1,4 @@
-#!@PERL@
+#!@PERL@ -w
 # vim: ts=4 sw=4 et
 #---------------------------------------------------------------------------#
 # Copyright (C) 1994-2001, 2003, 2005-2007 The University of Melbourne.

Ian.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list