Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 629246 - app-accessibility/espeak-1.48.04-r1::gentoo:there are files installed outside the prefix
Summary: app-accessibility/espeak-1.48.04-r1::gentoo:there are files installed outside...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-08-29 00:33 UTC by tastu teche
Modified: 2021-01-06 20:29 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
espeak-1.48.04-r1.ebuild with DATADIR prefixed and EROOT changed from ROOT (espeak-1.48.04-r1.ebuild,2.33 KB, text/plain)
2017-08-29 00:44 UTC, tastu teche
Details

Note You need to log in before you can comment on or make changes to this bug.
Description tastu teche 2017-08-29 00:33:31 UTC
* /usr/share/espeak-data/ro_dict
 * /usr/share/espeak-data/hbs_dict
 * ERROR: app-accessibility/espeak-1.48.04-r1::gentoo failed:
 *   Aborting due to QA concerns: there are files installed outside the prefix

Reproducible: Always

Steps to Reproduce:
1. emerge app-accessibility/espeak
Comment 1 tastu teche 2017-08-29 00:40:16 UTC
my fix:
--- /r/a/p/usr/portage/app-accessibility/espeak/espeak-1.48.04-r1.ebuild
+++ /r/a/p/usr/local/portage/app-accessibility/espeak/espeak-1.48.04-r1.ebuild
@@ -47,6 +47,7 @@
 
 src_compile() {
 	emake \
+		DATADIR="${EPREFIX}/usr/share/espeak-data" \
 		PREFIX="${EPREFIX}/usr" \
 		CC="$(tc-getCC)" \
 		CFLAGS="${CFLAGS}" \
@@ -59,6 +60,7 @@
 	einfo "Fixing byte order of phoneme data files"
 	pushd ../platforms/big_endian > /dev/null
 	emake \
+		DATADIR="${EPREFIX}/usr/share/espeak-data" \
 		CC="$(tc-getCC)" \
 		CFLAGS="${CFLAGS}"
 	./espeak-phoneme-data \
@@ -70,6 +72,7 @@
 
 src_install() {
 	emake \
+		DATADIR="${EPREFIX}/usr/share/espeak-data" \
 		PREFIX="${EPREFIX}/usr" \
 		LIBDIR="\$(PREFIX)/$(get_libdir)" \
 		DESTDIR="${D}" \
@@ -84,7 +87,7 @@
 }
 
 pkg_preinst() {
-	local voicedir="${ROOT}/usr/share/${PN}-data/voices/en"
+	local voicedir="${EROOT}/usr/share/${PN}-data/voices/en"
 	if [ -d "${voicedir}" ]; then
 		rm -rf "${voicedir}"
 	fi
Comment 2 tastu teche 2017-08-29 00:44:27 UTC
Created attachment 491036 [details]
espeak-1.48.04-r1.ebuild with DATADIR prefixed and EROOT changed from ROOT
Comment 3 Fabian Groffen gentoo-dev 2021-01-06 20:29:13 UTC
this is indeed necessary, yet the latest version of the ebuild changed.  We'd have to revisit this.