| 1 |
On Thu, Feb 15, 2018 at 03:16:22AM +0200, Mart Raudsepp wrote: |
| 2 |
> On Wed, 2018-02-14 at 23:43 +0100, Jonas Stein wrote: |
| 3 |
> > Did I miss something? |
| 4 |
> > Who can help to check with (an automatic) testenvironment, if these |
| 5 |
> > packages will survive? |
| 6 |
> |
| 7 |
> Don't check with test environments, read the ebuilds. |
| 8 |
|
| 9 |
I agree with this. Font ebuilds are usually quite trivial, so it might |
| 10 |
even be faster to just read them. The font.eclass has not been touched |
| 11 |
since the move to git, but I suspected few fonts would actually use |
| 12 |
eutils, so here is another grep that shows it may be easier than we |
| 13 |
think to get rid of inherit eutils, even if for all EAPIs: |
| 14 |
|
| 15 |
https://devmanual.gentoo.org/eclass-reference/eutils.eclass/index.html |
| 16 |
|
| 17 |
gentoo $ export eutils_funcs=(eqawarn ecvs_clean esvn_clean \ |
| 18 |
egit_clean emktemp edos2unix strip-linguas built_with_use make_wrapper \ |
| 19 |
path_exists use_if_iuse optfeature epause ebeep usex einstalldocs \ |
| 20 |
in_iuse) |
| 21 |
gentoo $ for func in ${eutils_funcs[@]}; do grep -rI 'inherit.*font' --include='*.ebuild' -l | \ |
| 22 |
xargs grep $func; done | cut -d: -f1 | sort | uniq |
| 23 |
|
| 24 |
app-office/lyx/lyx-2.2.3-r1.ebuild |
| 25 |
app-office/lyx/lyx-2.2.3-r2.ebuild |
| 26 |
media-fonts/terminus-font/terminus-font-4.39-r1.ebuild |
| 27 |
media-fonts/terminus-font/terminus-font-4.40.ebuild |
| 28 |
media-fonts/terminus-font/terminus-font-4.46.ebuild |
| 29 |
media-fonts/unifont/unifont-10.0.04.ebuild |
| 30 |
media-fonts/unifont/unifont-10.0.05.ebuild |
| 31 |
media-fonts/unifont/unifont-10.0.06.ebuild |
| 32 |
media-fonts/unifont/unifont-9.0.06.ebuild |
| 33 |
net-misc/suite3270/suite3270-3.4_p10.ebuild |
| 34 |
net-misc/suite3270/suite3270-3.5_p10.ebuild |
| 35 |
net-misc/suite3270/suite3270-3.5_p12.ebuild |
| 36 |
net-misc/suite3270/suite3270-3.6_p4.ebuild |
| 37 |
|
| 38 |
Most of the above only use usex, and terminus-font uses einstalldocs as well. |
| 39 |
I think these should be pretty easy to fix to not use eutils, or simply |
| 40 |
add 'inherit eutils' for these ebuilds, then remove from the eclass. |
| 41 |
|
| 42 |
I can help with testing by reading ebuilds more carefully and emerging |
| 43 |
them after the change in font.eclass. |
| 44 |
|
| 45 |
Cheers, |
| 46 |
-Guilherme |
Archives