automake
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] build: use AC_CONFIG_HEADERS, not AM_CONFIG_HEADER


From: Pavel Raiskup
Subject: Re: [PATCH] build: use AC_CONFIG_HEADERS, not AM_CONFIG_HEADER
Date: Tue, 05 Mar 2013 14:56:52 +0100

Dear Stefano, sorry for so late response,

> On 12/30/2012 10:35 AM, Paolo Bonzini wrote:
>> Il 29/12/2012 21:43, Stefano Lattarini ha scritto:
>>> On 12/29/2012 08:46 PM, Paolo Bonzini wrote:
>>>> Il 29/12/2012 17:32, Stefano Lattarini ha scritto:
>>>>> * configure.ac: Here.  The latter has been removed in Automake 1.13.
>>>>
>>>> Is there any reason for this,
>>>>
>>> Avoiding to keep too much cruft around the codebase, and smoothing
>>> possible future transitions to Automake-NG.
>>
>> Two lines of code are not "cruft".  It's only cruft if it happens
>> _outside_ the definition of AM_CONFIG_HEADER itself.
>>
>>>> apart from randomly breaking
>>>> perfectly-working packages?
>>>>
>>>> The right way to do this is to rely on the autoupdate machinery.
>>>>
>>> I thought I had deprecated this macro in the 1.12.x series already,
>>> with proper warnings.  Wasn't that the case?
>>
>> Deprecating is different from letting autoupdate convert it automatically.
>>
> At any rate, I agree the error message caused by the abrupt removal
> is horrible.  I'll soon post a patch to have still-exiting uses of
> AM_CONFIG_HEADER give a clear error message (as is done for the
> AM_C_PROTOTYPES since Automake 1.12).  Making that fix quickly
> available will be a good reason for a 1.13.1 release.

I'm still thinking about this resolution.  Could you please reconsider
again this situation?  We have in Fedora 18 about 700+ packages dependent
on automake.  I guess that other distributions have similar numbers.
Quite a lot of these packages are still dependent on AM_CONFIG_HEADER,
etc.

The future incompatibility is *not* big pain for developers; but mostly
for disto build systems :(.  Maintainers are thus forced not to do updates
for automake because of these problems ~> and users will not have then
easy access to the newest up2date automake source.  I know that because I
have done the automake update to 1.13 and it was **too** early.  My bad I
know - I should know that but it seems to be quite unnecessary.

Important to note is that I really don't want to make multiple packages like
automake113/automake114/(or whatever new version it will be).  I just
want to have one easy and stable package 'automake'.  I don't want to have the
same source in distribution multiple times — to fix some security/code problems
on multiple places each time they comes.

The only solution for me was revert as quickly as possible your changes —
re-add obsoleted macros back to automake downstream.  And next time I'll be
definitely **much more** careful.  Could you please look at it once again
please?  I can help you with patch preparation if you consider this as
suitable.  No big need for testing this — just re-add the AU_DEFUN statements
for old macros.

======================

I was thinking about this one quite a long time.  Firstly I was thinking
about "how to get rid of these macros in my distro".  There are some
possibilities worth to try.  But the main question is:  Why?! these old
macros are _still_ alive?  There are dead packages using legacy macros of
course - but it is a different story.  I think that the most important
thing is because you are not telling this clearly to users.  I just want
to say that the 'obsolete' warnings are disabled by default, why?  Could
we discuss this?  Btw., by grepping our spec files - I found three
packages using the -W option in autoconf/automake/autoreconf.

I don't want to set $WARNINGS system-wide.  Do you see another solutions?
I know that this more about autoconf - but still, you may stop me before I
raise autoconf mailing list.

First of all, please consider re-adding the obsolete macros back to
automake, it would be really appreciated.

Thanks for discussion :),
Pavel










reply via email to

[Prev in Thread] Current Thread [Next in Thread]