Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 699498 - media-gfx/iscan: dosym with path starting with ${ED}
Summary: media-gfx/iscan: dosym with path starting with ${ED}
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Matthew Schultz
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2019-11-07 00:57 UTC by Arfrever Frehtes Taifersar Arahesis
Modified: 2019-11-20 06:03 UTC (History)
1 user (show)

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 Arfrever Frehtes Taifersar Arahesis 2019-11-07 00:57:34 UTC
media-gfx/iscan/iscan-2.30.3.1.ebuild:  dosym "${ED}"/usr/bin/iscan "${plugindir}"/iscan
media-gfx/iscan/iscan-2.30.4.2.ebuild:  dosym "${ED%/}"/usr/bin/iscan "${gimpplugindir}"/iscan


Problems with above dosym command:

1. Initial ${D} or ${ED} is unnecessary in first argument (target of symbolic link), and PMS-compliant package managers will issue QA notice.
   PMS (https://projects.gentoo.org/pms/7/pms.html#x1-14800013.4.1):
   """
   13.4.1 Rewriting

   Any absolute symlink whose link starts with D must be rewritten with
   the leading D removed. The package manager should issue a notice when doing this. 
   """

2. Target of symbolic link could be made relative, instead of absolute.


Solution:

dosym $(realpath -s --relative-to="${gimpplugindir}" "${EPREFIX}/usr/bin/iscan") "${gimpplugindir}"/iscan
Comment 1 Matthew Schultz 2019-11-11 04:30:40 UTC
Why doesn't repoman emit a QA notice for this?
Comment 2 Arfrever Frehtes Taifersar Arahesis 2019-11-11 05:00:24 UTC
Install >=app-portage/repoman-2.3.18 and it will print warning.
Comment 3 Matthew Schultz 2019-11-11 05:31:04 UTC
Ok thanks.  That's a really new version.
Comment 4 Larry the Git Cow gentoo-dev 2019-11-20 06:03:36 UTC
The bug has been closed via the following commit(s):

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

commit 676396715b909dc72320cb5adddac3ea654fb76b
Author:     Matthew Schultz <mattsch@gmail.com>
AuthorDate: 2019-11-11 05:17:56 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2019-11-20 06:03:24 +0000

    media-gfx/iscan: Removed ${ED} for PMS-compliant package managers.
    
    Closes: https://bugs.gentoo.org/699498
    Package-Manager: Portage-2.3.76, Repoman-2.3.16
    Signed-off-by: Matthew Schultz <mattsch@gmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/13607
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 media-gfx/iscan/iscan-2.30.4.2.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)