Moved error / time from utils to core.

[sysdb.git] / src / daemon / config.c

diff --git a/src/daemon/config.c b/src/daemon/config.c
index aa5cb990b8612ff446c659c2cd9137d4a25df7ff..03bc2b857caad19d8151c46b3cb35e00db896ec0 100644 (file)
--- a/src/daemon/config.c
+++ b/src/daemon/config.c
@@ -27,7 +27,8 @@
 
 #include "sysdb.h"
 #include "core/plugin.h"
-#include "utils/time.h"
+#include "core/error.h"
+#include "core/time.h"
 
 #include "daemon/config.h"
 
@@ -55,15 +56,15 @@ config_get_interval(oconfig_item_t *ci, sdb_time_t *interval)
        assert(ci && interval);
 
        if (oconfig_get_number(ci, &interval_dbl)) {
-               fprintf(stderr, "config: Interval requires "
+               sdb_log(SDB_LOG_ERR, "config: Interval requires "
                                "a single numeric argument\n"
-                               "\tUsage: Interval SECONDS\n");
+                               "\tUsage: Interval SECONDS");
                return -1;
        }
 
        if (interval_dbl <= 0.0) {
-               fprintf(stderr, "config: Invalid interval: %f\n"
-                               "\tInterval may not be less than or equal to zero.\n",
+               sdb_log(SDB_LOG_ERR, "config: Invalid interval: %f\n"
+                               "\tInterval may not be less than or equal to zero.",
                                interval_dbl);
                return -1;
        }
@@ -101,13 +102,13 @@ daemon_load_backend(oconfig_item_t *ci)
        ctx.interval = default_interval;
 
        if (oconfig_get_string(ci, &name)) {
-               fprintf(stderr, "config: LoadBackend requires a single "
+               sdb_log(SDB_LOG_ERR, "config: LoadBackend requires a single "
                                "string argument\n"
-                               "\tUsage: LoadBackend BACKEND\n");
+                               "\tUsage: LoadBackend BACKEND");
                return -1;
        }
 
-       snprintf(plugin_name, sizeof(plugin_name), "backend/%s", name);
+       snprintf(plugin_name, sizeof(plugin_name), "backend::%s", name);
 
        for (i = 0; i < ci->children_num; ++i) {
                oconfig_item_t *child = ci->children + i;
@@ -117,9 +118,10 @@ daemon_load_backend(oconfig_item_t *ci)
                                return -1;
                }
                else {
-                       fprintf(stderr, "config: Unknown option '%s' inside 'LoadBackend' "
-                                       "-- see the documentation for details.\n", child->key);
-                       return -1;
+                       sdb_log(SDB_LOG_WARNING, "config: Unknown option '%s' "
+                                       "inside 'LoadBackend' -- see the documentation for "
+                                       "details.", child->key);
+                       continue;
                }
        }
 
@@ -137,9 +139,9 @@ daemon_configure_plugin(oconfig_item_t *ci)
        assert(ci);
 
        if (oconfig_get_string(ci, &name)) {
-               fprintf(stderr, "config: %s requires a single "
+               sdb_log(SDB_LOG_ERR, "config: %s requires a single "
                                "string argument\n"
-                               "\tUsage: LoadBackend BACKEND\n",
+                               "\tUsage: LoadBackend BACKEND",
                                ci->key);
                return -1;
        }
@@ -179,12 +181,14 @@ daemon_parse_config(const char *filename)
                }
 
                if (status) {
-                       fprintf(stderr, "config: Failed to parse option '%s'\n",
+                       sdb_error_set("config: Failed to parse option '%s'\n",
                                        child->key);
                        if (status > 0)
-                               fprintf(stderr, "\tUnknown option '%s' -- "
+                               sdb_error_append("\tUnknown option '%s' -- "
                                                "see the documentation for details\n",
                                                child->key);
+                       sdb_error_chomp();
+                       sdb_error_log(SDB_LOG_ERR);
                        retval = -1;
                }
        }