Re: Warning fix for automake on CVS Libtool

2001-05-05 Thread Pavel Roskin
On 5 May 2001, Tom Tromey wrote: > I think this is ok. > Please check it in. Ok. > Pavel> + elsif (/^\@def(code)?index (\w+)/) > Pavel> +push @clean_suffixes, $2; > > My understanding is that here $1 will not necessarily be defined, but > $2 will always be defined. Is that right?

Re: target dependency bug

2001-05-05 Thread Robert Collins
- Original Message - From: "Tom Tromey" <[EMAIL PROTECTED]> To: "Robert Collins" <[EMAIL PROTECTED]> Cc: <[EMAIL PROTECTED]> Sent: Sunday, May 06, 2001 3:23 PM Subject: Re: target dependency bug > > "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: > > Robert> I understand. Howe

objc.test

2001-05-05 Thread Tom Tromey
Is there any reason not to simply delete objc.test? Tom

Re: Warning fix for automake on CVS Libtool

2001-05-05 Thread Tom Tromey
> "Pavel" == Pavel Roskin <[EMAIL PROTECTED]> writes: Pavel> * automake.in (scan_texinfo_file): Consider @defindex and Pavel> @synindex in the same way as @defcodeindex and @syncodeindex Pavel> respectively. I think this is ok. Please check it in. Pavel> + elsif (/^\@def(code)?index

Re: Fix for PR 164 (?)

2001-05-05 Thread Tom Tromey
> "Russ" == Russ Allbery <[EMAIL PROTECTED]> writes: >> +grep ($cond_vals{$_} = 1, split (' ', $when)); Russ> It's generally considered poor Perl coding style to use grep for Russ> its side effects alone without checking the return value. Thanks. This idiom is used all over automake, b

Re: target dependency bug

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: Robert> I understand. However Robert> cf_gen.$(OBJEXT): cf_gen_defines.h Robert> is somewhat ugly for an automake file :] I agree. Perhaps it would make sense to add a dependency like this in the "dependency tracking allowed but disab

Re: Warning fix for automake on CVS Libtool

2001-05-05 Thread Pavel Roskin
Hi, Tom! > Pavel> -push @clean_suffixes, $predefined_index{$1}; > Pavel> +push @clean_suffixes, $predefined_index{$1} > Pavel> +if (defined $predefined_index{$1}); > > I think any possible value here is already defined. > So the `if' is not required. The warnings were onl

Re: More CVS head errors

2001-05-05 Thread Robert Collins
- Original Message - From: "Tom Tromey" <[EMAIL PROTECTED]> To: "Robert Collins" <[EMAIL PROTECTED]> Cc: "Akim Demaille" <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]> Sent: Sunday, May 06, 2001 7:09 AM Subject: Re: More CVS head errors > > "Robert" == Robert Collins <[EMAIL PROTECTED]> wr

Re: target dependency bug

2001-05-05 Thread Robert Collins
- Original Message - From: "Tom Tromey" <[EMAIL PROTECTED]> To: "Robert Collins" <[EMAIL PROTECTED]> Cc: <[EMAIL PROTECTED]> Sent: Sunday, May 06, 2001 6:42 AM Subject: Re: target dependency bug > > "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: > > I finally read this thread

Re: Fix for PR 164 (?)

2001-05-05 Thread Russ Allbery
Tom Tromey <[EMAIL PROTECTED]> writes: > +# Make a hash holding all the values from $WHEN. > +my (%cond_vals); > +grep ($cond_vals{$_} = 1, split (' ', $when)); It's generally considered poor Perl coding style to use grep for its side effects alone without checking the return value.

Fix for PR 164 (?)

2001-05-05 Thread Tom Tromey
Akim, what do you think of this patch? 2001-05-05 Tom Tromey <[EMAIL PROTECTED]> * automake.in (conditional_true_when): Use a hash, not index(). Also, a TRUE component always results in a true return. Fixes test cond10.test. For PR automake/164. I think the old code w

Re: automake debug output

2001-05-05 Thread Tom Tromey
> "Lars" == Lars J Aas <[EMAIL PROTECTED]> writes: Lars> When I run automake in certain projects, I get the below Lars> attached output. It's been like this for a while (figured it Lars> was a known issue that would soon get fixed), but now I suspect Lars> that I'm the only one who gets them

Re: dist-all.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> Quite straightforward (oh, how I love *.am only patches :). The Akim> only question is whether we want to factor tar too or not. It's up to you. It doesn't matter to me. Akim> * distdir.am (dist-all): Build all the flavors usin

Re: 125-better-file-contents.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> Still getting closer to what Tom doesn't want: the read_am_file Akim> removal. I would be lying if I pretended that this removal was Akim> easy: pretty many weaknesses in file_contents weaknesses are Akim> uncovered by the test suit

Re: 97.5-merge-finish-languages-dependencies.patch (Was: 98-no-depend2.patch)

2001-05-05 Thread Tom Tromey
Akim> I'm stuck. I tried to produce the single big patch which Akim> consists of them all, but after having fought with patch, I Akim> realized I must have forgotten a patch: that introducing Akim> handle_languages. Which patches does the 97.5 patch supercede? Here are patches I still have: Pa

Re: 122-use-next.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (am_install_var): Use `next' instead of `if' on the Akim> body of $X loop. Ok. Tom

