Deal with a second place where path could be overridden. Fixes: #274.…

… refs: #274
ARPA-SIMC · Oct 20, 2021 · 9547484 · 9547484
1 parent ad6967c
commit 9547484
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/arki/dataset/local.cc b/arki/dataset/local.cc
@@ -153,7 +153,8 @@ std::shared_ptr<core::cfg::Sections> Reader::read_configs(const std::string& pat
     auto sec = core::cfg::Section::parse(in);
     // Fill in missing bits
     sec->set("name", name);
-    sec->set("path", sys::abspath(path));
+    if (sec->value("type") != "remote")
+        sec->set("path", sys::abspath(path));
 
     // Return a Sections with only this section
     auto res = std::make_shared<core::cfg::Sections>();

diff --git a/arki/dataset/session-test.cc b/arki/dataset/session-test.cc
@@ -83,6 +83,11 @@ step = daily
     auto cfg = dataset::Session::read_config("testds");
     wassert(actual(cfg->value("name")) == "testds");
     wassert(actual(cfg->value("path")) == "http://example.org");
+
+    auto configs = dataset::Session::read_configs("testds");
+    cfg = configs->section("testds");
+    wassert(actual(cfg->value("name")) == "testds");
+    wassert(actual(cfg->value("path")) == "http://example.org");
 });
 
 }