Re: 07-use-w.patch

2001-03-06 Thread Akim Demaille
Akim Demaille <[EMAIL PROTECTED]> writes: > (&require_file_internal): Don't warn about installed files, just > output on STDERR. I'm fine with making this a warning again some day, but then it should be something like --warning=installing or something, set

Re: CVS Automake installs noinst_SCRIPTS.

2001-03-06 Thread Akim Demaille
Pavel Roskin <[EMAIL PROTECTED]> writes: > .PHONY uninstall-am: uninstall-%DIR%SCRIPTS I fixed this typo (missing ?INSTALL?). > -uninstall-%DIR%SCRIPTS: > - @$(NORMAL_UNINSTALL) > - @list='$(%DIR%_SCRIPTS)'; for p in $$list; do \ > - f="`echo $$p|sed '$(transform)'`"; \ > -

Re: 07-use-w.patch

2001-03-06 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >> (&require_file_internal): Don't warn about installed files, just > >> output on STDERR. > > Akim> I'm fine with making this a warning again some day, but then it > Akim> should be something like --warning=installing or something, set > Akim> by default

Re: 08-perl-is-not-sh.patch

2001-03-06 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> * automake.in: Backquote is inert in double quotes. > > Two problems with this patch: > > * You didn't writ

Re: 10-all-redirect.patch

