Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 896336 - games-emulation/gngeo-0.8-r2 - Missing licenses + potential licensing issues
Summary: games-emulation/gngeo-0.8-r2 - Missing licenses + potential licensing issues
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-24 18:46 UTC by orbea
Modified: 2023-02-25 17:56 UTC (History)
3 users (show)

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


Attachments
gngeo's licenses (COPYING,22.06 KB, text/plain)
2023-02-24 18:46 UTC, orbea
Details

Note You need to log in before you can comment on or make changes to this bug.
Description orbea 2023-02-24 18:46:29 UTC
Created attachment 854420 [details]
gngeo's licenses

gngeo lists only GPL-2 under the LICENSES variable, but this is only the main license for gngeo and there are several other custom non-free licenses included.

The COPYING from gngeo lists XMAME as well as several custom non-commercial licenses for cyclone, drz80, raze and starscream. However the mame code found at least in the mamez80 directory actually uses the old mame license which is not the same as XMAME in gentoo and mame code found in mame_layer, neoboot, neocrypt and perhaps elsewhere lack any license headers that would indicate what mame license they are using. Additionally starscream is only allowed to be distributed in unmodified form which perhaps gngeo-0.8-execstacks.patch even violates?

Looking at the configure.in it appears which of the above will actually be used depends on architecture. For example on amd64 I get mamez80 (old MAME license) and gen68k (GPL-2).

#setup default 68k core
if test $m68kcore = default ; then
        case $target_cpu in
        i?86) 
                        if test x$have_name=xtrue ; then
                                m68kcore=starscream
                        else
                                m68kcore=gen68k
                        fi              
                ;;
        arm) m68kcore=cyclone
                ;;
        *)       m68kcore=gen68k
                ;;
    esac
fi
#setup default z80 core
if test $z80core = default ; then
        case $target_cpu in
        i?86) 
                        if test x$have_name=xtrue ; then
                                z80core=raze
                        else
                                z80core=mamez80
                        fi              
                ;;
        arm) z80core=drz80
                ;;
        *)       z80core=mamez80
                ;;
    esac
fi
Comment 1 Ionen Wolkens gentoo-dev 2023-02-24 18:54:47 UTC
>is only allowed to be distributed in unmodified form which perhaps gngeo-0.8-execstacks.patch even violates?
fwiw things like this bit can be solved adding RESTRICT="bindist" given then we're not distributing a modified copy nor a build of it, it's just being patched on the user's system
Comment 2 orbea 2023-02-24 19:11:00 UTC
> fwiw things like this bit can be solved adding RESTRICT="bindist" given then we're not distributing a modified copy nor a build of it, it's just being patched on the user's system

In this case this would only be applicable to i686 systems given the configure.in case statement. Note that the have_name test is always true.