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

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

 



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@xxxxxxxxxx>

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

Attachment: signature.asc
Description: Digital signature

--
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