2001-03-06 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> FYI, here is the result on the fileutils: > > Akim> @@ -307,8 +307,8 @@ > Akim>END { for (i in files) p

Re: 07-use-w.patch

2001-03-06 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >> Sometimes this error is really an error. That is why we save and > >> optionally restore the exit status around the message printing. Let's > >> make this work correctly instead of breaking it. > > Akim> I believe my patch is correct, note the $exit_

Re: Optimizing Makefiles

2001-03-06 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> What is the general policy wrt `optimizations' in automake vs > Akim> leaving some job to make? > > Let make do it. &

Re: 08-perl-is-not-sh.patch

2001-03-07 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > >> * You didn't update aclocal.in as requested by the comment near > >> %obsolete_macros > > Akim> Didn't not

Re: PATCH 2/2: Unconditional use of install-sh for install-strip (Re: cross-install-strip summary)

2001-03-07 Thread Akim Demaille
Alexandre Duret-Lutz <[EMAIL PROTECTED]> writes: > I'm not sure this is the right place to document this. > > 2001-03-06 Alexandre Duret-Lutz <[EMAIL PROTECTED]> > > * automake.texi (Requirements): Document the use of the STRIP > variable in cross-compilation environments. I'm no

Re: 07-use-w.patch

2001-03-07 Thread Akim Demaille
Akim Demaille <[EMAIL PROTECTED]> writes: > As a first step, I can apply this patch *but* with this part and the > --Werror part in defs. Then we have -w in automake, the bad uses are > fixed (at least those triggered by the test suite and the fileutils), > the only thing is t

Re: 07-use-w.patch

2001-03-08 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > >>>>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > Akim> As a first step, I can apply this patch *but* with this part and > Akim> the --Werror part in defs. > > I don&#x

Re: PATCH 2/2: Unconditional use of install-sh for install-strip (Re: cross-install-strip summary)

2001-03-08 Thread Akim Demaille
Alexandre Duret-Lutz <[EMAIL PROTECTED]> writes: > >>> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > > [...] > > Akim> I don't understand why Automake would not always use > Akim> AC_CHECK_TOOL(STRIP, strip) in AM_INIT

11-conditional-string.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&conditional_string): Produce a unique string characterizing a condition stack. (&conditional_same): Remove, comparing two strings is now enough. (&variable_define

12-true-and-false.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (read_am_file): TRUE and FALSE are predefined conditionals. (&by_condition): Adjust. (&conditional_string): Recognize (TRUE|FALSE)_(TRUE|FALSE). (&make_

14-variable-define.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_define): Extract from... (&read_am_file): here. Index: automake.in --- automake.in Thu, 08 Mar 2001 22:46:13 +0100 akim (am/f/39_automake.i 1.127 755) +++ automake.

13-single-var-storage.patch

2001-03-08 Thread Akim Demaille
support if/endif in *.am files. Patch is several chunks because it took me several steps to reach the right one. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (%am_var_defs): Replace with... (%var_is_am): this. (&handle_i

15-variable-define.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_define, &read_am_file): More work for the former from the latter. Reorganize the latter. Index: automake.in --- automake.in Thu, 08 Mar 2001 22:47:46 +0100 akim (am/f/39_automa

16-rule-define.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&rule_define): Extract from... (&read_am_file): here. Index: automake.in --- automake.in Thu, 08 Mar 2001 22:57:30 +0100 akim (am/f/39_automake.i 1.129 755) +++ automake.in Th

17-file-contents-uses-variable-define.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_define): Don't rely on $1. Initialize the variable. Set its Automakism only if not defined or if given to the user. When concatenating values, insert a separator on

18-file-contents-uses-rule-define.patch

2001-03-08 Thread Akim Demaille
coming for the MAINTAINER-MODE in configure.am: %MAKEFILE-IN%: %MAINTAINER-MODE% %MAKEFILE-AM% %MAKEFILE-IN-DEPS% $(top_srcdir)/%CONFIGURE-AC% $(ACLOCAL_M4) Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&file_contents): Use rule_define. Index: M

19-if-in-scripts-am.patch

2001-03-08 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (IF_PATTERN): Unobfuscate the parens. (ENDIF_PATTERN): Allow a condition to be specified. (&file_contents): Use a @cond_stack. (&transform): At least for an easy transition,

20-file-contents-cond-string.patch

2001-03-08 Thread Akim Demaille
see many things converging together. More factorization to come! Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&file_contents): Maintain $cond_string sync with @cond_stack. Output rules only if not under `FALSE'. Define va

Re: 12-true-and-false.patch

2001-03-08 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> * automake.in (read_am_file): TRUE and FALSE are predefined > Akim> conditionals. > Akim> (&by_condition): Adjust. > Akim> (&conditional_string): Recognize (TRUE|FALSE)_(TRUE|FALSE). > Akim> (&make_condition):

Re: 13-single-var-storage.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> Patch is several chunks because it took me several steps to > Akim> reach the right one. > > That makes it a lot harder to read. Please don't do that. I didn't mean it :( It really that I discovered side effects later on. I should have built a i

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
"Lars J. Aas" <[EMAIL PROTECTED]> writes: > When I bootstrap with the current Automake CVS sources, the strudels (@s) > between the conditionals are missing: Hi Lars! > > >@SIMAGE_JPEG_SUPPORT_FALSESIMAGE_PIC_SUPPORT_FALSESIMAGE_PNG_SUPPORT_FALSESIMAGE_RGB_SUPPORT_FALSESIMAGE_TGA_SUPPORT_FALSE

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
Hm, in fact, could you try again? I applied one of the patches which were OK'ed, which is addressing issues in this area. Maybe it cures your problem. I couldn't build a test case reproducing your problem :(

Re: 14-variable-define.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> * automake.in (&variable_define): Extract from... > Akim> (&read_am_file): here. > > I don't like this. With this patch we have subroutines named > define_variable and variable_define. I understand. > How about a longer, more desc

Re: 16-rule-define.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> * automake.in (&rule_define): Extract from... > Akim> (&read_am_file): here. > > I'd prefer a name to parallel whatever happens with "variable_define". > > If these are just subroutines of read_am_file then one approach is to > say

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
"Lars J. Aas" <[EMAIL PROTECTED]> writes: > On Fri, Mar 09, 2001 at 05:26:29PM +0100, Akim Demaille wrote: > : Hm, in fact, could you try again? I applied one of the patches which > : were OK'ed, which is addressing issues in this area. Maybe it cures > : your

21-fix-cond3.patch

2001-03-10 Thread Akim Demaille
This is the bug reported by Lars. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_conditions_permutations): Separate the conditions. * tests/cond3.test: Improve the sed expression. Index: automake.in --- automake.in Fri, 09 M

24-variable-define-variable.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&define_variable): Use &variable_define. Index: automake.in --- automake.in Sat, 10 Mar 2001 12:17:17 +0100 akim (am/f/39_automake.i 1.138 755) +++ automake.in Sat, 10 Mar 2001 12:25

25-cond-string.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&read_am_file): Keep $cond_string up to date. Index: automake.in --- automake.in Sat, 10 Mar 2001 12:58:42 +0100 akim (am/f/39_automake.i 1.143 755) +++ automake.in Sat, 10 Mar 2001 13:1

