Gentoo Archives: gentoo-dev

From: Sam James <sam@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH] autotools.eclass: eclassdoc, cosmetic changes, drop old EAPIs
Date: Wed, 31 Mar 2021 05:43:59
Message-Id: E76C2631-9C66-4AD1-9EFE-D48B1374BBCA@gentoo.org
In Reply to: [gentoo-dev] [PATCH] autotools.eclass: eclassdoc, cosmetic changes, drop old EAPIs by Sam James
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

Attachments

File name MIME type
signature.asc application/pgp-signature