Hi, On Tue, 2016-06-21 at 08:06 +0000, Daniel Knezevic wrote: > Package: mame > Version: 0.173-6 > Severity: important > Tags: sid + patch > Justification: FTBFS > User: debian-m...@lists.debian.org > Usertags: mips-patch > > Hello, > Package mame FTBFS on mips64el with following error: > g++: error: unrecognized command line option '-m64' > > build log: > https://buildd.debian.org/status/fetch.php?pkg=mame&arch=mips64el&ver > =0.173-6&stamp=1464007306 > > -m64 flag is not supported on mips64el. > > I have created two patches. One patch is to remove -m64 flag for > mips64el. The other patch is for debian/rules, it adds -no-keep- > memory flag for mips64el to fix linker issues with memory > consumption. > > Patches are tested on mips64el, i386 and amd64. > With these patches package was build successfully.
Did you test the patches on mipsel as well? Your patch uses 'uname -m' to determine if it's building for mips64, but a lot (I think all?) of the mipsel buildds run 64-bit kernels and would match your check as well. Thanks, James
signature.asc
Description: This is a digitally signed message part
