bin/xbps-query: add --json flag as alternative to --format

Author: Duncaen

bin/xbps-query/defs.h

@@ -55,7 +55,7 @@ int	ownedby(struct xbps_handle *, const char *, bool, bool);
 int	list_manual_pkgs(struct xbps_handle *, xbps_object_t, const char *, void *, bool *);
 int	list_orphans(struct xbps_handle *, const char *);
 int	list_pkgs_pkgdb(struct xbps_handle *);
-int	list_pkgdb(struct xbps_handle *, int (*filter)(xbps_object_t), const char *format);
+int	list_pkgdb(struct xbps_handle *, int (*filter)(xbps_object_t), const char *format, int json);
 
 int	repo_list(struct xbps_handle *);
 

bin/xbps-query/list.c

@@ -36,6 +36,7 @@
 
 #include "defs.h"
 #include "xbps.h"
+#include "xbps/json.h"
 
 struct length_max_cb {
 	const char *key;
@@ -135,6 +136,7 @@ list_pkgs_pkgdb(struct xbps_handle *xhp)
 
 struct list_pkgdb_cb {
 	struct xbps_fmt *fmt;
+	struct xbps_json_printer *json;
 	int (*filter)(xbps_object_t obj);
 };
 
@@ -153,23 +155,34 @@ list_pkgdb_cb(struct xbps_handle *xhp UNUSED, xbps_object_t obj,
 			return 0;
 	}
 
-	r = xbps_fmt_dictionary(ctx->fmt, obj, stdout);
-	if (r < 0)
-		return r;
-	return 0;
+	if (ctx->fmt) {
+		r = xbps_fmt_dictionary(ctx->fmt, obj, stdout);
+	} else if (ctx->json) {
+		r = xbps_json_print_xbps_object(ctx->json, obj);
+		fprintf(ctx->json->file, "\n");
+	} else {
+		r = -ENOTSUP;
+	}
+	return r;
 }
 
 int
-list_pkgdb(struct xbps_handle *xhp, int (*filter)(xbps_object_t), const char *format)
+list_pkgdb(struct xbps_handle *xhp, int (*filter)(xbps_object_t), const char *format, int json)
 {
 	struct list_pkgdb_cb ctx = {.filter = filter};
+	struct xbps_json_printer pr = {0};
 	int r;
-
-	ctx.fmt = xbps_fmt_parse(format);
-	if (!ctx.fmt) {
-		r = -errno;
-		xbps_error_printf("failed to parse format: %s\n", strerror(-r));
-		return r;
+	if (json > 0) {
+		pr.indent = (json-1) * 2;
+		pr.file = stdout;
+		ctx.json = &pr;
+	} else if (format) {
+		ctx.fmt = xbps_fmt_parse(format);
+		if (!ctx.fmt) {
+			r = -errno;
+			xbps_error_printf("failed to parse format: %s\n", strerror(-r));
+			return r;
+		}
 	}
 	r = xbps_pkgdb_foreach_cb(xhp, list_pkgdb_cb, &ctx);
 	xbps_fmt_free(ctx.fmt);

bin/xbps-query/main.c

@@ -33,6 +33,8 @@
 #include <xbps.h>
 
 #include "defs.h"
+#include "xbps.h"
+#include "xbps/json.h"
 
 static void __attribute__((noreturn))
 usage(bool fail)
@@ -46,6 +48,7 @@ usage(bool fail)
 	    " -F, --format <format>     Format for list output\n"
 	    " -h, --help                Show usage\n"
 	    " -i, --ignore-conf-repos   Ignore repositories defined in xbps.d\n"
+	    " -J, --json                Print output as json\n"
 	    " -M, --memory-sync         Remote repository data is fetched and stored\n"
 	    "                           in memory, ignoring on-disk repodata archives\n"
 	    " -p, --property PROP[,...] Show properties for PKGNAME\n"
@@ -100,13 +103,14 @@ filter_repolock(xbps_object_t obj)
 int
 main(int argc, char **argv)
 {
-	const char *shortopts = "C:c:dF:f:hHiLlMmOo:p:Rr:s:S:VvX:x:";
+	const char *shortopts = "C:c:dF:f:hHiJLlMmOo:p:Rr:s:S:VvX:x:";
 	const struct option longopts[] = {
 		{ "config", required_argument, NULL, 'C' },
 		{ "cachedir", required_argument, NULL, 'c' },
 		{ "debug", no_argument, NULL, 'd' },
 		{ "help", no_argument, NULL, 'h' },
 		{ "ignore-conf-repos", no_argument, NULL, 'i' },
+		{ "json", no_argument, NULL, 'J' },
 		{ "list-repos", no_argument, NULL, 'L' },
 		{ "list-pkgs", no_argument, NULL, 'l' },
 		{ "list-hold-pkgs", no_argument, NULL, 'H' },
@@ -137,6 +141,7 @@ main(int argc, char **argv)
 	bool list_pkgs, list_repos, orphans, own, list_repolock;
 	bool list_manual, list_hold, show_prop, show_files, show_deps, show_rdeps;
 	bool show, pkg_search, regex, repo_mode, opmode, fulldeptree;
+	int json = 0;
 
 	rootdir = cachedir = confdir = props = pkg = catfile = format = NULL;
 	flags = rv = c = 0;
@@ -165,6 +170,9 @@ main(int argc, char **argv)
 		case 'F':
 			format = optarg;
 			break;
+		case 'J':
+			json++;
+			break;
 		case 'H':
 			list_hold = opmode = true;
 			break;
@@ -286,20 +294,21 @@ main(int argc, char **argv)
 		rv = repo_list(&xh);
 
 	} else if (list_hold) {
-		rv = list_pkgdb(&xh, filter_hold, format ? format : "{pkgver}\n") < 0;
+		rv = list_pkgdb(&xh, filter_hold, format ? format : "{pkgver}\n", json) < 0;
 
 	} else if (list_repolock) {
-		rv = list_pkgdb(&xh, filter_repolock, format ? format : "{pkgver}\n") < 0;
+		rv = list_pkgdb(&xh, filter_repolock, format ? format : "{pkgver}\n", json) < 0;
 
 	} else if (list_manual) {
-		rv = list_pkgdb(&xh, filter_manual, format ? format : "{pkgver}\n") < 0;
+		rv = list_pkgdb(&xh, filter_manual, format ? format : "{pkgver}\n", json) < 0;
 
 	} else if (list_pkgs) {
 		/* list available pkgs */
-		if (format)
-			rv = list_pkgdb(&xh, NULL, format);
-		else
+		if (format || json > 0) {
+			rv = list_pkgdb(&xh, NULL, format, json);
+		} else {
 			rv = list_pkgs_pkgdb(&xh);
+		}
 
 	} else if (orphans) {
 		/* list pkg orphans */

bin/xbps-query/xbps-query.1

@@ -100,6 +100,8 @@ Ignore repositories defined in configuration files.
 Only repositories specified in the command line via
 .Ar --repository
 will be used.
+.It Fl J, Fl -json
+Print output as json.
 .It Fl M, Fl -memory-sync
 For remote repositories, the data is fetched and stored in memory for the current
 operation.