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 |