1 |
On 11/06/17 04:02 PM, Mart Raudsepp wrote: |
2 |
> Ühel kenal päeval, P, 11.06.2017 kell 13:08, kirjutas William Hubbs: |
3 |
>> On Sun, Jun 11, 2017 at 07:14:52PM +0300, Mart Raudsepp wrote: |
4 |
>>> Ühel kenal päeval, P, 11.06.2017 kell 11:12, kirjutas William |
5 |
>>> Hubbs: |
6 |
>>>> On Sun, Jun 11, 2017 at 05:35:53PM +0200, Kristian Fiskerstrand |
7 |
>>>> wrote: |
8 |
>>>>> On 06/11/2017 05:24 PM, David Seifert wrote: |
9 |
>>>>>>> We can always patch the eclass at that point if that is |
10 |
>>>>>>> still a |
11 |
>>>>>>> big |
12 |
>>>>>>> concern, but I fundamentally agree with William on this, |
13 |
>>>>>>> starting |
14 |
>>>>>>> point |
15 |
>>>>>>> should be fixing it upstream, so can start with a tracking |
16 |
>>>>>>> bug |
17 |
>>>>>>> on |
18 |
>>>>>>> affected packages. |
19 |
>>>>>>> |
20 |
>>>>>> |
21 |
>>>>>> Here's a deal: you can start filing + fixing them. |
22 |
>>>>>> |
23 |
>>>>> |
24 |
>>>>> The [tracker] is already started, it was determined to add QA |
25 |
>>>>> warning |
26 |
>>>>> with info on filing upstream bugs in [bug 426262] and the |
27 |
>>>>> proper |
28 |
>>>>> solution is again iterated in [bug 546614], so its not like |
29 |
>>>>> this is |
30 |
>>>>> a |
31 |
>>>>> new approach that is being suggested, but sure, we should all |
32 |
>>>>> file |
33 |
>>>>> bugs |
34 |
>>>>> when we encounter them. |
35 |
>>>>> |
36 |
>>>>> References: |
37 |
>>>>> [tracker] https://bugs.gentoo.org/show_bug.cgi?id=530632 |
38 |
>>>>> |
39 |
>>>>> [bug 426262] |
40 |
>>>>> https://bugs.gentoo.org/show_bug.cgi?id=426262 |
41 |
>>>>> |
42 |
>>>>> [bug 546614] |
43 |
>>>>> https://bugs.gentoo.org/show_bug.cgi?id=546614 |
44 |
>>>> |
45 |
>>>> It seems that the proper fix to this, even for a package that |
46 |
>>>> won't |
47 |
>>>> do |
48 |
>>>> the fix upstream is to use WANT_AUTOCONF in the ebuild to force |
49 |
>>>> the |
50 |
>>>> version of autoconf you need. |
51 |
>>> |
52 |
>>> No. It appears you don't know how WANT_AUTOCONF works. |
53 |
>> |
54 |
>> |
55 |
>> From the eclass: |
56 |
>> |
57 |
>> # @ECLASS-VARIABLE: WANT_AUTOCONF |
58 |
>> # @DESCRIPTION: |
59 |
>> # The major version of autoconf your package needs |
60 |
>> |
61 |
>> That leads me to believe that you can set WANT_AUTOCONF="someversion" |
62 |
>> in your ebuild and your package will use that version of autoconf. |
63 |
>> |
64 |
>> If that's wrong, it needs to be fixed and that's a different bug |
65 |
>> entirely, but it doesn't change my position on adding this particular |
66 |
>> change to autotools.eclass. |
67 |
> |
68 |
> It is the major version, as documented. Yes, it could specify what |
69 |
> these valid versions currently are, as they really are: |
70 |
> |
71 |
> none |
72 |
> 2.1 |
73 |
> 2.5 |
74 |
> latest |
75 |
> |
76 |
> Currently latest equals 2.5 and is the default. |
77 |
> |
78 |
> In practice, none means not to add an autoconf atom to DEPEND (even |
79 |
> with the default AUTOTOOLS_AUTO_DEPEND=yes) - if ebuild/eclass |
80 |
> inheriting autotools.eclass handles its own exact autoconf depend atom |
81 |
> (eautoconf will get called in eautoreconf regardless). Only main tree |
82 |
> consumer of this appears to be gtk-sharp-module.eclass that adds its |
83 |
> own autoconf/automake atoms when needed, because eautoreconf is |
84 |
> conditional by variables used by that eclass and it needed autoconf |
85 |
> 2.61 or newer, not just 2.59. |
86 |
> |
87 |
> 2.1 means autoconf:2.1 |
88 |
> |
89 |
> 2.5 and latest currently means >=autoconf-2.59 |
90 |
> |
91 |
> Patches welcome to documentation, I suppose. |
92 |
> |
93 |
> |
94 |
> It is usually a bad sign if you need to change it away from latest for |
95 |
> some reason in the ebuild and ideally they'd all be the default |
96 |
> (latest). |
97 |
> |
98 |
> There was no configure.ac before autoconf-2.50, only configure.in, and |
99 |
> thus a warning doesn't make sense. The real warning here is the need |
100 |
> for WANT_AUTOCONF=2.1 |
101 |
> |
102 |
|
103 |
I'm not seeing the issue with what William's suggesting -- if the |
104 |
configure.in is non-trivial to just rename in the ebuild, then setting |
105 |
WANT_AUTOCONF=2.1 would seem to me to be the correct course of action. |
106 |
If there's a package that has a configure.in but also needs |
107 |
autoconf:2.5, well, that seems messed up and likely should be fixed in |
108 |
the ebuild (or upstream) too. |