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

[Wei Liu <wei.liu2@citrix.com>]

On Sat, Aug 13, 2016 at 05:51:21PM +0000, Stefan Wieser 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.

Line too long.

And thanks for having compatibility in mind.

> @@ -1235,9 +1273,19 @@ int main(int argc, char **argv)
>  		case 't':
>  			show_tmem = 1;
>  			break;
> +		case 'T':
> +			use_tabs = 1;
> +			break;
> +		case 'I':
> +			show_domid = 1;
> +			break;
>  		}
>  	}
>  
> +	if (use_tabs && !batch) {
> +		fail("Cannot use tabs in interactive mode.\n");
> +	}
> +

Minor nit: no need to use {} here.

No need to resend, though. Should be easy to fix if I am to commit this
patch.

All in all I have no opinion for a few new options. I will wait a few
gays so that other people can express their opinions.

Wei.

>  	/* Get xenstat handle */
>  	xhandle = xenstat_init();
>  	if (xhandle == NULL)
> -- 
> 1.9.1
> 

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