Blob Blame History Raw
From b429af7c61bb1abe791ceb34fb6b6cf7382f832b Mon Sep 17 00:00:00 2001
From: Matej Habrnal <mhabrnal@redhat.com>
Date: Fri, 12 Feb 2016 14:25:05 +0100
Subject: [PATCH] lib: introduce a new function returning base user conf
 directory

Signed-off-by: Jakub Filak <jfilak@redhat.com>
Signed-off-by: Matej Habrnal <mhabrnal@redhat.com>
---
 src/include/internal_libreport.h |  3 +++
 src/lib/configuration_files.c    |  9 +++++++++
 src/lib/user_settings.c          | 12 +++++-------
 3 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/src/include/internal_libreport.h b/src/include/internal_libreport.h
index b36cbd9..38d75a1 100644
--- a/src/include/internal_libreport.h
+++ b/src/include/internal_libreport.h
@@ -738,6 +738,9 @@ bool load_conf_file(const char *pPath, map_string_t *settings, bool skipKeysWith
 #define load_plugin_conf_file libreport_load_plugin_conf_file
 bool load_plugin_conf_file(const char *name, map_string_t *settings, bool skipKeysWithoutValue);
 
+#define get_user_conf_base_dir libreport_get_user_conf_base_dir
+const char *get_user_conf_base_dir(void);
+
 #define load_conf_file_from_dirs libreport_load_conf_file_from_dirs
 bool load_conf_file_from_dirs(const char *base_name, const char *const *directories, map_string_t *settings, bool skipKeysWithoutValue);
 
diff --git a/src/lib/configuration_files.c b/src/lib/configuration_files.c
index 4975d62..b145b9f 100644
--- a/src/lib/configuration_files.c
+++ b/src/lib/configuration_files.c
@@ -318,6 +318,15 @@ finalize:
     return retval;
 }
 
+const char *get_user_conf_base_dir(void)
+{
+    static char *base_dir = NULL;
+    if (base_dir == NULL)
+        base_dir = concat_path_file(g_get_user_config_dir(), "abrt/settings/");
+
+    return base_dir;
+}
+
 bool load_conf_file_from_dirs(const char *base_name, const char *const *directories, map_string_t *settings, bool skipKeysWithoutValue)
 {
     if (NULL == directories || NULL == *directories)
diff --git a/src/lib/user_settings.c b/src/lib/user_settings.c
index 4cd87ff..cdc8482 100644
--- a/src/lib/user_settings.c
+++ b/src/lib/user_settings.c
@@ -18,21 +18,19 @@
 #include "internal_libreport.h"
 #include <augeas.h>
 
-#define BASE_DIR_FOR_USER_CONFIG_FILE "abrt/settings/"
-
 static map_string_t *user_settings;
 static char *conf_path;
 
 static char *get_user_config_file_path(const char *name, const char *suffix)
 {
-    char *s, *conf;
+    char *s = NULL;
+    char *conf;
 
     if (suffix != NULL)
-        s = xasprintf(BASE_DIR_FOR_USER_CONFIG_FILE"%s.%s", name, suffix);
-    else
-        s = xasprintf(BASE_DIR_FOR_USER_CONFIG_FILE"%s", name);
+        s = xasprintf("%s.%s", name, suffix);
+
+    conf = concat_path_file(get_user_conf_base_dir(), s != NULL ? s : name);
 
-    conf = concat_path_file(g_get_user_config_dir(), s);
     free(s);
     return conf;
 }
-- 
1.8.3.1