22-variables-dump.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&conditional_dump): Rename as... (&variables_dump): this. Use... (&variable_dump): this new sub. Index: automake.in --- automake.in Sat, 10 Mar 2001 11:44:5

23-variable-pretty-define.patch

2001-03-11 Thread Akim Demaille
I used variables_dump to check that there are no difference (at the end of a run, the variables are in the same state before and after this patch), and there are none. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_define): Don't lose

26-define-once.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&read_am_file): Define the variables in a single shot. Factor the $saw_bk code. Index: automake.in --- automake.in Sat, 10 Mar 2001 13:19:51 +0100 akim (am/f/39_automake.i

27-variable-value.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_footer, &handle_installdirs) (&read_main_am_file): Don't read %contents directly, use &variable_value. Index: automake.in --- automake.in Sat, 10 M

28-variable-cleanups.patch

2001-03-11 Thread Akim Demaille
I had to simplify, clean up and harmonize notations :( Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_conditionally_defined): Rename as... (&variable_unconditionally_defined): this. (&variable_dum

29-remove-contents.patch

2001-03-11 Thread Akim Demaille
This patch is long, and has been quite difficult to write. I really could not make it smaller, I tried several times to split it, but systematically failed. Actually, more is needed, and cleanups are to come. But precisely I meant to keep it not even bigger. Index: ChangeLog from Akim

30-simplify-contents-traces.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_source_transform, &read_main_am_file): Simplify loops which were split in two parts, conditional vs unconditional variables. Index: automake.in --- automake.in Sat, 10 Mar

31-simplify-contents-traces.patch

2001-03-11 Thread Akim Demaille
ture we should issue warnings in these cases? It's her right to shoot in her foot, but law requires that you inform her of her rights :) Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_conditions_sub) (&variable_va

32-simplify-contents-traces.patch

2001-03-11 Thread Akim Demaille
Index: 0.162/ChangeLog --- 0.162/ChangeLog Sat, 10 Mar 2001 17:45:55 +0100 akim (am/f/47_ChangeLog 1.146 644) +++ 0.163(w)/ChangeLog Sat, 10 Mar 2001 18:20:38 +0100 akim (am/f/47_ChangeLog 1.147 +644) @@ -1,5 +1,15 @@ 2001-03-10 Akim Demaille <[EMAIL PROTECTED]> + * autom

33-simplify-contents-traces.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&generate_makefile) Use variable_define to define SOURCES and OBJECTS. (&variable_defined): Don't check for $conditional{VAR}{COND} as this would make perl create $condi

34-name-clash.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&read_am_file, &file_contents): Avoid name clashed on $cond. Index: automake.in --- automake.in Sat, 10 Mar 2001 18:48:14 +0100 akim (am/f/39_automake.i 1.153 755) +++ automake.in Sat, 10 Mar

35-avoid-complaints.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * depend2.am (@AMDEP@%FPFX%DEPMODE): Define it when GENERIC so that it does not get noticed twice by automake, which now complains for multiple definitions. * tests/vartar.test: s/INSTALL/install/g, au

36-fix-sub-conditions.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_source_transform): Use &variable_conditions, don't read $conditional{$var}. (&variable_conditions_sub): When the call is the top level call, generate all

37-fix-conditions-reduce.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_conditions_reduce): FALSE is absorbent. (&variable_conditions): Don't include `FALSE' in the result. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]>

