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

Re: [libvirt] [PATCH] virsh: Clarify behavior of domain list filtering.



On 09/17/2012 10:40 AM, Peter Krempa wrote:
> On 09/17/12 16:28, Laine Stump wrote:
>> On 09/17/2012 05:02 AM, Peter Krempa wrote:
>>> Some combinations of filtering flags produce no result. This patch
>>> tries
>>> to clarify this.
>>>
>>> Also fix unintentional double space.
>>> ---
>>>   tools/virsh.pod | 11 ++++++-----
>>>   1 file changed, 6 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/tools/virsh.pod b/tools/virsh.pod
>>> index c6f205a..2efb277 100644
>>> --- a/tools/virsh.pod
>>> +++ b/tools/virsh.pod
>>> @@ -395,10 +395,11 @@ into s3 state.
>>>   Normally only active domains are listed. To list inactive domains
>>> specify
>>>   I<--inactive> or I<--all> to list both active and inactive domains.
>>>
>>> -To filter the list of domains present on the hypervisor you may
>>> specify one or
>>> -more of filtering flags supported by the B<list> command.  These
>>> flags are
>>> -grouped by function. Specifying one or more flags from a group
>>> enables the
>>> -filter group. Supported filtering flags and groups:
>>> +To further filter the list of domains you may specify one or more
>>> of filtering
>>
>>
>> While you're here:  s/more of filtering/more of the filtering/
>>
>>
>>> +flags supported by the B<list> command. These flags are grouped by
>>> function.
>>> +Specifying one or more flags from a group enables the filter group.
>>> Note that
>>> +some combinations of flags may yield no results. Supported
>>> filtering flags and
>>> +groups:
>>>
>>>   =over 4
>>>
>>
>> ACK with that one small addition.
>>
>
> Bah, I just pushed it :(. I'll fix that in another patch.

Not a big deal - it's just one word, and it was there already :-)


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