[PATCH] news: Document new Xen hypervisor features

Jim Fehlig jfehlig at suse.com
Wed Apr 22 16:43:34 UTC 2020


On 4/22/20 9:10 AM, Andrea Bolognani wrote:
> On Wed, 2020-04-22 at 08:48 -0600, Jim Fehlig wrote:
>> On 4/22/20 1:51 AM, Andrea Bolognani wrote:
>>> On Tue, 2020-04-21 at 16:06 -0600, Jim Fehlig wrote:
>>>> +        <summary>
>>>> +          Xen: add support for e820_host hypervisor feature
>>>
>>> s/Xen: add/xen: Add/
>>
>> There is only one 'xen:' in news, all other instances are 'Xen:'. I should stick
>> with the majority.
> 
> If you look at most (all?) other entries, they look like
> 
>    subsystem: Change
> 
> which is the same style we use for commit messages.

If you really want to split hairs, the subsytem is 'libxl'. But I don't think 
that is good for news since in general folks know of the 'Xen' project, but not 
'libxl'. However, now that the old xen driver is removed libxl could be renamed 
to xen :-).

> Xen seems to be the outlier when it comes to following this style,
> both in release notes and to some extent in commit messages. It would
> be great if it could fall in line with the rest of the code base.

Sorry for being out of line. I'll try to be more conscious of that in the future.

Regards,
Jim





More information about the libvir-list mailing list