38-minor-tests-issues.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * tests/defs, tests/depend3.test: Remove useless code. Index: tests/defs --- tests/defs Wed, 07 Mar 2001 21:02:27 +0100 akim (am/f/2_defs 1.6 644) +++ tests/defs Sat, 10 Mar 2001 20:45:43 +0100 akim (am/f/2_defs 1.6 644) @@

39-fix-variable-defined.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_defined): Some callers, e.g., &am_primary_prefixes, really want to know if the variable is defined for any condition, not `TRUE' by default. (&am_primary

40-better-messages.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_dump): If a variable is undefined, say it. (&check_ambiguous_conditional): Give finer error messages. (&variable_define): Check that a `+=' v

41-locality.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_source_transform): Better locality of the variables. Use `next' to skip nonexistent variables. Index: automake.in --- automake.in Sun, 11 Mar 2001 12:53:20 +

42-override.patch

2001-03-11 Thread Akim Demaille
Index: 0.175/am.prj --- 0.175/am.prj +++ 0.175(w)/am.prj @@ -8,7 +8,10 @@ * automake.in (&handle_source_transform): Better locality of the variables. Use `next' to skip nonexistent variables.") -(New-Version-Log "") +(New-Version-Log "2001-

44-define-linker.patch

2001-03-11 Thread Akim Demaille
I must say I don't understand very well why this patch is needed, but it is. I found it back in the old code, which was doing it for unconditionally defined sources, but not for conditionally defined sources (which was a bug). Now it works for both. Index: ChangeLog from Akim Dem

46-make-paragraphs.patch

2001-03-11 Thread Akim Demaille
it's a lot of fun. See the next patch for a demonstration. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&make_paragraphs): Extract from &file_contents. Make it more robust than the previous RE based scheme. (&file_co

47-use-if-endif.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * distdir.am: Use and abuse of if/endif. * scripts.am: Using if/endif with variables is fine. Index: distdir.am --- distdir.am Sat, 10 Mar 2001 16:24:49 +0100 akim (am/h/17_distdir.am 1.15 644) +++ distdir.am Sun,

48-false-confusion.patch

2001-03-11 Thread Akim Demaille
There was some confusion between `#' and `FALSE' representing negated conditions. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&make_condition): Return '#' when FALSE. (&conditional_string): Return FALSE, not 

49-use-if-install.patch

2001-03-11 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * data.am, header.am, java.am, libs.am, lisp.am, ltlib.am, * progs.am, python.am: Use if/endif instead of ?INSTALL?. * java.am (_am_installdirs): Be sure to set it. Index: data.am --- data.am Mon, 05 Mar 2

43-load-header-vars-once.patch

2001-03-11 Thread Akim Demaille
I had to write this one under the pressure of variable_define which doesn't want one variable to be defined several times. Given that it was due to a hack, fixing it using the regular mechanism seems the right choice. In addition the code is much smaller. Index: ChangeLog from Akim Dem

45-fix-yacc2-test.patch

2001-03-11 Thread Akim Demaille
I am not sure it is right or wrong to use the C++ variables to compile C code, so I am leaving the complaints from automake. If it is _not_ meant, then quite some work is needed, as it's like this at least since 1.4b. If that's meant, it just means that CXXMODE must be set some other w

Re: ok, new libtool for cygwin updates

2001-03-12 Thread Akim Demaille
"edward" <[EMAIL PROTECTED]> writes: > cond3.test fails for three reasons. Thanks, the fix has been applied. > 3) cond3.test has an incorrect comparison test function (the sed script > skips a line!) sorry, i'm not a big fan of sed. oh well. I had reworked the snippet in the meanwhile. It

Re: automake updates (was Re: ok, new libtool for cygwin updates)

2001-03-12 Thread Akim Demaille
"edward" <[EMAIL PROTECTED]> writes: > prior to this patch, automake would not generate rules for > foo_SOURCES -> foo_OBJECTS etc. now it does, and the conditionals determine > which get executed I have plenty of patches in the queue that address things related to this issue :( > 1) a

