Bug 106997 – check_license from eutils.eclass fails to find the license

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Vasudeva Martin 2005-09-23 07:09:52 UTC]

I'm using Portage 2.0.51.22-r2
The check_license function failed to find the license.
After changing something in the eutils file it works fine.
Here is the corrected part of the check_license() function:
<check_license()>
if [ -z "${lic}" ] ; then
        lic="${PORTDIR}/licenses/${LICENSE}"
    else
        if [ -e "${PORTDIR}/licenses/${lic}" ] ; then
            lic="${PORTDIR}/licenses/${lic}"
        elif [ -e "${PWD}/${lic}" ] ; then
            lic="${PWD}/${lic}"
        elif [ -e "${lic}" ] ; then
            lic="${lic}"
        fi
    fi
    [ ! -f "${lic}" ] && die "Could not find requested license ${lic}"
</check_license()>
In the original sourcecode there is instead of ${lic} ${src} in many places.
I don't know if all my replacement's are neccesary, but I know that
check_license can find the license with this version.

[Comment 1 Brian Harring (RETIRED) 2005-09-23 11:42:25 UTC]

Not a portage bug, goes to the eclass authors

[Comment 2 SpanKY 2005-09-23 13:19:45 UTC]

the code you posted is already in eutils.eclass

no real info here though as to what the bug is

[Comment 3 SpanKY 2005-09-23 13:21:00 UTC]

err, the die message is different (you correctly use $lic instead of $src), but
otherwise the code appears to be the same

[Comment 4 SpanKY 2005-09-23 13:22:19 UTC]

actually i see your point

in the future, post a diff, otherwise it's hard to figure out what's changed

[Comment 5 Chris Gianelloni (RETIRED) 2005-09-23 13:44:59 UTC]

Fixed in CVS... guess I missed one when I changed them...