]> git.notmuchmail.org Git - notmuch/commitdiff
CLI/config: restore "notmuch config get built_with.*"
authorDavid Bremner <david@tethera.net>
Sun, 18 Jul 2021 17:58:22 +0000 (14:58 -0300)
committerDavid Bremner <david@tethera.net>
Wed, 18 Aug 2021 00:09:56 +0000 (17:09 -0700)
We need to special case the config section "built_with" because it is
not (currently) handled by the library. This seems consist with the
other sub-sub-commands 'list' and 'set'.

notmuch-config.c
test/T030-config.sh

index d9390c4d81ea10ff7e82196a5d034025b4915250..e5b4db45fdf53dcb4254bec0b321e1dfcbcf719e 100644 (file)
@@ -538,11 +538,18 @@ notmuch_config_command_get (notmuch_database_t *notmuch, char *item)
 {
     notmuch_config_values_t *list;
 
-    for (list = notmuch_config_get_values_string (notmuch, item);
-        notmuch_config_values_valid (list);
-        notmuch_config_values_move_to_next (list)) {
-       const char *val = notmuch_config_values_get (list);
-       puts (val);
+    if (STRNCMP_LITERAL (item, BUILT_WITH_PREFIX) == 0) {
+       if (notmuch_built_with (item + strlen (BUILT_WITH_PREFIX)))
+           puts ("true");
+       else
+           puts ("false");
+    } else {
+       for (list = notmuch_config_get_values_string (notmuch, item);
+            notmuch_config_values_valid (list);
+            notmuch_config_values_move_to_next (list)) {
+           const char *val = notmuch_config_values_get (list);
+           puts (val);
+       }
     }
     return EXIT_SUCCESS;
 }
index bc9904b83fe72cf8ef3104d1821febaf4b5ad895..f889747137a2e8c8444e57d245782c4bbc5620e3 100755 (executable)
@@ -148,7 +148,6 @@ test_begin_subtest "set built_with.* yields error"
 test_expect_code 1 "notmuch config set built_with.compact false"
 
 test_begin_subtest "get built_with.{compact,field_processor} prints true"
-test_subtest_known_broken
 for key in compact field_processor; do
     notmuch config get built_with.${key}
 done > OUTPUT
@@ -159,7 +158,6 @@ EOF
 test_expect_equal_file EXPECTED OUTPUT
 
 test_begin_subtest "get built_with.nonexistent prints false"
-test_subtest_known_broken
 output=$(notmuch config get built_with.nonexistent)
 test_expect_equal "$output" "false"