qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* iPXE: update submodule
@ 2019-12-02 11:19 Philippe Mathieu-Daudé
  2019-12-02 12:18 ` Gerd Hoffmann
  0 siblings, 1 reply; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-12-02 11:19 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: qemu-devel

Hi Gerd,

'make -C roms efirom' is failing on Fedora 30.

Can you update the iPXE submodule so we get these buildsys commits:

c742c576 [build] Move predefined all-drivers build shortcut to Makefile
a4f8c6e3 [build] Do not apply WORKAROUND_CFLAGS for host compiler
1dd56dbd [build] Workaround compilation error with gcc 9.1
412acd78 [build] Fix "'%s' directive argument is null" error

Thanks,

Phil.



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: iPXE: update submodule
  2019-12-02 11:19 iPXE: update submodule Philippe Mathieu-Daudé
@ 2019-12-02 12:18 ` Gerd Hoffmann
  2019-12-02 12:24   ` Thomas Huth
  2019-12-02 12:31   ` Philippe Mathieu-Daudé
  0 siblings, 2 replies; 4+ messages in thread
From: Gerd Hoffmann @ 2019-12-02 12:18 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé; +Cc: qemu-devel

On Mon, Dec 02, 2019 at 12:19:50PM +0100, Philippe Mathieu-Daudé wrote:
> Hi Gerd,
> 
> 'make -C roms efirom' is failing on Fedora 30.
> 
> Can you update the iPXE submodule so we get these buildsys commits:
> 
> c742c576 [build] Move predefined all-drivers build shortcut to Makefile
> a4f8c6e3 [build] Do not apply WORKAROUND_CFLAGS for host compiler
> 1dd56dbd [build] Workaround compilation error with gcc 9.1
> 412acd78 [build] Fix "'%s' directive argument is null" error

Hmm.  Sure, can do that, but the question is for 4.2 or 5.0.  Updating
ipxe that close to the release makes me nervous, but shipping a ipxe
version which doesn't build with recent compilers in the release tarball
isn't great either.

I'd tend to go the 5.0 route.
Comments?
Suggestions?

cheers,
  Gerd



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: iPXE: update submodule
  2019-12-02 12:18 ` Gerd Hoffmann
@ 2019-12-02 12:24   ` Thomas Huth
  2019-12-02 12:31   ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 4+ messages in thread
From: Thomas Huth @ 2019-12-02 12:24 UTC (permalink / raw)
  To: Gerd Hoffmann, Philippe Mathieu-Daudé; +Cc: Peter Maydell, qemu-devel

On 02/12/2019 13.18, Gerd Hoffmann wrote:
> On Mon, Dec 02, 2019 at 12:19:50PM +0100, Philippe Mathieu-Daudé wrote:
>> Hi Gerd,
>>
>> 'make -C roms efirom' is failing on Fedora 30.
>>
>> Can you update the iPXE submodule so we get these buildsys commits:
>>
>> c742c576 [build] Move predefined all-drivers build shortcut to Makefile
>> a4f8c6e3 [build] Do not apply WORKAROUND_CFLAGS for host compiler
>> 1dd56dbd [build] Workaround compilation error with gcc 9.1
>> 412acd78 [build] Fix "'%s' directive argument is null" error
> 
> Hmm.  Sure, can do that, but the question is for 4.2 or 5.0.  Updating
> ipxe that close to the release makes me nervous, but shipping a ipxe
> version which doesn't build with recent compilers in the release tarball
> isn't great either.
> 
> I'd tend to go the 5.0 route.
> Comments?
> Suggestions?

I think the average users don't rebuild the rom binaries on their own
from the release tarballs ... so IMHO this should rather go into 5.0.
Just my 0.02 € of course.

 Thomas



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: iPXE: update submodule
  2019-12-02 12:18 ` Gerd Hoffmann
  2019-12-02 12:24   ` Thomas Huth
@ 2019-12-02 12:31   ` Philippe Mathieu-Daudé
  1 sibling, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-12-02 12:31 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: qemu-devel

On 12/2/19 1:18 PM, Gerd Hoffmann wrote:
> On Mon, Dec 02, 2019 at 12:19:50PM +0100, Philippe Mathieu-Daudé wrote:
>> Hi Gerd,
>>
>> 'make -C roms efirom' is failing on Fedora 30.
>>
>> Can you update the iPXE submodule so we get these buildsys commits:
>>
>> c742c576 [build] Move predefined all-drivers build shortcut to Makefile
>> a4f8c6e3 [build] Do not apply WORKAROUND_CFLAGS for host compiler
>> 1dd56dbd [build] Workaround compilation error with gcc 9.1
>> 412acd78 [build] Fix "'%s' directive argument is null" error
> 
> Hmm.  Sure, can do that, but the question is for 4.2 or 5.0.  Updating
> ipxe that close to the release makes me nervous, but shipping a ipxe
> version which doesn't build with recent compilers in the release tarball
> isn't great either.
> 
> I'd tend to go the 5.0 route.

I think 5.0 is OK, because if distributions build it, the are probably 
already backporting these patches, so they can backport the QEMU patch 
instead.



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-12-02 12:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-12-02 11:19 iPXE: update submodule Philippe Mathieu-Daudé
2019-12-02 12:18 ` Gerd Hoffmann
2019-12-02 12:24   ` Thomas Huth
2019-12-02 12:31   ` Philippe Mathieu-Daudé

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).