Akim --
Your recent patches broke a couple of test cases.
I looked at the `subobj4' failure. I think the new code in
add_depend2 is wrong.
First, this is wrong:
. &transform ('$(' . $pfx . 'COMPILE)'
=> $rule,
&transform puts @...@ around the
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
Akim> + local ($traces) = "$ENV{amtraces} ";
Well, I'm applying this anyway, as , but for this envvar, you OK'ed
it. It makes it possible to toy with it for those who read this list.
It'll be easy to clean up.
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
Akim> * texinfos.am, tags.am, subdirs.am, multilib.am, mans.am: Add a
Akim> .PHONY target. * automake.in (handle_texinfo, handle_man_pages,
Akim> handle_multilib) (handle_etags, handle
| Akim> +local ($traces) = "$ENV{amtraces} ";
|
| Do we really want to use an environment variable? Eventually automake
| will default to using the autoconf trace facility. At that time I
| suppose we'll generate a rule in Makefile.in that looks like:
|
| $(AUTOMAKE) --autoconf=$(AUTO
On Mon, Jan 29, 2001 at 07:21:21AM -0200, Alexandre Oliva wrote:
> On Jan 29, 2001, Steve Robbins <[EMAIL PROTECTED]> wrote:
>
> > There is a patch for "automake" in the CVS libtool README.
> > Apart from that, are there any things I need to look out for,
> > if I mix CVS libtool with the release
On Jan 29, 2001, Steve Robbins <[EMAIL PROTECTED]> wrote:
> There is a patch for "automake" in the CVS libtool README.
> Apart from that, are there any things I need to look out for,
> if I mix CVS libtool with the released automake and autoconf?
Not AFAIK.
--
Alexandre Oliva Enjoy Guarana',
| Akim> + &prog_error ("version is incorrect: $VERSION")
| Akim> + if ($VERSION !~ /([0-9]+)\.([0-9]+)([a-z])?/);
|
| The automake style is to write trailing `if's without parens:
|
| + &prog_error ("version is incorrect: $VERSION")
| +
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
Tom> I think we should add `require 5.0', or whatever it is (my
Tom> knowledge of Perl after Perl 4 is abysmal) just to ensure that
Tom> the wrong Perl is never invoked.
OK.
Tom> I think the `join remove' part of the patch requires Perl 5
| > "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
|
| Akim> * automake.in (handle_phony): Sort.
| Akim> * automake.in (file_contents_with_transform): Catch `.PHONY:'.
| Akim> * check.am: New file.
|
| check.am wasn't in the patch. However, I trust it is just a
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
Tom> I didn't read this in enough detail to verify that all that you
Tom> did was remove the `if'. However, I assume that is so.
Yep, it is.
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
Tom> The parens are there so that tab in Emacs will do the right
Tom> thing. Could you please add them back? Then this is ok.
OK, and I'll use them in the future. Thanks!
BTW, if you know the Emacs settings you use for automake.in, would
> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:
> "Akim" == Akim Demaille <[EMAIL PROTECTED]> writes:
Tom> Aii! I was dreading the day that the Demaillator turned his
Tom> sights on automake...
:) :) :)
Tom> and now it has arrived!
Pavel told me to leave Autoconf in peace ;)
12 matches
Mail list logo
