From 759b099861a3065f01384d4d1121a7726771e5fe Mon Sep 17 00:00:00 2001 From: Jonathan Lebon Date: Wed, 24 Oct 2018 11:15:18 -0400 Subject: [PATCH] ostree/main: Wrap `Version` in `--version` in quotes Wrap the `Version` key in the YAML-compatible output of `ostree --version` with quotes so that it's parsed as a string. The issues with the previous approach in a nutshell: ``` In [5]: yaml.load("asdf: 2018.10") Out[5]: {'asdf': 2018.1} ``` It's treating the version number as a floating-point. Now, this is technically a backwards incompatible change, but given that the previous approach is inherently broken for our needs, I don't see a way around breaking it now. Closes: #1761 Approved by: cgwalters --- src/ostree/ot-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ostree/ot-main.c b/src/ostree/ot-main.c index 148c8371..a1449aef 100644 --- a/src/ostree/ot-main.c +++ b/src/ostree/ot-main.c @@ -354,7 +354,7 @@ ostree_option_context_parse (GOptionContext *context, * possible to parse */ g_auto(GStrv) features = g_strsplit (OSTREE_FEATURES, " ", -1); g_print ("%s:\n", PACKAGE_NAME); - g_print (" Version: %s\n", PACKAGE_VERSION); + g_print (" Version: '%s'\n", PACKAGE_VERSION); if (strlen (OSTREE_GITREV) > 0) g_print (" Git: %s\n", OSTREE_GITREV); #ifdef BUILDOPT_IS_DEVEL_BUILD