From: Jonas Stein <jstein@gentoo.org>
To: gentoo-dev <gentoo-dev@lists.gentoo.org>
Subject: [gentoo-dev] rfc: Remove inherit eutils from font.eclass for EAPI=6
Date: Wed, 14 Feb 2018 23:43:55 +0100 [thread overview]
Message-ID: <98bd92fe-6583-a9e9-0590-2fe9204ff2ec@gentoo.org> (raw)
[-- Attachment #1.1: Type: text/plain, Size: 2490 bytes --]
Hi,
I think we do not need
inherit eutils
https://github.com/gentoo/gentoo/blob/master/eclass/font.eclass#L9
for the case that EAPI=6
I suggest to change the line to
[[ ${EAPI:-0} == [012345] ]] && inherit eutils
we would have to test the following packages (the grep line does not
list matches with '\' line breaks)
$ grep -rI 'inherit.*font' --include='*.ebuild' -l |xargs grep EAPI
|grep ':.*6' |cut -d: -f1
app-misc/xmind/xmind-3.7.6_p201801311814.ebuild
media-fonts/essays1743/essays1743-2.000.ebuild
media-fonts/essays1743/essays1743-2.001.ebuild
media-fonts/fontawesome/fontawesome-5.0.4.ebuild
media-fonts/fontawesome/fontawesome-4.7.0.ebuild
media-fonts/baekmuk-fonts/baekmuk-fonts-2.2-r2.ebuild
media-fonts/terminus-font/terminus-font-4.46.ebuild
media-fonts/terminus-font/terminus-font-4.40.ebuild
media-fonts/unifont/unifont-10.0.05.ebuild
media-fonts/unifont/unifont-10.0.04.ebuild
media-fonts/unifont/unifont-9.0.06.ebuild
media-fonts/unifont/unifont-10.0.06.ebuild
media-fonts/jisx0213-fonts/jisx0213-fonts-20040425-r2.ebuild
media-fonts/paratype-astra/paratype-astra-1001.ebuild
media-fonts/ahem/ahem-1.0.ebuild
media-fonts/nunito/nunito-1.0-r1.ebuild
media-fonts/stix-fonts/stix-fonts-2.0.0.ebuild
media-fonts/dina/dina-2.93.ebuild
media-fonts/mplus-outline-fonts/mplus-outline-fonts-0_pre062.ebuild
media-fonts/mplus-outline-fonts/mplus-outline-fonts-0_pre063.ebuild
media-fonts/umeplus-fonts/umeplus-fonts-20160402.ebuild
media-fonts/opendesktop-fonts/opendesktop-fonts-1.4.2.ebuild
media-fonts/noto/noto-20170403.ebuild
media-fonts/roboto/roboto-2.138.ebuild
media-fonts/hack/hack-3.001.ebuild
media-fonts/noto-cjk/noto-cjk-20150615.ebuild
media-fonts/cm-unicode/cm-unicode-0.7.0-r1.ebuild
media-fonts/comic-neue/comic-neue-2.3.ebuild
media-fonts/dejavu/dejavu-2.37.ebuild
media-fonts/source-han-sans/source-han-sans-1.004.ebuild
media-fonts/croscorefonts/croscorefonts-1.31.0.ebuild
media-fonts/open-sans/open-sans-1-r1.ebuild
media-fonts/heuristica/heuristica-1.0.2.ebuild
media-fonts/source-pro/source-pro-20170111.ebuild
media-fonts/source-pro/source-pro-20160608.ebuild
kde-plasma/oxygen-fonts/oxygen-fonts-5.4.3.ebuild
app-office/wps-office/wps-office-10.1.0.5707_alpha21-r1.ebuild
app-office/lyx/lyx-2.2.3-r1.ebuild
app-office/lyx/lyx-2.2.3-r2.ebuild
Did I miss something?
Who can help to check with (an automatic) testenvironment, if these
packages will survive?
--
Best,
Jonas
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 981 bytes --]
next reply other threads:[~2018-02-14 22:44 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-14 22:43 Jonas Stein [this message]
2018-02-15 1:16 ` [gentoo-dev] rfc: Remove inherit eutils from font.eclass for EAPI=6 Mart Raudsepp
2018-02-15 7:46 ` Guilherme Amadio
2018-02-15 11:07 ` Jonas Stein
2018-02-15 12:07 ` Mart Raudsepp
2018-05-05 19:52 ` Jonas Stein
2018-02-15 6:42 ` Ulrich Mueller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=98bd92fe-6583-a9e9-0590-2fe9204ff2ec@gentoo.org \
--to=jstein@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox