Re: [PATCH v2 03/12] qemu: Introduce pr-manager-helper capability

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/02/2018 02:58 AM, John Ferlan wrote:
> 
> 
> On 02/21/2018 01:11 PM, Michal Privoznik wrote:
>> The capability tracks if qemu has pr-manager-helper object. At
>> this time don't actually detect if qemu has the capability. Not
>> just yet. Only after the code is written the feature will be
>> enabled.
>>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>>  src/qemu/qemu_capabilities.c | 1 +
>>  src/qemu/qemu_capabilities.h | 1 +
>>  2 files changed, 2 insertions(+)
>>
> 
> Very strange to not see the replies or xml files adjusted here.
> 
> And why is patch 12 not merged in here?  Like it was in v1 where you got
> an ACK from Peter?

Because Peter also said that the patches should be ordered in such way
that after each patch nothing is broken. So I had two options:

a) rewrite all the patches from scratch, or
b) separate the capability detection into its own patch so effectively
there is no way to use the feature until the very last patch and thus
there's nothing to break. This was used fairly often historically when
we were introducing new features in multiple patches.

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux