qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: quintela@redhat.com
To: qemu-devel@nongnu.org
Cc: Juan Quintela <quintela@redhat.com>
Subject: [Qemu-devel] [PATCH 00/19]  More Build System cleanups (v2)
Date: Thu,  2 Jul 2009 16:46:24 +0200	[thread overview]
Message-ID: <cover.1246545655.git.quintela@redhat.com> (raw)

From: Juan Quintela <quintela@redhat.com>

Hi

This series rebase my changes on top of today git.  Also add more
cleanups.  Anthony, feel free to cherry pick any patches that you
like, if you don't want all of them.

- The bigger stuff here is move to only one qemu compilation rule for
  both bsd-user/darwin-user/linux-user/softmmu (the only thing that
  really changed was ARLIBS

- Start moving CFLAGS/LDFLAGS setup to ./configure.  Just
  removed OS_* and ARCH_*, planing how to move the other ones.

Comments?  For yesterday series, only comment was that we needed more of it :)

Later, Juan "compiling for qemu ifdef removal prize"

v1:

More cleanups, 1st two are two compile fixes for my previous series
(the XEN one already sent to the list.

Generate TARGET_BASE_ARCH and TARGET_CPU on configure, and then we
don't complicate Makefile.target having to regerate it.

Switch all objects that depends of one traget to
obj-$(TARGET_BASE_ARCH)-y variable, that way we would be able to share
Makefiles at some point.

Once there, fix kvm, xen and kqemu to not being duplicated for each
arch that support i.t


Juan Quintela (19):
  fix XEN Build
  fix sparc not solaris build
  Remove duplicated definition
  use block-nested-y for files inside block/
  use nwfpe-obj-y for consistence
  Remove unused Makefile variable
  already defined several lines before in block-obj-y
  configure already knows what TARGET_BASE_ARCH we need, no need to put
    the logic in Makefile.target
  TARGET_ARCH2 is already known at configure time and it is called
    target_cpu Remove re-construction in Makefile.target
  chang ifdefs by obj-
  configure xen in a single place
  configure kvm in a single place
  configure kqemu in a single place
  BASE_LDFLAGS is not used anywhere else
  TARGET_ARCH == TARGET_BASE_ARCH in m68k and arm
  Refactor a single rule to generate all qemu-* executables
  qemu binary don't exist anymore.  I guess not much people is
    compiling on alpha
  We can wrap OS_CFLAGS/OS_LDFLAGS in CFLAGS/LDFLAGS at configure time
  We can wrap ARCH_CFLAGS/ARCH_LDFLAGS in CFLAGS/LDFLAGS at configure
    time

 Makefile        |   26 +++---
 Makefile.hw     |    3 -
 Makefile.target |  290 ++++++++++++++++++++++---------------------------------
 configure       |  101 ++++++++-----------
 4 files changed, 170 insertions(+), 250 deletions(-)

             reply	other threads:[~2009-07-02 14:48 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-02 14:46 quintela [this message]
2009-07-02 14:46 ` [Qemu-devel] [PATCH 01/19] fix XEN Build quintela
2009-07-02 15:14   ` Anthony Liguori
2009-07-02 15:48     ` Avi Kivity
2009-07-02 17:12     ` [Qemu-devel] " Juan Quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 02/19] fix sparc not solaris build quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 03/19] Remove duplicated definition quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 04/19] use block-nested-y for files inside block/ quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 05/19] use nwfpe-obj-y for consistence quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 06/19] Remove unused Makefile variable quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 07/19] already defined several lines before in block-obj-y quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 08/19] configure already knows what TARGET_BASE_ARCH we need, no need to put the logic in Makefile.target quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 09/19] TARGET_ARCH2 is already known at configure time and it is called target_cpu Remove re-construction " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 10/19] chang ifdefs by obj- quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 11/19] configure xen in a single place quintela
2009-07-02 15:24   ` Christoph Egger
2009-07-02 15:48     ` Paul Brook
2009-07-02 17:06     ` [Qemu-devel] " Juan Quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 12/19] configure kvm " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 13/19] configure kqemu " quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 14/19] BASE_LDFLAGS is not used anywhere else quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 15/19] TARGET_ARCH == TARGET_BASE_ARCH in m68k and arm quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 16/19] Refactor a single rule to generate all qemu-* executables quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 17/19] qemu binary don't exist anymore. I guess not much people is compiling on alpha quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 18/19] We can wrap OS_CFLAGS/OS_LDFLAGS in CFLAGS/LDFLAGS at configure time quintela
2009-07-02 14:46 ` [Qemu-devel] [PATCH 19/19] We can wrap ARCH_CFLAGS/ARCH_LDFLAGS " 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.1246545655.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).