All of lore.kernel.org
 help / color / mirror / Atom feed
* Should QEMU's configure script check for bzip2 ?
@ 2019-11-07 19:43 Thomas Huth
  2019-11-07 19:59 ` Wainer dos Santos Moschetta
  2019-11-08  9:48 ` Daniel P. Berrangé
  0 siblings, 2 replies; 4+ messages in thread
From: Thomas Huth @ 2019-11-07 19:43 UTC (permalink / raw)
  To: QEMU Developers; +Cc: Laszlo Ersek, Philippe Mathieu-Daudé


  Hi,

I just tried to compile QEMU on a freshly installed system. "configure" 
finished without problems, but during "make" I hit this error:

   BUNZIP2 pc-bios/edk2-i386-secure-code.fd.bz2
/bin/sh: bzip2: command not found
make: *** [Makefile:305: pc-bios/edk2-i386-secure-code.fd] Error 127
make: *** Deleting file 'pc-bios/edk2-i386-secure-code.fd'
make: *** Waiting for unfinished jobs....

Sure, it's easy to fix, but maybe "configure" should already check for 
the availablity of "bzip2", so that we then either skip the installation 
of the edk2 images if "bzip2" is not available, or bail out during 
"configure" already?

  Thomas



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

* Re: Should QEMU's configure script check for bzip2 ?
  2019-11-07 19:43 Should QEMU's configure script check for bzip2 ? Thomas Huth
@ 2019-11-07 19:59 ` Wainer dos Santos Moschetta
  2019-11-08  9:48 ` Daniel P. Berrangé
  1 sibling, 0 replies; 4+ messages in thread
From: Wainer dos Santos Moschetta @ 2019-11-07 19:59 UTC (permalink / raw)
  To: Thomas Huth, QEMU Developers; +Cc: Philippe Mathieu-Daudé, Laszlo Ersek


On 11/7/19 5:43 PM, Thomas Huth wrote:
>
>  Hi,
>
> I just tried to compile QEMU on a freshly installed system. 
> "configure" finished without problems, but during "make" I hit this 
> error:
>
>   BUNZIP2 pc-bios/edk2-i386-secure-code.fd.bz2
> /bin/sh: bzip2: command not found
> make: *** [Makefile:305: pc-bios/edk2-i386-secure-code.fd] Error 127
> make: *** Deleting file 'pc-bios/edk2-i386-secure-code.fd'
> make: *** Waiting for unfinished jobs....

This error happened with me a few times as well. And I always wondered 
why bzip2 isn't checked on configure.

>
>
> Sure, it's easy to fix, but maybe "configure" should already check for 
> the availablity of "bzip2", so that we then either skip the 
> installation of the edk2 images if "bzip2" is not available, or bail 
> out during "configure" already?

IMHO, it should be checked since bzip2 is a build dependency.

Thanks,

Wainer

>
>
>  Thomas
>
>



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

* Re: Should QEMU's configure script check for bzip2 ?
  2019-11-07 19:43 Should QEMU's configure script check for bzip2 ? Thomas Huth
  2019-11-07 19:59 ` Wainer dos Santos Moschetta
@ 2019-11-08  9:48 ` Daniel P. Berrangé
  2019-11-08 10:20   ` Philippe Mathieu-Daudé
  1 sibling, 1 reply; 4+ messages in thread
From: Daniel P. Berrangé @ 2019-11-08  9:48 UTC (permalink / raw)
  To: Thomas Huth; +Cc: Philippe Mathieu-Daudé, Laszlo Ersek, QEMU Developers

On Thu, Nov 07, 2019 at 08:43:27PM +0100, Thomas Huth wrote:
> 
>  Hi,
> 
> I just tried to compile QEMU on a freshly installed system. "configure"
> finished without problems, but during "make" I hit this error:
> 
>   BUNZIP2 pc-bios/edk2-i386-secure-code.fd.bz2
> /bin/sh: bzip2: command not found
> make: *** [Makefile:305: pc-bios/edk2-i386-secure-code.fd] Error 127
> make: *** Deleting file 'pc-bios/edk2-i386-secure-code.fd'
> make: *** Waiting for unfinished jobs....
> 
> Sure, it's easy to fix, but maybe "configure" should already check for the
> availablity of "bzip2", so that we then either skip the installation of the
> edk2 images if "bzip2" is not available, or bail out during "configure"
> already?

The general rule is that if we run a binary we should check for it upfront
so users immediately see any missing pre-requisites, rather than wasting
30 minutes waiting for QEMU to build & then fail.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



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

* Re: Should QEMU's configure script check for bzip2 ?
  2019-11-08  9:48 ` Daniel P. Berrangé
@ 2019-11-08 10:20   ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-11-08 10:20 UTC (permalink / raw)
  To: Daniel P. Berrangé, Thomas Huth; +Cc: Laszlo Ersek, QEMU Developers

On 11/8/19 10:48 AM, Daniel P. Berrangé wrote:
> On Thu, Nov 07, 2019 at 08:43:27PM +0100, Thomas Huth wrote:
>>
>>   Hi,
>>
>> I just tried to compile QEMU on a freshly installed system. "configure"
>> finished without problems, but during "make" I hit this error:
>>
>>    BUNZIP2 pc-bios/edk2-i386-secure-code.fd.bz2
>> /bin/sh: bzip2: command not found
>> make: *** [Makefile:305: pc-bios/edk2-i386-secure-code.fd] Error 127
>> make: *** Deleting file 'pc-bios/edk2-i386-secure-code.fd'
>> make: *** Waiting for unfinished jobs....
>>
>> Sure, it's easy to fix, but maybe "configure" should already check for the

You found a bug :)

>> availablity of "bzip2", so that we then either skip the installation of the
>> edk2 images if "bzip2" is not available, or bail out during "configure"
>> already?
> 
> The general rule is that if we run a binary we should check for it upfront
> so users immediately see any missing pre-requisites, rather than wasting
> 30 minutes waiting for QEMU to build & then fail.

Yes, I'll send a fix.

Regards,

Phil.


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

end of thread, other threads:[~2019-11-08 10:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-07 19:43 Should QEMU's configure script check for bzip2 ? Thomas Huth
2019-11-07 19:59 ` Wainer dos Santos Moschetta
2019-11-08  9:48 ` Daniel P. Berrangé
2019-11-08 10:20   ` Philippe Mathieu-Daudé

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.