qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH 00/20] Make --enable-foo a requirement
Date: Wed, 12 Aug 2009 18:20:21 +0200	[thread overview]
Message-ID: <cover.1250093910.git.quintela@redhat.com> (raw)

Hi

--enable-foo in current qemu just means to enable foo if it founds it, otherwise just
not enable it silently.  This patches changed everything to use the convention:

  * foo="no", feature will only be used if --enable-foo arg is given
  * foo="", feature will be searched for, and if found, will be used
  * foo="yes", this value vill only be set by --enable-foo flag.
                   feature will searched for, if not found, configure exits with error

It is important for distributions. Otherwise, if something is not installed, things
don't fail to compile, just are disabled.

Once there, add --enable-foo for all things that already have --disable-foo and make sense.
There are still a few bits that donot' have both --enable-foo/--disable-foo, but they are
the minimal things.

kvm: the extra error message is only given if you do a --enable-kvm, by default, if kvm
is not found, it just fails, as everything else.

Questions:
- sdl: anyone really uses sdl + static?  I am not sure that they compile correctly, and
  they are just complicated.
- Do we want to sort things on the file?  I.e. libraries detection toguether, features
  detection together, ....  Just new they are a completely mess, completely unsorted
  as far as I can see.

Later, Juan.

Juan Quintela (20):
  Make vnc configure options less verbose.
  Use the same structure for list of libs in curses and pthread
  Add feature configure help
  Add error message for feature not found
  Add brlapi to new feature convencion
  Add vde to new feature convencion
  Add vnc_tls to new feature convencion
  Add vnc_sasl to new feature convencion
  Add curses to new feature convencion
  Add curl to new feature convencion
  Add pthread to new feature convencion
  Add nptl to new feature convencion
  Add bluez to new feature convencion
  Add build_docs to new feature convention
  Rename build_docs to docs
  Add sdl to new feature convention
  Add fdt to new feature convention
  Add xen to new feature convention
  Add kvm to new feature convention
  Add sparse to new feature convention

 configure |  319 +++++++++++++++++++++++++++++++++++++++++++-----------------
 1 files changed, 228 insertions(+), 91 deletions(-)

             reply	other threads:[~2009-08-12 16:22 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12 16:20 Juan Quintela [this message]
2009-08-12 16:20 ` [Qemu-devel] [PATCH 01/20] Make vnc configure options less verbose Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 02/20] Use the same structure for list of libs in curses and pthread Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 03/20] Add feature configure help Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 04/20] Add error message for feature not found Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 05/20] Add brlapi to new feature convencion Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 06/20] Add vde " Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 07/20] Add vnc_tls " Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 08/20] Add vnc_sasl " Juan Quintela
2009-08-12 16:20 ` [Qemu-devel] [PATCH 09/20] Add curses " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 10/20] Add curl " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 11/20] Add pthread " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 12/20] Add nptl " Juan Quintela
2009-08-12 17:16   ` Nathan Froyd
2009-08-12 18:31     ` [Qemu-devel] " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 13/20] Add bluez " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 14/20] Add build_docs to new feature convention Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 15/20] Rename build_docs to docs Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 16/20] Add sdl to new feature convention Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 17/20] Add fdt " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 18/20] Add xen " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 19/20] Add kvm " Juan Quintela
2009-08-12 16:29 ` [Qemu-devel] [PATCH 20/20] Add sparse " Juan Quintela

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=cover.1250093910.git.quintela@redhat.com \
    --to=quintela@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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).