1 |
-----BEGIN PGP SIGNED MESSAGE----- |
2 |
Hash: SHA256 |
3 |
|
4 |
On 08/06/14 09:04 AM, Rich Freeman wrote: |
5 |
> Most of the proposed EAPI6 features seem ripe for acceptance, but |
6 |
> I figured I'd comment on a few here in case others want to weigh |
7 |
> in. |
8 |
>> b) User patches Bug #475288 [15], PMS wording [16] - Needs 4a) - |
9 |
>> Current wording of the spec requires that every ebuild must |
10 |
>> include a call to the function in src_prepare, which is |
11 |
>> controversial. - Names "apply_user_patches" or "eapply_user" have |
12 |
>> been suggested. |
13 |
>> |
14 |
> |
15 |
> I'm generally fine with it (we can bikeshed the name to death |
16 |
> later), but is there any reason to not just simply apply the |
17 |
> patches after src_prepare is done and not make maintainers call it? |
18 |
> If we're concerned about sequencing with other activities in |
19 |
> src_prepare then that is fine - I don't have a big problem with it |
20 |
> being a fatal error if it doesn't get called. |
21 |
|
22 |
|
23 |
General guidelines for epatch_user is that it should be placed after |
24 |
all ${S} manipulation but before eautoreconf. Otherwise anything that |
25 |
modifies configure.ac won't take effect. |
26 |
|
27 |
If src_prepare is defined, that would mean it'd have to be put right |
28 |
in the middle, still. If it isn't defined, then either an eclass |
29 |
would need to handle it as appropriate (i think many do; |
30 |
autotools-utils.eclass iirc has some code for this?), or the default |
31 |
src_prepare phase function would need to get a -lot- more complicated |
32 |
in order to handle patches that touch certain build-systems. |
33 |
|
34 |
Or would we just ban build-system patches from user patches? |
35 |
-----BEGIN PGP SIGNATURE----- |
36 |
Version: GnuPG v2.0.22 (GNU/Linux) |
37 |
|
38 |
iF4EAREIAAYFAlOVxU4ACgkQ2ugaI38ACPDiawD+KVaaVq5EKS9Gj+KVe6U0mjgP |
39 |
kZhVJEzPUuGDq+MygzgA/i7t43Uq4oIV4USZzTFCPRv9gtNFD8IOvx5B1P9bO8Ck |
40 |
=bsWX |
41 |
-----END PGP SIGNATURE----- |