All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v8 01/11] host-qemu: new package
Date: Fri, 5 Oct 2012 09:19:50 +0200	[thread overview]
Message-ID: <20121005091950.5833bed7@skate> (raw)
In-Reply-To: <CA+-urNSi+VTS93wP5cEFuLwZn7SCKawpDp6XR6OLqumofBXJyA@mail.gmail.com>

Frank,

On Thu, 4 Oct 2012 18:09:48 -0400, Frank Hunleth wrote:

> >> --- /dev/null
> >> +++ b/package/qemu/Config.in.host
> >> @@ -0,0 +1,15 @@
> >> +config BR2_PACKAGE_HOST_QEMU
> >> +     bool "host qemu"
> >> +     help
> >> +       QEMU is a generic and open source machine emulator and virtualizer.
> >> +
> >> +       In user mode emulation, QEMU runs single cross-compiled programs.
> >> +       Fast cross-compilation and cross-debugging are the main targets
> >> +       for user-mode emulation.
> >> +
> >> +       In system mode emulation, QEMU emulates a full computer system,
> >> +       including peripherals, and handles the filesystem image generated
> >> +       by Buildroot.
> >> +       (Buildroot is shipped with many configs/qemu_*_defconfig files).
> >> +
> >> +       http://qemu.org/
> >
> > I think you should not add a Kconfig option for the host-qemu package
> > for now. Until it supports the system emulation, it is kind of useless
> > to have it listed in the host tools. For the perl stuff, you only need
> > the user emulation, so let's add support for the user emulation only at
> > the moment.
> 
> Forgive me if I'm misunderstanding, but I think that listing qemu in
> Kconfig may be desirable to me even if only user emulation is
> supported. I currently run a set of regression tests as a post-build
> step. I use qemu user emulation mode for this. Right now, I use a
> system-installed version of qemu-arm but would prefer to use a
> buildroot version.

Sure. But for example the Kconfig help text was mentioning system mode
emulation, even though the current package didn't support it. My point
was: for the sake of building Perl, we don't need to have host-qemu in
Kconfig, so let's not do this, and keep the focus of the patch set.

Fran?ois has already done 8 versions of this patch set, and I am pretty
sure he wants to see it merged. And the best way to see a patch set
merged is to remain absolutely focused on the problem you're trying to
solve, and not to solve a gazillions of other unrelated problems at the
same time.

We will definitely welcome further patches improving the host-qemu
package.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  reply	other threads:[~2012-10-05  7:19 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-25 17:18 [Buildroot] [PATCH v8 01/11] host-qemu: new package Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 02/11] perl: " Francois Perrad
2012-10-04 20:40   ` Thomas Petazzoni
2012-10-06 13:29     ` François Perrad
2012-10-04 21:18   ` Thomas Petazzoni
2012-10-06 13:33     ` François Perrad
2012-10-06 17:18       ` Thomas Petazzoni
2012-10-06 19:37         ` François Perrad
2012-10-08 16:57           ` François Perrad
2012-10-06 12:12   ` Arnout Vandecappelle
2012-10-06 13:34     ` François Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 03/11] perl: add an option for miniperl Francois Perrad
2012-10-04 20:41   ` Thomas Petazzoni
2012-10-06 12:17     ` Arnout Vandecappelle
2012-10-06 13:38       ` François Perrad
2012-10-06 14:58         ` Arnout Vandecappelle
2012-10-06 12:15   ` Arnout Vandecappelle
2012-09-25 17:18 ` [Buildroot] [PATCH v8 04/11] perl: add DB_File Francois Perrad
2012-10-06 12:23   ` Arnout Vandecappelle
2012-10-06 13:40     ` François Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 05/11] gdbm: new package Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 06/11] perl: add GDBM_File Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 07/11] cpanminus: new package Francois Perrad
2012-10-04 20:46   ` Thomas Petazzoni
2012-10-06 13:43     ` François Perrad
2012-10-04 21:01   ` Thomas Petazzoni
2012-10-05  8:12     ` François Perrad
2012-10-06 13:44     ` François Perrad
2012-10-06 12:29   ` Arnout Vandecappelle
2012-09-25 17:18 ` [Buildroot] [PATCH v8 08/11] perl: add option "custom install" Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 09/11] perl: relax microperl dependencies Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 10/11] microperl: mark as DEPRECATED Francois Perrad
2012-09-25 17:18 ` [Buildroot] [PATCH v8 11/11] microperl: remove it Francois Perrad
2012-09-25 21:15 ` [Buildroot] [PATCH v8 01/11] host-qemu: new package Arnout Vandecappelle
2012-09-26  7:33   ` Thomas Petazzoni
2012-10-04 20:33 ` Thomas Petazzoni
2012-10-04 22:09   ` Frank Hunleth
2012-10-05  7:19     ` Thomas Petazzoni [this message]
2012-10-06 12:37       ` Arnout Vandecappelle
2012-10-06 13:51   ` François Perrad

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121005091950.5833bed7@skate \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.