1 |
>>>>> On Fri, 24 May 2019, Mike Gilbert wrote: |
2 |
|
3 |
> On Thu, May 23, 2019 at 7:16 PM David Seifert <soap@g.o> wrote: |
4 |
>> Given that there are no ebuilds in the tree using this eclass and being |
5 |
>> in EAPI 0, 1 or 2 ( |
6 |
>> https://qa-reports.gentoo.org/output/eapi-per-eclass/savedconfig.eclass/ |
7 |
>> ), wouldn't it make more sense to just whitelist EAPI >= 4 and clean up |
8 |
>> this backwards compatibility cruft instead? |
9 |
|
10 |
> I'm fixing a bug with the least invasive change possible. I'm not |
11 |
> trying to rework the eclass. |
12 |
|
13 |
AFAICS, that backwards compatibility code consists of two case |
14 |
statements, and the chance that removing them would break anything is |
15 |
close to zero. So I wouldn't call it a "rework". :) |
16 |
|
17 |
I'd rather remove than update that code for deprecated EAPIs. No ebuild |
18 |
would ever use it, so your updated code would never be tested. |
19 |
|
20 |
Ulrich |