From: Jaeyun <jy1210.jung@samsung.com>
Date: Tue, 17 May 2022 09:30:01 +0000 (+0900)
Subject: [Conf] subplugin path from extra conf
X-Git-Tag: accepted/tizen/unified/20220526.143642~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=567b6dbd5191a6220fa23e28b494b6ad5f8524e3;p=platform%2Fupstream%2Fnnstreamer.git

[Conf] subplugin path from extra conf

Add subplugin path loaded from extra config file.

Signed-off-by: Jaeyun <jy1210.jung@samsung.com>
---

diff --git a/gst/nnstreamer/nnstreamer_conf.c b/gst/nnstreamer/nnstreamer_conf.c
index 9b31a20..8eeaf9c 100644
--- a/gst/nnstreamer/nnstreamer_conf.c
+++ b/gst/nnstreamer/nnstreamer_conf.c
@@ -71,7 +71,8 @@ typedef enum
   CONF_SOURCE_ENVVAR = 0,
   CONF_SOURCE_INI = 1,
   CONF_SOURCE_HARDCODE = 2,
-  CONF_SOURCE_END,
+  CONF_SOURCE_EXTRA_CONF = 3, /**< path from extra config file */
+  CONF_SOURCE_END
 } conf_sources;
 
 typedef struct
@@ -316,6 +317,20 @@ _fill_in_vstr (gchar *** fullpath_vstr, gchar *** name_vstr,
   g_slist_free (lstN);
 }
 
+/** @brief Private function to fill subplugin path */
+static void
+_fill_subplugin_path (confdata * conf, GKeyFile * key_file, conf_sources src)
+{
+  conf->conf[NNSCONF_PATH_FILTERS].path[src] =
+      g_key_file_get_string (key_file, "filter", "filters", NULL);
+  conf->conf[NNSCONF_PATH_DECODERS].path[src] =
+      g_key_file_get_string (key_file, "decoder", "decoders", NULL);
+  conf->conf[NNSCONF_PATH_CUSTOM_FILTERS].path[src] =
+      g_key_file_get_string (key_file, "filter", "customfilters", NULL);
+  conf->conf[NNSCONF_PATH_CONVERTERS].path[src] =
+      g_key_file_get_string (key_file, "converter", "converters", NULL);
+}
+
 /** @brief Public function defined in the header */
 gboolean
 nnsconf_loadconf (gboolean force_reload)
@@ -404,17 +419,30 @@ nnsconf_loadconf (gboolean force_reload)
 
       conf.extra_conffile =
           g_key_file_get_string (key_file, "common", "extra_config_path", NULL);
-      conf.conf[NNSCONF_PATH_FILTERS].path[CONF_SOURCE_INI] =
-          g_key_file_get_string (key_file, "filter", "filters", NULL);
-      conf.conf[NNSCONF_PATH_DECODERS].path[CONF_SOURCE_INI] =
-          g_key_file_get_string (key_file, "decoder", "decoders", NULL);
-      conf.conf[NNSCONF_PATH_CUSTOM_FILTERS].path[CONF_SOURCE_INI] =
-          g_key_file_get_string (key_file, "filter", "customfilters", NULL);
-      conf.conf[NNSCONF_PATH_CONVERTERS].path[CONF_SOURCE_INI] =
-          g_key_file_get_string (key_file, "converter", "converters", NULL);
+
+      _fill_subplugin_path (&conf, key_file, CONF_SOURCE_INI);
     }
 
     g_key_file_free (key_file);
+
+    /* load from extra config file */
+    if (conf.extra_conffile) {
+      if (g_file_test (conf.extra_conffile, G_FILE_TEST_IS_REGULAR)) {
+        key_file = g_key_file_new ();
+        g_assert (key_file != NULL); /** Internal lib error? out-of-memory? */
+
+        if (g_key_file_load_from_file (key_file, conf.extra_conffile,
+                G_KEY_FILE_NONE, NULL)) {
+          _fill_subplugin_path (&conf, key_file, CONF_SOURCE_EXTRA_CONF);
+        }
+
+        g_key_file_free (key_file);
+      } else {
+        ml_logw
+            ("Cannot find config file '%s'. You should set a valid path to load extra configuration.",
+            conf.extra_conffile);
+      }
+    }
   } else {
     /**
      * Failed to get the configuration.