[libvirt] [PATCH] docs: Add some changes to news.xml for this release

Martin Kletzander mkletzan at redhat.com
Mon Oct 2 08:54:50 UTC 2017


On Mon, Oct 02, 2017 at 10:07:01AM +0200, Andrea Bolognani wrote:
>On Fri, 2017-09-29 at 16:49 +0200, Martin Kletzander wrote:
>> +        <description>
>> +          While waiting for a write to disk to be finished, e.g. during save,
>> +          even simple operations like <code>virsh list</code> would be blocking
>> +          due to domain lock.  This is now resolved by unlocking the domain
>> +          in places where it is not needed.
>
>s/  / /g
>
>here and further below. I know you prefer double-space-after-period,
>but we are consistently using a single space in this file :)
>

Sure, it is a muscle memory at this point.  Anyway:

  :%s_\.  \([A-Z]\)_. \1_gc

showed one already existing line that I fixed as a part of this.

>> +        <description>
>> +          Previously the documentation used multiples of 1000, but now it is
>> +          fixe to use multiples of 1024.
>
>s/fixe/fixed/
>
>> +      <change>
>> +        <summary>
>> +          Fix message decoding which was caused very strange bug
>
>s/caused/causing/
>s/bug/bugs/
>

I believe it was a bug, so I added the indefinite article instead of this last change.

>
>With these issues fixed,
>
>  Reviewed-by: Andrea Bolognani <abologna at redhat.com>
>
>Thanks for once again picking up the slack :)
>

Sure, you could also cut me some slack with the double spaces since this is
fixed width style file ;)

Pushed now, thanks.

>--
>Andrea Bolognani / Red Hat / Virtualization
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171002/81cbf5d3/attachment-0001.sig>


More information about the libvir-list mailing list