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

2013-11-26 Thread Andrey Borzenkov
On Tue, Nov 26, 2013 at 11:53 AM, Beeblebrox wrote: > OK, I'll try it. But I need clarification: > > * Should I copy the repo and do a git rollback on the copy? Just setup separate tree to perform build. The actual state does not really matter; you can use current master, ignore doc build error f

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

2013-11-25 Thread Beeblebrox
OK, I'll try it. But I need clarification: * Should I copy the repo and do a git rollback on the copy? * Easier to copy only grub/docs to another folder, but how do I start the build then? The Makefile in grub/docs will fail just as when run from top-level. >> just copy grub.texi over >From my cu

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

2013-11-25 Thread Andrey Borzenkov
On Tue, Nov 26, 2013 at 11:31 AM, Beeblebrox wrote: > It's OK, I don't need it really. Unless you need me to test and get > back to you with results. Would be nice; as new release is planned in not so distant future, we should try to iron out problems. ___

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

2013-11-25 Thread Beeblebrox
> Did earlier versions grub.texi compiled? Yes, earlier versions of grub.texi did compile. > Setup build tree outside of git checkout and just copy > grub.texi over; it is self contained. It's OK, I don't need it really. Unless you need me to test and get back to you with results. ___

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

2013-11-25 Thread Andrey Borzenkov
On Tue, Nov 26, 2013 at 11:19 AM, Beeblebrox wrote: > Still have to manually clean out the "docs" references in Makefile however. > Did earlier versions grub.texi compiled? In this case you could try to bisect it. Setup build tree outside of git checkout and just copy grub.texi over; it is self

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

2013-11-25 Thread Beeblebrox
>> Or just update to current trunk, I committed this patch. Updated to trunk, it all works & gets compiled. Still have to manually clean out the "docs" references in Makefile however. Thanks for the swift work. ___ Grub-devel mailing list Grub-devel@gn

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

2013-11-25 Thread Andrey Borzenkov
В Mon, 25 Nov 2013 19:19:00 +0100 Vladimir 'φ-coder/phcoder' Serbinenko пишет: > On 25.11.2013 19:13, 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 easi

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

2013-11-25 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 25.11.2013 19:13, 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