Re: 123-file-contents-is-am.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&file_contents_internal): Accept $IS_AM. Akim> (&handle_compile, &define_standard_variables, &file_contents): Akim> Adjust. Ok. Tom

Re: 120-language-Name.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (Language): Add attributes `Name' and `config_vars'. Akim> (&finish): Work properly if there is no _finish. Akim> (Automake): Register language Names and AC_SUBST dependencies. Akim> Register Fortran 77 variable

Re: 119-language-ld-lder.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (Language): Add attributes `lder' and `ld'. Akim> (®ister_language): Specify for cxx, objc, f77, gcj. Akim> (&define_linker_variable): New. Akim> (&lang_cxx_finish, &lang_f77_finish, &lang_objc_finish) Akim> (

Re: 117-lex-seen-bye-bye.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_lib_objects_cond): Don't take $LEX_SEEN as Akim> argument, as you don't use it. Akim> Hence... Akim> (&handle_lib_objects): Don't take $LEX_SEEN as argument, as you Akim> don't use it. Akim> Hence..

Re: 116-language-link.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> The goal is to handle LINK and the like just the way we handle Akim> COMPILE and the like. First I have to understand completely Akim> what's so special wrt C, but I'm confident we can reach Akim> something generic and shorter (many

Re: 115-define-compiler-var.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&define_compiler_variables): Use only $LANG as Akim> argument. Akim> (&handle_languages): Adjust. Ok. Tom

Re: 112-formatting-changes.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in: Formatting changes. Akim> (variable_dump, variables_dump): Rename as... Akim> (macro_dump, macros_dump): these. Ok. I think if we agree on what something should be called then simple renaming patches like this d

Re: 111-if-bang.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> Support `if !COND', `else COND', `end COND'. Akim> * automake.texi (Conditionals): Document it. Akim> * automake.in ($WHITE_PATTERN, $MACRO_PATTERN, $BOGUS_MACRO_PATTERN) Akim> ($GNITS_VERSION_PATTERN, $INCLUDE_PATTERN): Use

Re: 110-per-object-rules.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> Uniform handling of per-object compilation rules. Akim> * automake.in (&handle_languages): Output per object rules for all Akim> the objects, not only for those which language supports dependency Akim> tracking. Akim> When

Re: crosscompiling dll linux->mingw32

2001-05-05 Thread Gary V . Vaughan
On Saturday 05 May 2001 11:34 pm, Tom Tromey wrote: > > "Gary" == Gary V Vaughan <[EMAIL PROTECTED]> writes: > > Gary> A short while before the libtool-1.4 release, Edward M. Lee > Gary> posted a joint patch to libtool and automake which takes care of > Gary> the worst of this sort of thing.

Re: 109-handle-languages-morphing-9.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_languages): `ext-compile.am' and Akim> `depend2.am' are now equivalent for generic rules: output only the Akim> latter. Akim> * ext-compile.am: Remove. Ok. Tom

Re: 108-handle-languages-morphing-8.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (Language): Replace the attribute `output_arg' with Akim> `compile_flag' and `output_flag'. Akim> (Automake): Adjust language registrations. Akim> (&handle_languages): Transform `-c' and `-o' for both suffix and

