From: "Andreas Färber" <afaerber@suse.de>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/8] qemu-common.h header cleanups
Date: Mon, 26 Nov 2012 20:44:46 +0100 [thread overview]
Message-ID: <50B3C6AE.9070909@suse.de> (raw)
In-Reply-To: <1353605251-22218-1-git-send-email-ehabkost@redhat.com>
Hi,
Am 22.11.2012 18:27, schrieb Eduardo Habkost:
> These are the remaining header cleanups that didn't enter the tree yet. They
> should help breaking some circular header dependencies involving qemu-common.h.
>
> Basically they were reordered by Igor to keep the tree bisectable, and some
> changes were made on the qga/channel-posix.c patch to eliminate the
> qemu-common.h dependency.
>
> Eduardo Habkost (8):
> user: Move *-user/qemu-types.h to main directory
> user: Rename qemu-types.h to qemu-user-types.h
> ui/vnc-palette.c: Include headers it needs
> qapi/qmp-registry.c: Include headers it needs
Patches 1-4 seem to match exactly what I had previously queued for 1.3.
Thanks, merged them into qom-cpu:
https://github.com/afaerber/qemu-cpu/commits/qom-cpu
I'll continue reviewing the changed parts later today.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
next prev parent reply other threads:[~2012-11-26 19:44 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-22 17:27 [Qemu-devel] [PATCH 0/8] qemu-common.h header cleanups Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 1/8] user: Move *-user/qemu-types.h to main directory Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 2/8] user: Rename qemu-types.h to qemu-user-types.h Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 3/8] ui/vnc-palette.c: Include headers it needs Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 4/8] qapi/qmp-registry.c: " Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 5/8] qga/channel-posix.c: " Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 6/8] qlist.h: do not include qemu-common.h Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 7/8] create qemu-types.h for struct typedefs Eduardo Habkost
2012-11-22 17:27 ` [Qemu-devel] [PATCH 8/8] sysemu.h: include qemu-types.h instead of qemu-common.h Eduardo Habkost
2012-11-26 19:44 ` Andreas Färber [this message]
2012-11-30 20:53 ` [Qemu-devel] [PATCH 0/8] qemu-common.h header cleanups Andreas Färber
2012-12-04 8:50 ` Paolo Bonzini
2012-12-04 13:52 ` Andreas Färber
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=50B3C6AE.9070909@suse.de \
--to=afaerber@suse.de \
--cc=ehabkost@redhat.com \
--cc=imammedo@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 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.