1 |
> On 27 Mar 2021, at 22:21, Sam James <sam@g.o> wrote: |
2 |
> |
3 |
|
4 |
Given how uncontroversial most of it is, the positive reviews, and the fact that I pushed a load |
5 |
of other eclassdoc changes (for other eclasses) but CI is still down, it makes |
6 |
sense to throw in as much as we can to not waste the big cache regeneration that’s going to occur |
7 |
anyway. |
8 |
|
9 |
I’ve therefore pushed the uninteresting parts of this (all cosmetic for eclassdocs or whitespace, |
10 |
nothing functional involving EAPIs or anything like that). |
11 |
|
12 |
I’ll send out a v2 either later today or tomorrow with the parts I’ve cherry-picked removed. |
13 |
|
14 |
v2 will look roughly like: |
15 |
> (Relatively) significant changes: |
16 |
> * inherit eutils for < EAPI 7 for eqawarn |
17 |
> * drop support for < EAPI 5 |
18 |
> |
19 |
> [Needed for the EBUILD_PHASE_FUNC change. |
20 |
> < EAPI 5 is no longer in ::gentoo (since December).] |
21 |
> |
22 |
> eclassdoc fixes: |
23 |
> * explicitly blank and mark variables as @DEFAULT_UNSET |
24 |
> * add @DESCRIPTION for _at_uses_pkg |
25 |
> * document AUTOTOOLS_DEPEND |
26 |
> |
27 |
> Cosmetic changes: |
28 |
> * convert ewarn to eqawarn |
29 |
> * consistent 'case' style |
30 |
> * consistent variable references |
31 |
> * consistent use of ${ECLASS}, not "autotools.eclass" |
32 |
> * use same ${WANT_AUTOCONF} comparison test throughout |
33 |
|
34 |
If we’re sticking with the dropping of EAPIs earlier than 5, I think there’s some more cruft I can clear out too. |
35 |
|
36 |
So, as you can see, not much interesting actually got cherry-picked. For transparency - although |
37 |
you could read git yourself, I cherry-picked the following: |
38 |
|
39 |
$ git shortlog 3f1cd0c161eeec3a5b0173da31f8949fa5eb38b5..HEAD |
40 |
Sam James (6): |
41 |
autotools.eclass: mark AT_M4DIR as @DEFAULT_UNSET |
42 |
autotools.eclass: mark WANT_AUTO*, AUTOTOOLS_AUTO_DEPEND as @PRE_INHERIT |
43 |
autotools.eclass: mark AT_SYS_M4DIR variable as @DEFAULT_UNSET |
44 |
autotools.eclass: consistent references to bugs in comments |
45 |
autotools.eclass: fix whitespace/phrasing in comment |
46 |
autotools.eclass: minor cosmetic changes to various elogs |