Re: 21-fix-cond3.patch

2001-03-12 Thread Akim Demaille
"Lars J. Aas" <[EMAIL PROTECTED]> writes: > On Sat, Mar 10, 2001 at 11:48:05AM +0100, Akim Demaille wrote: > : This is the bug reported by Lars. > > Thanks, it did the trick... I'm applying it, it's an obvious bug fix and I just noticed there are other messages related to this.

Re: ok, new libtool for cygwin updates

2001-03-12 Thread Akim Demaille
| --- orig/automake.in.orig Mon Mar 12 06:44:59 2001 | +++ automake.in Mon Mar 12 07:44:57 2001 | @@ -1048,8 +1048,17 @@ | # If OBJEXT/EXEEXT were not set in configure.in, do it, it | # simplifies our task, and anyway starting with Autoconf 2.50, it | # will always be defined, and

Re: ok, new libtool for cygwin updates

2001-03-12 Thread Akim Demaille
> "tailbert" == tailbert writes: >> Rather the proper fix seems to have the failing tests include >> AC_EXEEXT and AC_OBJEXT in their configure.in. tailbert> Akim, I mean in the general case, even outside of the test tailbert> cases. On windows platforms, executables get a .exe tailbert> e

Re: pr19.test

2001-03-13 Thread Akim Demaille
> make[2]: Leaving directory `/usr/src/automake/tests/testSubDir' > make[2]: Entering directory `/usr/src/automake/tests/testSubDir' > chmod -R a+w ./am_lex_bug-0.1.1 >/dev/null 2>&1; rm -rf > ./am_lex_bug-0.1.1 > mkdir ./am_lex_bug-0.1.1 > cp: ./foo.c: No such file or directory I have this. ma

Re: pr19.test

2001-03-13 Thread Akim Demaille
| From | http://sources.redhat.com/ml/automake/2001-03/msg00183.html | | 2) lines 4946-4947 of automake.in version 1.977 | | # FIXME: nodist. | &push_dist_common ($pfx . $base . '.' . $ext); | | commenting out the last line removes foo.c (which is a temporary file) | from the m

Re: plea for case-insensitive, but case preserving (was: Re: 45-fix-yacc2-test.patch)

2001-03-13 Thread Akim Demaille
> "Lars" == Lars Hecking <[EMAIL PROTECTED]> writes: Lars> edward writes: >> hi akim, >> >> this is a plea to avoid using constructs like the "cp Makefile.am >> Makefile.AM" below. this kills us cygwin/windows people, and i for >> one, very much would like to keep automake, etc. working on w

Re: Default postscript cleans miss *.cps & *.fns.

2001-03-14 Thread Akim Demaille
hi Derek, Could you `grep indexcode' your texi sources? Thanks!

Re: Default postscript cleans miss *.cps & *.fns.

2001-03-15 Thread Akim Demaille
| Akim Demaille wrote: | > hi Derek, | | Hi. :) | | | > Could you `grep indexcode' your texi sources? Thanks! | | [dprice@empress doc]$ fgrep indexcode cvs.texinfo | [dprice@empress doc]$ Actually I meant _all_ your sources. And in fact, @include would be useful too.

Re: Understanding objc.test

2001-03-16 Thread Akim Demaille
> "tailbert" == tailbert writes: tailbert> hi akim link is defined because tailbert> .m.o: tailbert> causes a trigger in finish_languages Thanks, I know this. My question is more whether this is really what should happen. I fail to understand why if the user overrides a suffix rule whi

51-clean-am-status.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&make_paragraphs): Transform TOPDIR_P and TOPDIR. (&handle_dist): Don't. (&handle_clean): Don't handle config.status here, let... * clean.am: ... do it.

52-yacc-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&condition_negate): New. (&variable_conditions_permutations): Use it. (&read_am_file, &file_contents): Use it on `else' clauses to support `if FALSE'.

53-lex-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&output_lex_build_rule): Output this... * lex.am: New file. Index: Makefile.am --- Makefile.am Sun, 11 Mar 2001 20:15:57 +0100 akim (am/f/46_Makefile.a 1.17 644) +++ Makefile.am Sun, 11 Mar 200

