Gentoo Archives: gentoo-portage-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCHES] Random issues and refactoring before EAPI6 series
Date: Fri, 12 Sep 2014 07:18:16
Message-Id: 20140912091511.5d0313cd@pomiot.lan
In Reply to: [gentoo-portage-dev] [PATCHES] Random issues and refactoring before EAPI6 series by "Michał Górny"
1 Dnia 2014-08-18, o godz. 19:51:35
2 Michał Górny <mgorny@g.o> napisał(a):
3
4 > dol-sen asked me to send all of them in a serie. Those are minor
5 > bugfixes and code refactoring done to prepare for EAPI6 code.
6 >
7 > 1 & 2 cleans up econf. In EAPI6 we will likely be adding --docdir
8 > and --htmldir there, so we should add it in nice code rather than
9 > copying the bad one.
10 >
11 > 3-4 aim to clean up EAPI handling a bit. It's far from perfect, but 3 at
12 > least fixes the bug that EAPI 5 is considered unsupported. Further
13 > changes can be done in the future.
14 >
15 > 5 fixes PMS incompatibility that causes 'nonfatal' to prevent 'die' from
16 > working correctly. I'm pretty sure the bug was mentioned by PMS team
17 > already but nobody considered it important enough to report a bug. EAPI
18 > 6 'nonfatal die' support will land in place of that conditional.
19 >
20 > 6 aims to simplify the phase setting function that is hard to understand
21 > and the moment, and has a few unnecessary, hard-to-understand side
22 > effects like declaring non-existent __eapi2_src_*() phases. The new code
23 > is much cleaner and makes adding new phases in future EAPIs trivial. It
24 > also prevents from calling default_* function for another phase as
25 > forbidden by PMS.
26
27 1-4 & 6 committed.
28
29 5 needs a second look since it seems that not all functions use
30 __helper_die correctly. While at it, __eapi0_src_test should use emake,
31 and it has comment about nonfatal emake being too verbose -- we should
32 investigate that too. It doesn't make sense for nonfatal logic to
33 output warnings.
34
35 --
36 Best regards,
37 Michał Górny

Attachments

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