Re: [PATCH] tools/xl: Add 'xen_version' to `xl info`

[George Dunlap <george.dunlap@eu.citrix.com>]

On 11/07/13 10:10, Andrew Cooper wrote:
> On 11/07/13 10:02, George Dunlap wrote:
>> On Wed, Jul 10, 2013 at 5:46 PM, Andrew Cooper
>> <andrew.cooper3@citrix.com> wrote:
>>> Getting the full Xen version in an easily scriptable way is awkward,
>>> especially if trying to piece together from xen_{major,minor,extra}.
>>>
>>> This reflects $(XEN_FULLVERSION) in the build system (but under a more
>>> sensible name, as $(XEN_VERSION) is just the major number).
>>>
>>> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>> I can see the point, but it seems a bit redundant to have the version
>> info printed twice.
>>
>> This makes me wonder if it might make sense to have options to print
>> just individual items; e.g.:
>>
>> xl info --xen_version
>>
>> or perhaps
>>
>> xi info --format="%{xen_version}"
>>
>> or something.
>>
>> -George
> Ah - that was part 2 which I am working on in my copious free time.
>
> At the moment, xl info takes no parameters, or an -h or -n.
>
> I was thinking it could be modifed to optionally take a positional
> paramater, such as
>
> xl info xen_version
>
> Even with this however, my original point still stands.  Getting the
> version number is awkward.

Is there actually a reason to print it separately at all?  Why not 
replace the individual lines with the unified line?

  -George