50-use-strict.patch

2001-03-19 Thread Akim Demaille
I don't know how to fix the `use strict refs' part. In particular, it dies on computed sub names: & $my_sub (); fails, and I don't know how to rewrite it the way strict refs wants it. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automak

54-de-remake-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * remake.am: Merge into... * configure.am: ... here. * automake.in (&handle_configure): Adjust. Index: Makefile.am --- Makefile.am Sun, 11 Mar 2001 20:30:44 +0100 akim (am/f/46_Makefile.a 1.18 644) +++ M

55-aclocal-m4-in-configure-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_aclocal_m4): Rename as... (&scan_aclocal_m4): this. Return the list of aclocal.m4 dependencies. (&handle_configure): Invoke it, and use it when loading...

57-fix-transform.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&transform): Fix an incredible bug which was breaking the if/endif system. ($ELSE_PATTERN): Admit an optional argument. Index: automake.in --- automake.in Sun, 11 Mar 2001 21:03:14 +0100

56-de-com-vars.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * comp-vars.am: Remove, merged into... * compile.am: here. * automake.in (&get_object_extension): Adjust. Index: Makefile.am --- Makefile.am Sun, 11 Mar 2001 20:39:39 +0100 akim (am/f/46_Makefile.

58-ansi2knr-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&get_object_extension): Use ansi2knr.am. * clean-kr.am, kr-extra.am: Remove, merged into... * ansi2knr.am: this new file. Index: Makefile.am --- Makefile.am Sun, 11 Mar 2001 21:03:14 +010

59-accumulate-dist-common.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (%dist_common): Remove. (%configure_dist_common): Replace with... ($configure_dist_common): this. (&generate_makefile): Read the user file before using push_dist_common,

60-dist-common.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&am_install_var): Transform DIST, and let the *.am files handle DIST_COMMON. * data.am, header.am, java.am, list.am, python.am, script.am: Set DIST_COMMON. Index: automake.in --- a

61-pessimize.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_configure): Don't bother with optimizing macro uses. Index: automake.in --- automake.in Mon, 12 Mar 2001 21:28:36 +0100 akim (am/f/39_automake.i 1.179 755) +++ auto

62-variable-pretty-output.patch

2001-03-19 Thread Akim Demaille
hould consider their length before substitution for filling? Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_output): Admit a list of @CONDS. (&variable_pretty_output): New. (&define_pretty_variable): Use it.

63-rename-var-vars.patch

2001-03-19 Thread Akim Demaille
} } I fail to understand why the user is not allowed to += a nonexistent variable. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (%var_was_plus_eq): Rename as... (%var_type): this. (%def_type): Remove. (&variable_define)

64-var-override-rules.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&variable_dump): Use %var_type properly. (&variable_define): Enforce better overriding rules. Handle the special case that used to handle... (&define_pretty_variable): t

65-man-vars.patch

2001-03-19 Thread Akim Demaille
$(SHELL) ./config.status + +man1dir = $(mandir)/man1 install-man1: $(man1_MANS) $(man_MANS) @$(NORMAL_INSTALL) $(mkinstalldirs) $(DESTDIR)$(man1dir) Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_man_pages): Rely on mans.am to define m

66-recursive-targets.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * subdirs.am (RECURSIVE_TARGETS): New variable. Use it. * automake.in (&handle_subdirs): Output it. (&file_contents_internal): Support value spread on several lines. Index: Makefile.in --- M

67-texinfo-recursive.patch

