From 7428f809a76044fcbc98fd5f718e331ce183941d Mon Sep 17 00:00:00 2001
From: Martin Schwenke <martin@meltin.net>
Date: Wed, 19 Nov 2014 18:19:50 +1100
Subject: [PATCH] ctdb-tools: Add -X option for machine parsable output with
 separator '|'

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
---
 ctdb/doc/ctdb.1.xml | 14 ++++++++++++++
 ctdb/tools/ctdb.c   |  5 +++++
 2 files changed, 19 insertions(+)

diff --git a/ctdb/doc/ctdb.1.xml b/ctdb/doc/ctdb.1.xml
index b57e595c5c3..d4e94137b5e 100644
--- a/ctdb/doc/ctdb.1.xml
+++ b/ctdb/doc/ctdb.1.xml
@@ -152,6 +152,20 @@
       </listitem>
       </varlistentry>
 
+      <varlistentry><term>-X</term>
+      <listitem>
+	<para>
+	  Produce output in machine readable form for easier parsing
+	  by scripts. This uses a field delimiter of '|'.  Not all
+	  commands support this option.
+	</para>
+	<para>
+	  This is equivalent to "-x|" and avoids some shell quoting
+	  issues.
+	</para>
+      </listitem>
+      </varlistentry>
+
       <varlistentry><term>-t <parameter>TIMEOUT</parameter></term>
       <listitem>
 	<para>
diff --git a/ctdb/tools/ctdb.c b/ctdb/tools/ctdb.c
index 3a1985bf22d..8a4d3bdde61 100644
--- a/ctdb/tools/ctdb.c
+++ b/ctdb/tools/ctdb.c
@@ -6378,6 +6378,7 @@ int main(int argc, const char *argv[])
 {
 	struct ctdb_context *ctdb;
 	char *nodestring = NULL;
+	int machineparsable = 0;
 	struct poptOption popt_options[] = {
 		POPT_AUTOHELP
 		POPT_CTDB_CMDLINE
@@ -6385,6 +6386,7 @@ int main(int argc, const char *argv[])
 		{ "node",      'n', POPT_ARG_STRING, &nodestring, 0, "node", "integer|all" },
 		{ "machinereadable", 'Y', POPT_ARG_NONE, &options.machinereadable, 0, "enable machine readable output", NULL },
 		{ NULL, 'x', POPT_ARG_STRING, &options.machineseparator, 0, "specify separator for machine readable output", "char" },
+		{ NULL, 'X', POPT_ARG_NONE, &machineparsable, 0, "enable machine parsable output with separator |", NULL },
 		{ "verbose",    'v', POPT_ARG_NONE, &options.verbose, 0, "enable verbose output", NULL },
 		{ "maxruntime", 'T', POPT_ARG_INT, &options.maxruntime, 0, "die if runtime exceeds this limit (in seconds)", "integer" },
 		{ "print-emptyrecords", 0, POPT_ARG_NONE, &options.printemptyrecords, 0, "print the empty records when dumping databases (catdb, cattdb, dumpdbbackup)", NULL },
@@ -6442,6 +6444,9 @@ int main(int argc, const char *argv[])
 		}
 	}
 
+	if (machineparsable) {
+		options.machineseparator = "|";
+	}
 	if (options.machineseparator != NULL) {
 		if (strlen(options.machineseparator) != 1) {
 			printf("Invalid separator \"%s\" - "
-- 
2.11.4.GIT