Re: 107-handle-languages-morphing-7.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_languages): Don't transform %COMPILER%. Akim> Use `$lang->compiler' instead of `$pfx' to transform generic Akim> %COMPILE% and %LTCOMPILE%. Akim> * ext-compile.am: Use %COMPILE%, %COMPILER% and %SOURCE%

Re: 106-handle-languages-morphing-6.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_languages): Use the same `%transform' for Akim> both `depend2.am' and `ext-compile.am'. Akim> Move the definition of `$flag' where it is used, and rename as Akim> `$flags'. Ok. Tom

Re: 104-handle-languages-morphing-5.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_languages): Compute `$ltoutarg' and Akim> `$outarg' independently from dependency code. Akim> There is no use looping on a language's possible extensions since Akim> we loop over used extensions. Akim>

Re: 103-handle-languages-morphing-4.patch

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&handle_languages): Don't use $comp. Ok. Tom

Re: 97.5-merge-finish-languages-dependencies.patch (Was: 98-no-depend2.patch)

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: Akim> * automake.in (&finish_languages): Rename as... Akim> (&handle_languages): this. Akim> Include the body of... Akim> (&handle_dependency): this. Akim> Remove. Ok. Tom

Re: crosscompiling dll linux->mingw32

2001-05-05 Thread Tom Tromey
> "Gary" == Gary V Vaughan <[EMAIL PROTECTED]> writes: Gary> A short while before the libtool-1.4 release, Edward M. Lee Gary> posted a joint patch to libtool and automake which takes care of Gary> the worst of this sort of thing. I applied the libtool parts Gary> before the release but the

Re: libtool 1.4: "make uninstall" broken?

2001-05-05 Thread Tom Tromey
> "Peter" == Peter Eisentraut <[EMAIL PROTECTED]> writes: Peter> The reason is that install-info is often installed under /sbin, Peter> so it's not in the ordinary user's path. The "install" targets Peter> work around this by not calling "install-info" in that case, Peter> but "uninstall" do

Re: --add-missing broken

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > "Ralf" == Ralf Corsepius <[EMAIL PROTECTED]> writes: Ralf> Related question: Which directory is am_dir/--am-dir supposed to Ralf> point to, now? <..>/automake/ or <..>/automake/am? Akim> Good question. No idea what Tom will prefer.

Re: symlinked files break "dist"-target

2001-05-05 Thread Tom Tromey
> "Felix" == Felix Natter <[EMAIL PROTECTED]> writes: Felix> I created a minimal "hello-world"-package for this: Felix> http://www.ndh.net/home/natter/hello-1.0.tar.gz Congratulations -- you found a `make dist' oddity that we never considered. The problem is that by default we use `ln' to m

Re: unnecessary suffix handling?

