Re: [PATCH] ylwrap
Tom> Alexandre, is ylwrap still maintained in the automake repository? adl> Yes. Do you think we should mention Automake in the headings of adl> all similar auxiliary files? Sure, but it doesn't matter much to me. A note in HACKING would suffice as well. Tom
Re: [PATCH] ylwrap
Ok thanks, my bad, I should have read/joined the list sooner :)
btw, there's another patch waiting on the website, but I don't know if
the author manifested himself.
/// didier
On Fri, 2003-10-24 at 21:39, Alexandre Duret-Lutz wrote:
> >>> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
>
> >> "Didier" == dc <[EMAIL PROTECTED]> writes:
> Didier> I've made a patch several months ago concerning ylwrap, and
> Didier> posted it on http://savannah.gnu.org/patch/?group=automake ,
> Didier> but it seems that it wasn't included yet. Since there wasn't
> Didier> any response so far, I joined the list to ask about its
> Didier> validity.
>
> Thank you Didier. Sorry nobody noticed you patch, but Automake
> does not use Savannah. (See sources.redhat.com/automake/)
>
> Tom> I think this is obviously correct.
>
> I've installed the fix on HEAD and branch-1-7 as follows.
>
> Tom> Alexandre, is ylwrap still maintained in the automake repository?
>
> Yes. Do you think we should mention Automake in the headings of
> all similar auxiliary files?
>
>
> 2003-10-24 Didier Cassirame <[EMAIL PROTECTED]> (tiny change)
>
> * lib/ylwrap: Typo in regex.
>
> Index: lib/ylwrap
> ===
> RCS file: /cvs/automake/automake/lib/ylwrap,v
> retrieving revision 1.21
> diff -u -r1.21 ylwrap
> --- lib/ylwrap19 Apr 2002 20:54:45 - 1.21
> +++ lib/ylwrap24 Oct 2003 19:29:11 -
> @@ -1,7 +1,7 @@
> #! /bin/sh
> # ylwrap - wrapper for lex/yacc invocations.
> #
> -# Copyright (C) 1996, 1997, 1998, 1999, 2001, 2002 Free Software
> +# Copyright (C) 1996, 1997, 1998, 1999, 2001, 2002, 2003 Free Software
> # Foundation, Inc.
> #
> # Written by Tom Tromey <[EMAIL PROTECTED]>.
> @@ -134,7 +134,7 @@
>TARGET=`echo "$2" | sed \
> -e 'y/abcdefghijklmnopqrstuvwxyz/ABCDEFGHIJKLMNOPQRSTUVWXYZ/'\
> -e 's/[^ABCDEFGHIJKLMNOPQRSTUVWXYZ]/_/g'`
> - sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FORM,$TO,;}" "$from" >"$target" ||
> + sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FROM,$TO,;}" "$from" >"$target" ||
> status=$?
>else
> Index: THANKS
> ===
> RCS file: /cvs/automake/automake/THANKS,v
> retrieving revision 1.186.2.35
> diff -u -r1.186.2.35 THANKS
> --- THANKS21 Oct 2003 13:07:04 - 1.186.2.35
> +++ THANKS24 Oct 2003 19:32:45 -
> @@ -46,6 +46,7 @@
> David Zaroski[EMAIL PROTECTED]
> Dean Povey [EMAIL PROTECTED]
> Derek R. Price [EMAIL PROTECTED]
> +Didier Cassirame [EMAIL PROTECTED]
> Dieter Baron [EMAIL PROTECTED]
> Dmitry Mikhin[EMAIL PROTECTED]
> Doug Evans [EMAIL PROTECTED]
Re: [PATCH] ylwrap
>>> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
>> "Didier" == dc <[EMAIL PROTECTED]> writes:
Didier> I've made a patch several months ago concerning ylwrap, and
Didier> posted it on http://savannah.gnu.org/patch/?group=automake ,
Didier> but it seems that it wasn't included yet. Since there wasn't
Didier> any response so far, I joined the list to ask about its
Didier> validity.
Thank you Didier. Sorry nobody noticed you patch, but Automake
does not use Savannah. (See sources.redhat.com/automake/)
Tom> I think this is obviously correct.
I've installed the fix on HEAD and branch-1-7 as follows.
Tom> Alexandre, is ylwrap still maintained in the automake repository?
Yes. Do you think we should mention Automake in the headings of
all similar auxiliary files?
2003-10-24 Didier Cassirame <[EMAIL PROTECTED]> (tiny change)
* lib/ylwrap: Typo in regex.
Index: lib/ylwrap
===
RCS file: /cvs/automake/automake/lib/ylwrap,v
retrieving revision 1.21
diff -u -r1.21 ylwrap
--- lib/ylwrap 19 Apr 2002 20:54:45 - 1.21
+++ lib/ylwrap 24 Oct 2003 19:29:11 -
@@ -1,7 +1,7 @@
#! /bin/sh
# ylwrap - wrapper for lex/yacc invocations.
#
-# Copyright (C) 1996, 1997, 1998, 1999, 2001, 2002 Free Software
+# Copyright (C) 1996, 1997, 1998, 1999, 2001, 2002, 2003 Free Software
# Foundation, Inc.
#
# Written by Tom Tromey <[EMAIL PROTECTED]>.
@@ -134,7 +134,7 @@
TARGET=`echo "$2" | sed \
-e 'y/abcdefghijklmnopqrstuvwxyz/ABCDEFGHIJKLMNOPQRSTUVWXYZ/'\
-e 's/[^ABCDEFGHIJKLMNOPQRSTUVWXYZ]/_/g'`
- sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FORM,$TO,;}" "$from" >"$target" ||
+ sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FROM,$TO,;}" "$from" >"$target" ||
status=$?
else
Index: THANKS
===
RCS file: /cvs/automake/automake/THANKS,v
retrieving revision 1.186.2.35
diff -u -r1.186.2.35 THANKS
--- THANKS 21 Oct 2003 13:07:04 - 1.186.2.35
+++ THANKS 24 Oct 2003 19:32:45 -
@@ -46,6 +46,7 @@
David Zaroski [EMAIL PROTECTED]
Dean Povey [EMAIL PROTECTED]
Derek R. Price [EMAIL PROTECTED]
+Didier Cassirame [EMAIL PROTECTED]
Dieter Baron [EMAIL PROTECTED]
Dmitry Mikhin [EMAIL PROTECTED]
Doug Evans [EMAIL PROTECTED]
--
Alexandre Duret-Lutz
Re: [PATCH] ylwrap
> "Didier" == dc <[EMAIL PROTECTED]> writes: Didier> I've made a patch several months ago concerning ylwrap, and Didier> posted it on http://savannah.gnu.org/patch/?group=automake , Didier> but it seems that it wasn't included yet. Since there wasn't Didier> any response so far, I joined the list to ask about its Didier> validity. I think this is obviously correct. Alexandre, is ylwrap still maintained in the automake repository? Tom
[PATCH] ylwrap
Hi,
I've made a patch several months ago concerning ylwrap, and posted it on
http://savannah.gnu.org/patch/?group=automake , but it seems that it
wasn't included yet. Since there wasn't any response so far, I joined
the list to ask about its validity.
This patch fixes a small typo on a variable name, and comes attached
with this mail.
Didier Cassirame
--- ./ylwrap 2003-03-05 00:52:19.0 +0100
+++ ./automake-1.7/ylwrap 2003-03-05 00:53:02.0 +0100
@@ -134,7 +134,7 @@
TARGET=`echo "$2" | sed \
-e 'y/abcdefghijklmnopqrstuvwxyz/ABCDEFGHIJKLMNOPQRSTUVWXYZ/'\
-e 's/[^ABCDEFGHIJKLMNOPQRSTUVWXYZ]/_/g'`
- sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FORM,$TO,;}" "$from" >"$target" ||
+ sed "/^#/{s,$input_rx,,;s,$from,$2,;s,$FROM,$TO,;}" "$from" >"$target" ||
status=$?
else
# A missing file is only an error for the first file. This
