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

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



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


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.