Bug 668638

Summary:	sys-apps/portage-2.3.51: QA Notice: .desktop files: triggered with EAPI-6, but not with EAPI-7
Product:	Portage Development	Reporter:	Andreas Sturmlechner <asturm>
Component:	Core	Assignee:	Portage team <dev-portage>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mgorny
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://github.com/gentoo/gentoo/pull/10110
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	671498

Description Andreas Sturmlechner gentoo-dev

2018-10-14 17:33:52 UTC

A given ebuild with EAPI-6 and missing mimeinfo cache update reports:

 * QA Notice: .desktop files with MimeType= were found installed
 * but desktop mimeinfo cache has not been updated:
 *   /usr/share/applications/ghostwriter.desktop
 * Please make sure to call xdg_desktop_database_update()
 * in pkg_postinst() and pkg_postrm() phases of appropriate pkgs.

However, the same ebuild, only with EAPI-7, does not trigger the QA warning.

Comment 1 Michał Górny archtester

2018-10-25 14:21:54 UTC

Yay, figured out the problem.  It was the trailing slash change after all:

  cd "${EROOT}" || die

(which started evaluating to 'cd' ;-))

Comment 2 Larry the Git Cow gentoo-dev

2018-10-25 15:17:13 UTC

The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/portage.git/commit/?id=53c4e1ac66f9ccfcb68b800cc1489a5da9923bb8

commit 53c4e1ac66f9ccfcb68b800cc1489a5da9923bb8
Author:     Michał Górny <mgorny@gentoo.org>
AuthorDate: 2018-10-25 14:22:54 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2018-10-25 15:16:44 +0000

    *-qa-check.d: fix entering EROOT in EAPI 7
    
    Fix running QA checks from EROOT in EAPI 7 if the value is empty.
    Previously, this wrongly caused 'cd' to be run with empty value implying
    home directory.
    
    Bug: https://bugs.gentoo.org/668638
    Signed-off-by: Michał Górny <mgorny@gentoo.org>
    Reviewed-by: Zac Medico <zmedico@gentoo.org>

 bin/misc-functions.sh                  | 2 +-
 bin/postinst-qa-check.d/50gnome2-utils | 2 +-
 bin/postinst-qa-check.d/50xdg-utils    | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Comment 3 Zac Medico gentoo-dev

2019-04-10 04:34:39 UTC

Fixed in portage-2.3.62.