Bug 681346

Summary:	app-emulation/qemu-3.1.0-r1 USE=filecaps - pkg_postinst() checks for DESTDIR instead of system
Product:	Gentoo Linux	Reporter:	Garry Filakhtov <filakhtov>
Component:	Current packages	Assignee:	Matthias Maier <tamiko>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ao, redneb, virtualization
Priority:	Normal	Keywords:	PATCH
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://bugs.gentoo.org/show_bug.cgi?id=675110
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	qemu-3.1.0-r1 ebuild filecaps flag fix

Description Garry Filakhtov 2019-03-23 04:26:16 UTC

Created attachment 570336 [details, diff]
qemu-3.1.0-r1 ebuild filecaps flag fix

While https://bugs.gentoo.org/675110 was fixed, it broke filecaps flag. The problem is that check is done in temporary install directory, however because action is performed in post-install phase, it should actually perform this change on the actual root directory.

Without this change fcaps is just never executed, because "${D}/usr/libexec/qemu-bridge-helper" does not exist in the first place.

Comment 1 Matthias Maier gentoo-dev

2019-04-08 01:56:54 UTC

*** Bug 677152 has been marked as a duplicate of this bug. ***

Comment 2 Larry the Git Cow gentoo-dev

2019-04-08 02:52:23 UTC

The bug has been closed via the following commit(s):

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

commit 2fd0e188e866fd41fd2075c692ccf41684d9fabe
Author:     Matthias Maier <tamiko@gentoo.org>
AuthorDate: 2019-04-08 02:00:15 +0000
Commit:     Matthias Maier <tamiko@gentoo.org>
CommitDate: 2019-04-08 02:29:42 +0000

    app-emulation/qemu: Fix check in pkg_postinst
    
    Thanks to Garry Filakhtov for triaging and fixing this issue.
    
    Closes: https://bugs.gentoo.org/681346
    Package-Manager: Portage-2.3.62, Repoman-2.3.12
    Signed-off-by: Matthias Maier <tamiko@gentoo.org>

 app-emulation/qemu/{qemu-3.1.0-r3.ebuild => qemu-3.1.0-r4.ebuild} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)