[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic |
Date: |
Thu, 22 Aug 2013 11:50:43 -0500 |
User-agent: |
Notmuch/0.15.2+202~g0c4b8aa (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Laszlo Ersek <address@hidden> writes:
> On 08/21/13 19:06, Paolo Bonzini wrote:
>> Il 21/08/2013 19:07, Michael S. Tsirkin ha scritto:
>
>>> NACK
>>
>> You know that a single developer's NACK counts nothing (it can be you,
>> it can be me), don't you?
>
> going meta...
>
> What's this?
>
> All I know (... I think I know) about patch acceptance is that Anthony
> prefers to have at least one R-b. As far as I've seen this is not a hard
> requirement (for example, maintainers sometimes send unreviewed patches
> in a pull request, and on occasion they are merged).
I look very poorly on anyone nacking anything. I value constructive
feedback.
Nacking does not add any value to the conversation. I admire the fact
that we've been able to maintain a very high level of conversation over
the years on qemu-devel and throwing around nacks just lowers the
overall tone.
If you can't think of anything better to say than NACK, don't even
bother sending the email in the first place.
Regards,
Anthony Liguori
>
> No words have been spent on NAKs yet (... since my subscription, that
> is). Is this stuff formalized somewhere?
>
> Sorry for wasting time...
>
> Thanks,
> Laszlo
- Re: [Qemu-devel] [PATCH 2/3] pc: get rid of builtin pvpanic, (continued)
- [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Paolo Bonzini, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Michael S. Tsirkin, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Paolo Bonzini, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Michael S. Tsirkin, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Paolo Bonzini, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Michael S. Tsirkin, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Laszlo Ersek, 2013/08/22
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Paolo Bonzini, 2013/08/22
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Michael S. Tsirkin, 2013/08/25
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic,
Anthony Liguori <=
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Michael S. Tsirkin, 2013/08/25
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Andreas Färber, 2013/08/21
- Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic, Paolo Bonzini, 2013/08/21
Re: [Qemu-devel] [RFC PATCH v2 0/3] Start fixing the pvpanic mess, Daniel P. Berrange, 2013/08/21
Re: [Qemu-devel] [RFC PATCH v2 0/3] Start fixing the pvpanic mess, Daniel P. Berrange, 2013/08/21