Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 594712 - media-gfx/geeqie-1.3 - respect EPREFIX
Summary: media-gfx/geeqie-1.3 - respect EPREFIX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-21 18:22 UTC by Martin Mokrejš
Modified: 2016-11-09 12:46 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Mokrejš 2016-09-21 18:22:19 UTC
>>> Completed installing geeqie-1.3 into /scratch/mmokrejs/gentoo_rap/var/tmp/portage/media-gfx/geeqie-1.3/image/

 * Final size of build directory: 14118 KiB
 * Final size of installed tree: 1122 KiB

 * QA Notice: the following files are outside of the prefix:
 * /usr
 * /usr/share
 * /usr/share/doc
 * /usr/share/doc/geeqie-1.3
 * /usr/share/doc/geeqie-1.3/README
 * /usr/share/doc/geeqie-1.3/ChangeLog
 * /usr/share/doc/geeqie-1.3/README.lirc
 * /usr/share/doc/geeqie-1.3/TODO
 * ERROR: media-gfx/geeqie-1.3::gentoo failed:
 *   Aborting due to QA concerns: there are files installed outside the prefix
 * 
 * Call stack:
 *   misc-functions.sh, line 603:  Called install_qa_check
 *   misc-functions.sh, line 217:  Called source 'install_symlink_html_docs'
 *            05prefix, line 114:  Called install_qa_check_prefix
 *            05prefix, line  27:  Called die
 * The specific snippet of code:
 *                      die "Aborting due to QA concerns: there are files installed outside the prefix"
 * 
 * If you need support, post the output of `emerge --info '=media-gfx/geeqie-1.3::gentoo'`,
 * the complete build log and the output of `emerge -pqv '=media-gfx/geeqie-1.3::gentoo'`.
 * The complete build log is located at '/scratch/mmokrejs/gentoo_rap/var/tmp/portage/media-gfx/geeqie-1.3/temp/build.log'.
 * The ebuild environment file is located at '/scratch/mmokrejs/gentoo_rap/var/tmp/portage/media-gfx/geeqie-1.3/temp/environment'.
 * Working directory: '/scratch/mmokrejs/gentoo_rap/var/tmp/portage/media-gfx/geeqie-1.3/image/scratch/mmokrejs/gentoo_rap'
 * S: '/scratch/mmokrejs/gentoo_rap/var/tmp/portage/media-gfx/geeqie-1.3/work/geeqie-1.3'
Comment 1 Bernard Cafarelli gentoo-dev 2016-11-08 16:56:46 UTC
Thanks for the report, and sorry for the delay handling this.

I think I know where I have to fix the ebuild, but how can I trigger the prefix QA checks (to be sure I actually fix it)?

Anyway the fix should be to add ${EPREFIX} at the beginning of the "--with-readmedir" parameter
Comment 2 Martin Mokrejš 2016-11-09 12:09:17 UTC
This works for me:

--- geeqie-1.3.ebuild.old       2016-11-09 11:52:14.000000000 +0100
+++ geeqie-1.3.ebuild   2016-11-09 11:53:05.000000000 +0100
@@ -48,7 +48,7 @@
 src_configure() {
        # clutter does not compile, gps depends on clutter
        local myconf="--disable-dependency-tracking
-               --with-readmedir=/usr/share/doc/${PF}
+               --with-readmedir="${EPREFIX}"/usr/share/doc/${PF}
                $(use_enable debug debug-log)
                $(use_enable gpu-accel)
                $(use_enable gtk3)


And here is what was qmerged:

$ equery files geeqie
 * Searching for geeqie ...
 * Contents of media-gfx/geeqie-1.3:
/scratch/mmokrejs/gentoo_rap/usr
/scratch/mmokrejs/gentoo_rap/usr/bin
/scratch/mmokrejs/gentoo_rap/usr/bin/geeqie
/scratch/mmokrejs/gentoo_rap/usr/lib
/scratch/mmokrejs/gentoo_rap/usr/lib/geeqie
/scratch/mmokrejs/gentoo_rap/usr/lib/geeqie/geeqie-import
/scratch/mmokrejs/gentoo_rap/usr/lib/geeqie/geeqie-rotate
/scratch/mmokrejs/gentoo_rap/usr/lib/geeqie/geeqie-symlink
/scratch/mmokrejs/gentoo_rap/usr/lib/geeqie/geeqie-ufraw
/scratch/mmokrejs/gentoo_rap/usr/share
/scratch/mmokrejs/gentoo_rap/usr/share/applications
/scratch/mmokrejs/gentoo_rap/usr/share/applications/geeqie.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/doc
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3/COPYING.bz2
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3/ChangeLog.bz2
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3/README
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3/README.lirc.bz2
/scratch/mmokrejs/gentoo_rap/usr/share/doc/geeqie-1.3/TODO
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/geeqie-import-geeqie.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/geeqie-import-gqview.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/geeqie-ufraw-id.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/geeqie-ufraw-recursive.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/geeqie-ufraw.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/rotate.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/applications/symlink.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/geeqie/template.desktop
/scratch/mmokrejs/gentoo_rap/usr/share/locale
/scratch/mmokrejs/gentoo_rap/usr/share/locale/cs
/scratch/mmokrejs/gentoo_rap/usr/share/locale/cs/LC_MESSAGES
/scratch/mmokrejs/gentoo_rap/usr/share/locale/cs/LC_MESSAGES/geeqie.mo
/scratch/mmokrejs/gentoo_rap/usr/share/man
/scratch/mmokrejs/gentoo_rap/usr/share/man/man1
/scratch/mmokrejs/gentoo_rap/usr/share/man/man1/geeqie.1.bz2
/scratch/mmokrejs/gentoo_rap/usr/share/pixmaps
/scratch/mmokrejs/gentoo_rap/usr/share/pixmaps/geeqie.png
Comment 3 Bernard Cafarelli gentoo-dev 2016-11-09 12:46:59 UTC
Thanks for testing :) The fix is now in tree in geeqie-1.3-r1