1 |
On 11/29/2016 07:05 PM, Mike Gilbert wrote: |
2 |
> On Tue, Nov 29, 2016 at 6:24 PM, Mike Pagano <mpagano@g.o> wrote: |
3 |
>> 2nd round. As per mgorny's advice, fix eapply_user as per PMS spec and |
4 |
>> execute in src_prepare. Support older EAPIs with epatch_user. |
5 |
>> |
6 |
>> --- |
7 |
>> eclass/kernel-2.eclass | 25 ++++++++++++++++++------- |
8 |
>> 1 file changed, 18 insertions(+), 7 deletions(-) |
9 |
>> |
10 |
>> diff --git a/eclass/kernel-2.eclass b/eclass/kernel-2.eclass |
11 |
>> index 3f5fb3b..e516aff 100644 |
12 |
>> --- a/eclass/kernel-2.eclass |
13 |
>> +++ b/eclass/kernel-2.eclass |
14 |
>> @@ -95,7 +95,7 @@ has "${EAPI:-0}" 0 1 2 3 4 5 || die "kernel-2.eclass |
15 |
>> is unsupported for EAPI ${E |
16 |
>> PYTHON_COMPAT=( python{2_6,2_7} ) |
17 |
>> |
18 |
>> inherit eutils toolchain-funcs versionator multilib python-any-r1 |
19 |
>> -EXPORT_FUNCTIONS pkg_setup src_unpack src_compile src_test src_install |
20 |
>> pkg_preinst pkg_postinst pkg_postrm |
21 |
>> +EXPORT_FUNCTIONS pkg_setup src_prepare src_unpack src_compile src_test |
22 |
>> src_install pkg_preinst pkg_postinst pkg_postrm |
23 |
> |
24 |
> I don't think calling "EXPORT_FUNCTIONS src_prepare" is legal in EAPI |
25 |
> 0 or 1. You should probably move this behind an EAPI check. |
26 |
> |
27 |
> Also, for EAPI, 2, 3, 4, and 5, this change might break existing |
28 |
> ebuilds; they may define src_prepare themselves, or inherit some other |
29 |
> eclass that also exports src_prepare. |
30 |
> |
31 |
> Otherwise, this seems like a good next step. |
32 |
> |
33 |
|
34 |
Thank-you, I did see that in another eclass. (EXPORT) |
35 |
I will revise and resend tomorrow. |