qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Riku Voipio <riku.voipio@iki.fi>
To: josch <josch@pyneo.org>
Cc: stappers@stappers.nl, Riku Voipio <riku.voipio@iki.fi>,
	qemu-devel@nongnu.org, Johannes Schauer <j.schauer@email.de>,
	632192@bugs.debian.org
Subject: Re: [Qemu-devel] [PATCH] add QEMU_LD_PREFIX environment variable
Date: Thu, 28 Jul 2011 11:41:09 +0300	[thread overview]
Message-ID: <20110728084109.GA20285@afflict.kos.to> (raw)
In-Reply-To: <1311400070-9051-1-git-send-email-josch@pyneo.org>

On Sat, Jul 23, 2011 at 07:47:49AM +0200, josch wrote:
> This could be avoided by setting the proposed environment variable 
> QEMU_LD_PREFIX to the just
> created debian rootfs. As mentioned earlier, the usage of the -L option
> is not possible in this scenario because qemu-user is only implicitly
> called by the binfmt mechanism.

What worries me here is that we are beginning to add a enviroment variable
for each and every command line option of qemu linux-user. I think it would
be better to have a wrapper binary to be registered as the binfmt runner.
Alternatively we should have a generic setup for mapping enviroment variables
to command line options. Now we get special per-option code every time someone 
needs to setup a command line option from binfmt.

Riku

  parent reply	other threads:[~2011-07-28  8:41 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-23  5:47 [Qemu-devel] [PATCH] add QEMU_LD_PREFIX environment variable josch
2011-07-23  5:47 ` josch
2011-07-28  8:41 ` Riku Voipio [this message]
2011-07-28 11:24   ` Johannes Schauer
2011-07-28 16:50     ` Geert Stappers
2011-07-28 17:28       ` Alexander Graf
2011-07-29 12:52   ` [Qemu-devel] Bug#632192: " Vagrant Cascadian
2011-07-29 15:21     ` Johannes Schauer
2011-07-30 13:58       ` Riku Voipio
2011-07-31 11:51         ` [Qemu-devel] [PATCH] introduce environment variables for all qemu-user options j.schauer
2011-07-31 12:12           ` Peter Maydell
2011-07-31 21:40             ` Johannes Schauer
2011-08-05 10:04               ` Peter Maydell
2011-08-06  6:54                 ` Johannes Schauer
2011-08-20 17:29                   ` Yann Dirson
  -- strict thread matches above, loose matches on Subject: below --
2011-07-06  9:15 [Qemu-devel] [PATCH] add QEMU_LD_PREFIX environment variable Johannes Schauer
2011-07-06  9:15 ` Johannes Schauer

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=20110728084109.GA20285@afflict.kos.to \
    --to=riku.voipio@iki.fi \
    --cc=632192@bugs.debian.org \
    --cc=j.schauer@email.de \
    --cc=josch@pyneo.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stappers@stappers.nl \
    /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 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).