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

Re: [Xen-devel] [PATCH] Adds options for tabs-separators, and including the domain ID in the output.



>>> On 19.10.15 at 21:24, <swieser@xxxxxxxxxx> wrote:
> This change adds two options to xentop:
>         -T      adds a tabulator (\t) character after each field, to allow
>                 easier and more robust parsing. This affects batch mode only.
>         -I      includes a column with the domain ID in the output (both the
>                 graphical output, and the batch output)
> 
> This makes the output easier to parse for automated tools.
> If none of the options are passed, the output is unchanged, so none of them
> would break existing tools.
> 
> Signed-off-by: Stefan Wieser <swieser@xxxxxxxxxx>
> ---
>  tools/xenstat/xentop/xentop.c | 88 
> +++++++++++++++++++++++++++++++++----------
>  1 file changed, 68 insertions(+), 20 deletions(-)

This and the Cc list don't seem to match up: You want to Cc tools
maintainers, not "THE REST" ones (which only partly overlap).
Also please prefix your patch title, in this case e.g. by "xentop: ".

Jan


_______________________________________________
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®.