larous25 / mupen64plus

Automatically exported from code.google.com/p/mupen64plus
0 stars 0 forks source link

mupen64plus dos not build with gcc-4.6 #425

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Describe your system:
 - Operating System (be specific): Gentoo Linux
 - Machine type (32-bit or 64-bit): intel 32 bit
 - Mupen64Plus version: 1.5
 - Plugins used: none

Describe the problem:

Does not build with gcc-4.6

Please provide any additional information below.

New gcc accept linker option only in the form -Wl,something so it does not 
accept -m 32 or -m elf_i386 on the gcc command line.

I patched pre.mk in this way to make it work

--- pre.mk.old  2011-04-14 20:39:21.000000000 +0200
+++ pre.mk      2011-04-14 20:39:57.000000000 +0200
@@ -319,7 +319,7 @@
   # tweak flags for 32-bit build on 64-bit system
   ifeq ($(ARCH), 64BITS_32)
     CFLAGS += -m32
-    LDFLAGS += -m32 -m elf_i386
+    LDFLAGS += -Wl,-m32 -Wl,-m,elf_i386
   endif
 endif
 ifeq ($(CPU_ENDIANNESS), BIG)

Original issue reported on code.google.com by atup...@gmail.com on 15 Apr 2011 at 11:53

GoogleCodeExporter commented 8 years ago

Original comment by richard...@gmail.com on 21 May 2011 at 2:48

GoogleCodeExporter commented 8 years ago
I had the unpleasant experience that your patch is incorrect. -m32 is not for 
the linker, but for the compiler. He has to provide the search paths for the 
linker - and thus the line has to be "LDFLAGS += -m32 -Wl,-m,elf_i386" it 
should be fixed in all repos of richard42. Can you please check it?

Original comment by sven@narfation.org on 9 Jul 2011 at 9:55

GoogleCodeExporter commented 8 years ago
What you say seems reasonable, well I hope it is. I changed the patch for 
building it, but it is now hard for me to retest, as my amd64 platform is not 
running.

Original comment by atup...@gmail.com on 13 Jul 2011 at 11:52

GoogleCodeExporter commented 8 years ago

Original comment by s...@narfation.org on 29 Aug 2011 at 2:52