jacquesg/p5-Git-Raw

-m32 option makes build fail on most 32 bit arches

Zugschlus opened this issue · 4 comments

Zugschlus commented

Hi,

in Debian, the compiler of most 32 bit arches (including mips, mipsel, m68k, hppa) does not have the -m32 option. The build this unconditionally fails on those arches.

Since Debian doesn't need that hack (we do things right on 64 bit arches), I have removed the option in the Debian package completely (and do hope that I didn't introduce other breakage by doing that).

Please consider being less generous with the -m32 option and only add it on arches where it is actually needed. The current method of checking for the ptrsize and then excluding arm hurts non-arm 32 bit arches.

Greetings
Marc

jacquesg commented

Pull requests welcome.

jacquesg commented

The patch is insufficient. A better mechanism is needed to detect a multilib gcc installation, and add -m32 for these scenarios.

jacquesg commented

Closing this seeing that there hasn't been any activity.