Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 640242 - x11-wm/lumina-1.4.0_p1: Aborting due to QA concerns
Summary: x11-wm/lumina-1.4.0_p1: Aborting due to QA concerns
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major
Assignee: Andrey Grozin
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-08 11:21 UTC by José Romildo Malaquias
Modified: 2018-05-27 10:34 UTC (History)
3 users (show)

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


Attachments
x11-wm:lumina-1.4.0_p1:20171207-200629.log.gz (x11-wm:lumina-1.4.0_p1:20171207-200629.log.gz,45.84 KB, application/gzip)
2017-12-08 11:26 UTC, José Romildo Malaquias
Details
Output of `emerge --info '=x11-wm/lumina-1.4.0_p1::gentoo'` (nada.txt,15.22 KB, text/plain)
2017-12-08 11:28 UTC, José Romildo Malaquias
Details
Output of `emerge -pqv '=x11-wm/lumina-1.4.0_p1::gentoo'` (nada.txt,444 bytes, text/plain)
2017-12-08 11:30 UTC, José Romildo Malaquias
Details
lumina-1.4.0_p1-r1.ebuild.patch (lumina-1.4.0_p1-r1.ebuild.patch,1.44 KB, patch)
2018-05-26 07:55 UTC, PhobosK
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description José Romildo Malaquias 2017-12-08 11:21:51 UTC
Emerging lumina-1.4.0_p1 on my ~amd64 gentoo system is failing with the error:

[...]
>>> Completed installing lumina-1.4.0_p1 into /tmp/portage/x11-wm/lumina-1.4.0_p1/image/

 * Final size of build directory: 114312 KiB (111.6 MiB)
 * Final size of installed tree:   18896 KiB ( 18.4 MiB)

 * QA Notice: files installed in ${D}/${D}:
 *   /
 *   /usr
 * ERROR: x11-wm/lumina-1.4.0_p1::gentoo failed:
 *   Aborting due to QA concerns: 2 files installed in /tmp/portage/x11-wm/lumina-1.4.0_p1/image/tmp/portage/x11-wm/lumina-1.4.0_p1/image/
 * 
 * Call stack:
 *   misc-functions.sh, line 666:  Called install_qa_check
 *   misc-functions.sh, line 217:  Called source 'install_symlink_html_docs'
 *          05double-D, line  15:  Called DD_check
 *          05double-D, line  11:  Called die
 * The specific snippet of code:
 *   		die "Aborting due to QA concerns: ${#files[@]} files installed in ${D%/}${D}"
Comment 1 José Romildo Malaquias 2017-12-08 11:26:01 UTC
Created attachment 508786 [details]
x11-wm:lumina-1.4.0_p1:20171207-200629.log.gz
Comment 2 José Romildo Malaquias 2017-12-08 11:28:39 UTC
Created attachment 508788 [details]
Output of `emerge --info '=x11-wm/lumina-1.4.0_p1::gentoo'`
Comment 3 José Romildo Malaquias 2017-12-08 11:30:27 UTC
Created attachment 508790 [details]
Output of `emerge -pqv '=x11-wm/lumina-1.4.0_p1::gentoo'`
Comment 4 younky.yang 2017-12-10 14:06:28 UTC
Same issue here. which is weird as I don't know which files are the concern.
Comment 5 Andrey Grozin gentoo-dev 2018-04-10 17:14:14 UTC
I cannot reproduce this behaviour. I've just successfully re-emerges lumina-1.4.0_p1. I use it as my main desktop environment on 2 computers for quite some time.
Comment 6 Perfect Gentleman 2018-04-23 11:23:26 UTC
confirm
Comment 7 PhobosK 2018-05-25 20:51:33 UTC
I confirm this error when with IUSE flag "desktop-utils" set.
Comment 8 PhobosK 2018-05-26 07:55:08 UTC
Created attachment 533354 [details, diff]
lumina-1.4.0_p1-r1.ebuild.patch

The current portage ebuild x11-wm/lumina-1.4.0_p1 has some problems that lead to this error....

I propose a new one (versioning can be altered to whatever is needed) that IMHO should replace all of the existing ones at the moment...

The problems fixed are:
1. Missing dependency on dev-qt/qtprintsupport:5 (see reported bug #652860)
2. Removed 1.3.0-OS-detect.patch because it is already in the 1.4.0 upstream OS-detect.pri file
3. Removed the DESTDIR variable in the configure phase. It's usage leads to putting the desktop-utils files (if the IUSE flag is used) into the image at compile phase which is totally unacceptable (sorry I do not have the time to look further in the QT pri and pro upstream files to fix it there)
4. Removed all the "tricks" like "mv ${D}/${D}/etc ${D}/etc || die" etc., that are needed because of the above problem 3. They are not needed anymore...
5. Fixed the removal of the locale files using ${ED%/} instead of ${D%/}

Thanks
Comment 9 Larry the Git Cow gentoo-dev 2018-05-27 10:16:11 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f6a8222042139764516f9e70bbcda27e3a32cb10

commit f6a8222042139764516f9e70bbcda27e3a32cb10
Author:     Andrey Grozin <grozin@gentoo.org>
AuthorDate: 2018-05-27 10:15:37 +0000
Commit:     Andrey Grozin <grozin@gentoo.org>
CommitDate: 2018-05-27 10:15:37 +0000

    x11-wm/lumina: ebuild cleanup; depend on dev-qt/qtprintsupport
    
    Suggested-by: PhobosK <phobosk@fastmail.fm>
    Closes: https://bugs.gentoo.org/640242
    Closes: https://bugs.gentoo.org/652860
    Package-Manager: Portage-2.3.19, Repoman-2.3.6

 x11-wm/lumina/lumina-1.4.0_p1-r1.ebuild | 92 +++++++++++++++++++++++++++++++++
 1 file changed, 92 insertions(+)
Comment 10 Andrey Grozin gentoo-dev 2018-05-27 10:34:14 UTC
(In reply to PhobosK from comment #8)
> 3. Removed the DESTDIR variable in the configure phase. It's usage leads to
> putting the desktop-utils files (if the IUSE flag is used) into the image at
> compile phase which is totally unacceptable (sorry I do not have the time to
> look further in the QT pri and pro upstream files to fix it there)
> 4. Removed all the "tricks" like "mv ${D}/${D}/etc ${D}/etc || die" etc.,
> that are needed because of the above problem 3. They are not needed
> anymore...
Thank you for this clean-up. *.pro files in lumina are definitely inconsistent. With DESTDIR in eqmake5, if I use INSTALL_ROOT="${D}", some group of files installs to wrong places; if I omit it, this group is installed correctly, but the remaining files are installed to wrong places. I had to insert a number of mv and rm, a terrible hack. Your version, without DESTDIR in eqmake5, is much cleaner.
However, I must say that this QA check gave a false positive. With these mv and rm lines in place, the old ebuild finally installed everything to correct places. So, I think, this particular QA check should be improved.