Re: [PATCH] Explicitly check for linking format to use for efiemu64 module

2013-11-25 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 25.11.2013 18:42, Andrey Borzenkov wrote: > В Mon, 25 Nov 2013 05:22:58 +0100 > Vladimir 'φ-coder/phcoder' Serbinenko пишет: > >>> + CFLAGS="-m64 -nostdlib -O2 -mcmodel=large -mno-red-zone" >>> + LDFLAGS="-m64 -Wl,$format -nostdlib" >> You need -static as otherwise on Apple systems i

Re: [PATCH] Explicitly check for linking format to use for efiemu64 module

2013-11-25 Thread Andrey Borzenkov
В Mon, 25 Nov 2013 05:22:58 +0100 Vladimir 'φ-coder/phcoder' Serbinenko пишет: > > + CFLAGS="-m64 -nostdlib -O2 -mcmodel=large -mno-red-zone" > > + LDFLAGS="-m64 -Wl,$format -nostdlib" > You need -static as otherwise on Apple systems it will try to pull in > the dynamic linker which we

Re: [PATCH] Explicitly check for linking format to use for efiemu64 module

2013-11-24 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 24.11.2013 18:21, Andrey Borzenkov wrote: > Similar to check for target linking format, also check for efiemu64 > instead of hardcoding -melf_x86_64. This fixes compilation on *BSD > variants. We cannot easily reuse main target check because platforms > are different (main target is 32 bit and e

Re: [PATCH] Explicitly check for linking format to use for efiemu64 module

2013-11-24 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 24.11.2013 18:21, Andrey Borzenkov wrote: > +for format in -melf_x86_64 -melf_x86_64_fbsd -melf_x86_64_obsd > -melf_x86_64_haiku -mx86_64pe -arch,x86_64; do Remove x86_64pe. We skip it in first check as well since we have no converter for 64-bit windows binaries and the format is name i386p

[PATCH] Explicitly check for linking format to use for efiemu64 module

2013-11-24 Thread Andrey Borzenkov
Similar to check for target linking format, also check for efiemu64 instead of hardcoding -melf_x86_64. This fixes compilation on *BSD variants. We cannot easily reuse main target check because platforms are different (main target is 32 bit and efiemu64 - 64 bit). This commit adds EFIEMU64_LINK_FO