2001-05-05 Thread Tom Tromey
> "Dave" == Dave Morrison <[EMAIL PROTECTED]> writes: Dave> I have a project in which there are some header files (they Dave> happen to have the suffix `.idl') that I want installed. Dave> Nothing else in the project depends directly on them, I'd just Dave> like to install them. Dave> includ

Re: Automake::Struct II

2001-05-05 Thread Tom Tromey
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes: > "Russ" == Russ Allbery <[EMAIL PROTECTED]> writes: Russ> Since it's pretty unlikely that there's going to be another Russ> Automake::Class floating around, that should be equivalent, Russ> except that it has the nice plus of then let

Re: $MAKE and ansi.test

2001-05-05 Thread Tom Tromey
> "Patrick" == Patrick Welche <[EMAIL PROTECTED]> writes: Patrick> From configure: Patrick> checking whether make sets ${MAKE}... (cached) yes This means that make sets the make variable `MAKE'... Patrick> From defs: Patrick> # User can set MAKE to choose which make to use. Must use GN

Re: Warning fix for automake on CVS Libtool

2001-05-05 Thread Tom Tromey
> "Pavel" == Pavel Roskin <[EMAIL PROTECTED]> writes: Pavel> * automake.in (scan_texinfo_file): Don't push undefined values Pavel> to @clean_suffixes. Pavel> -push @clean_suffixes, $predefined_index{$1}; Pavel> +push @clean_suffixes, $predefined_index{$1} Pavel> +

Re: PATCH: subdirs and conditionals

2001-05-05 Thread Tom Tromey
> "Richard" == Richard Boulton <[EMAIL PROTECTED]> writes: Richard> The gstreamer project, however, has a Makefile.am with Richard> conditionals for each of a set of conditions dictating which Richard> SUBDIRS are to be built. The file currently has 16 separate Richard> conditionally defined

Re: register_language() ?

2001-05-05 Thread Tom Tromey
> "Pavel" == Pavel Roskin <[EMAIL PROTECTED]> writes: Pavel> Applied after a short fight with CVS. The version you are using Pavel> on sourceware.cygnus.com tends to commit only some of the files Pavel> in a single run (either only those from the current directory Pavel> or all files except t

Re: PATCH: Fix dependency tracking (revised)

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Boehne <[EMAIL PROTECTED]> writes: Robert> 2001-04-25 Robert Boehne <[EMAIL PROTECTED]> Robert> * configure.in: Add _am_dep_true='@AMDEP_TRUE@' and AC_SUBST it. Robert> * automake.in: (handle_dependencies) Replace @AMDEP_TRUE@ in Robert> $output_r

Re: More CVS head errors

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: Robert> I'm currently having to distribute two patches for automake to Robert> achieve the pass on make distcheck - the openbsd install-am Robert> target, and the workaround I put together for subobj5.test. If Robert> you guys have some

Re: patch: missing uninstall-am target breaks non-gnu make

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: Robert> This is just a patch for the fix suggested to me. Robert> + 'uninstall-am' =3D> 1,=0A= I checked this in. Thanks. Tom

Re: Bug when specifying --disable-dependency-tracking (AM_DEP_TRACK)

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Boehne <[EMAIL PROTECTED]> writes: Robert> foreach my $iter (@deplist) Robert> { Robert> $output_rules .= (' @_am_include@ @_am_quote@' Robert> . $iter . '@_am_quote@' . "\n"); Robert> }

Re: Bad dependencies generation

2001-05-05 Thread Tom Tromey
> "Paul" == Paul F Kunz <[EMAIL PROTECTED]> writes: Paul> To answer my own question about bad dependency generation with Paul> the combination of Red Hat 7.1beta, gcc 2.96, and Paul> autoconf/automake released version. It is gcc 2.96 causing the Paul> problem, as it went away after installin

Re: target dependency bug

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Collins <[EMAIL PROTECTED]> writes: I finally read this thread. Robert> cf_gen_SOURCES = cf_gen.c defines.h Robert> nodist_cf_gen_SOURCES = cf_gen_defines.h Robert> BUILT_SOURCES = cf_gen_defines.h Robert> I think cf_gen_defines.h should be one of the dependencies of Ro

Re: @_am_quote@ not replaced by 'configure'

2001-05-05 Thread Tom Tromey
> "Robert" == Robert Boehne <[EMAIL PROTECTED]> writes: Robert> On a related note, I wonder if it would be reasonable to add a Robert> test case to Automake similar to the style that Libtool uses. Robert> A test that actually compiles code and attempted to use Robert> dependencies, invoke lib

Adv: Featured in the Wall Street Journal!

2001-05-05 Thread seanb
All our mailings are sent complying to the proposed United States Federal requirements for commercial e-mail: Section 301 Paragraph (a) (2) (C) of S.618. Please see the bottom of this message for further information and removal instructions. Dear Friend & Future Millionaire, YOU REALLY OWE

Re: Help!

2001-05-05 Thread Tom Tromey
> "David" == Zhu, David <[EMAIL PROTECTED]> writes: David> Could you send me examples of how to use autoconf and automake. http://sources.redhat.com/autobook/ Tom

Re: symlinked files break "dist"-target

2001-05-05 Thread Felix Natter
Alexandre Oliva <[EMAIL PROTECTED]> writes: > On May 2, 2001, Felix Natter <[EMAIL PROTECTED]> wrote: > > > I have some source-files which are shared between some autoconf/automake- > > projects. So I use symlinks in my projects. > > > But using automake, this breaks the dist-target ("make dis