2001-03-19 Thread Akim Demaille
Isn't this beautiful? A *.am file only change! Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * subdirs.am: Don't define info related recursive targets. * texinfos.am: Do. Index: Makefile.in --- Makefile.in Tue, 13 Mar 2001 01:18:40 +0100 akim (am

68-suffixes.patch

2001-03-19 Thread Akim Demaille
, they all should be discovered (but for non `.*' pathological case, which anyway Automake does not recognize IIRC). (In fact, I've just discovered it's coming from finish_language. Once I fully understand it, I'll finish cleaning up @suffixes). Index: ChangeLog from Akim Dem

69-single-quote-regexps.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in: Use simple quotes to define the _PATTERN variables to unobfuscate the regexps. ($AM_CONDITIONAL_PATTERN): Let the user quote the variable. Index: automake.in --- automake.in W

70-fix-tags-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * tags.am: Fix missing leading tabs. Index: tags.am --- tags.am Wed, 14 Mar 2001 23:14:29 +0100 akim (am/g/12_tags.am 1.8 644) +++ tags.am Wed, 14 Mar 2001 23:15:34 +0100 akim (am/g/12_tags.am 1.8 644) @@ -45,11

71-amdep-uses-if.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> Use AM_CONDITIONAL and if/endif for AMDEP. * m4/depend.m4 (AM_DEP_TRACK): Use AM_CONDITIONAL to define AMDEP. * automake.in (&add_depend2): Transform %AMDEP% into `AMDEP' (leading to a co

72-handle-compile.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_compile): Extract from... (&get_object_extension): here. (&read_am_file): Call it. ($included_generic_compile, $included_knr_compile) ($includ

73-ext-compile-am.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * ext-compile.am: New file. * automake.in (&finish_languages): Output it. Require a C linker if there are several registered source suffixes. (&handle_compile): No longer push the `.c',

74-source-suffix-pattern.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in ($source_suffix_pattern): Remove, unused. %extension_map seems to have replaced it. Index: automake.in --- automake.in Thu, 15 Mar 2001 00:48:39 +0100 akim (am/f/39_automake.i 1.195 755) +++ automake.

75-simplifications.patch

2001-03-19 Thread Akim Demaille
Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]> * automake.in (&handle_single_transform_list): Remove $xbase, unused. (&handle_source_transform): All the variables have conditions now, simplify. (&variable_delete): Admit

76-tags-in-tags-am.patch

2001-03-19 Thread Akim Demaille
stion is why don't we use the regular -am, -recursive mechanism? Given that we always output a `tags' and `TAGS' target, it should be equivalent and much simpler: just RECURSIVE_TARGATS += tags-recursive. Index: ChangeLog from Akim Demaille <[EMAIL PROTECTED]>

Re: `missing' needs patch for Ultrix4.4

2001-03-19 Thread Akim Demaille
| Alexandre Oliva <[EMAIL PROTECTED]> wrote: | | On Mar 17, 2001, Jim Meyering <[EMAIL PROTECTED]> wrote: | | | | > * missing (--run): Use `eval' to run `"$@"'. Otherwise, Ultrix4.4's | | > /bin/sh fails and outputs garbage. | | | | Won't this evaluate the arguments one too many

Re: `missing' needs patch for Ultrix4.4

2001-03-19 Thread Akim Demaille
> "Jim" == Jim Meyering <[EMAIL PROTECTED]> writes: Jim> IMHO, it is always best to `do it right'. Evaluating `$1' one Jim> extra time is probably just fine for all current uses of `missing Jim> --run PROG_AND_ARGS', but someone may well have to use it on a Jim> program whose name contains s

Re: 50-use-strict.patch

2001-03-19 Thread Akim Demaille
> "Russ" == Russ Allbery <[EMAIL PROTECTED]> writes: Russ> (If you want to rewrite the code, you can use a hash of sub refs Russ> instead of the computed name method, but that isn't always any Russ> cleaner or more readable.) In fact, since I wrote that patch, I learned about objects in Perl

Re: `missing' needs patch for Ultrix4.4

2001-03-19 Thread Akim Demaille
> "Jim" == Jim Meyering <[EMAIL PROTECTED]> writes: Jim> Hi Akim, Jim> So consider a name containing some other shell meta-character. I don't understand what difference you make between the two approaches wrt meta-characters. Wer are talking about ( exec "$@" ) vs prog=$1 shift "$prog

<    1   2   3   4   5   6   7   8   9   10   >