[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define
From: |
Juan Quintela |
Subject: |
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define |
Date: |
Sat, 25 Jul 2009 20:57:57 +0200 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.3 (gnu/linux) |
malc <address@hidden> wrote:
> On Sat, 25 Jul 2009, Paolo Bonzini wrote:
>
>> On 07/25/2009 01:13 PM, malc wrote:
>> > On Fri, 24 Jul 2009, Juan Quintela wrote:
>> >
>> > > Signed-off-by: Juan Quintela<address@hidden>
>> > > ---
>> > > audio/audio.c | 25 ++++++++++++++++++++++++-
>> > > configure | 3 ---
>> > > 2 files changed, 24 insertions(+), 4 deletions(-)
>> >
>> > That's how it used to be before, i see no reason of going back to this
>> > style.
>>
>> Personally I prefer Juan's. It makes things more uniform; even though for
>> example I disagree on changing autoconfy HAVE_* defines to CONFIG_*, it
>> wouldn't be a problem at all to have CONFIG_* in qemu.
>
> For starters this is not a functional equivalent of the things currently
> in place, please read the commit message of: 0c58ac1c7
Thanks very much for the review. I understand now why it was done the
way it is, searching for a solution.
Will try to think a better way of being able to autogenerate
config-host.h and maintaining the functionality of AUDIO_DRIVERS.
Later, Juan.
- [Qemu-devel] [PATCH 41/46] test if xen is enabled only once, (continued)
- [Qemu-devel] [PATCH 41/46] test if xen is enabled only once, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 42/46] define ENOTSUP the same that the other errors, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 43/46] rename NEEDS_LIBSUNMATH to CONFIG_NEEDS_LIBSUNMATH, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 44/46] rename USE_NPTL to CONFIG_USE_NPTL, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 46/46] Generate config-host.h from config-host.mak, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, Juan Quintela, 2009/07/24
- Re: [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Juan Quintela, 2009/07/26
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, malc, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, malc, 2009/07/26
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26