Don't print the configuration header in help for empty configuration

[builder.git] / source / config.cpp

diff --git a/source/config.cpp b/source/config.cpp
index c6dcb70b25ee9a31f90fa0ffce557592ce36c850..75784e370d19525bef4e391ffd15d80ff873279a 100644 (file)
--- a/source/config.cpp
+++ b/source/config.cpp
@@ -24,48 +24,6 @@ void Config::add_option(const string &n, const string &v, const string &d)
        options.insert(OptionMap::value_type(n, opt));
 }
 
-const Config::Option &Config::get_option(const string &name) const
-{
-       return get_item(options, name);
-}
-
-bool Config::is_option(const string &name) const
-{
-       return options.count(name);
-}
-
-bool Config::update(const StringMap &opts)
-{
-       bool changed_now = false;
-       for(StringMap::const_iterator i=opts.begin(); i!=opts.end(); ++i)
-       {
-               if(set_option(i->first, i->second) && i->first!="profile")
-                       changed_now = true;
-       }
-
-       if(changed_now)
-       {
-               mtime = Time::now();
-               changed = true;
-       }
-
-       return changed_now;
-}
-
-void Config::save() const
-{
-       if(!changed)
-               return;
-
-       FS::Path fn = package.get_source()/".config";
-
-       package.get_builder().get_logger().log("files", format("Writing %s", fn));
-       IO::BufferedFile out(fn.str(), IO::M_WRITE);
-
-       for(OptionMap::const_iterator i=options.begin(); i!=options.end(); ++i)
-               IO::print(out, "option \"%s\" \"%s\";\n", i->second.name, i->second.value);
-}
-
 bool Config::set_option(const string &opt, const string &val)
 {
        bool result = false;
@@ -77,6 +35,7 @@ bool Config::set_option(const string &opt, const string &val)
                {
                        result = true;
                        changed = true;
+                       mtime = Time::now();
                }
                i->second.value = val;
        }
@@ -84,10 +43,19 @@ bool Config::set_option(const string &opt, const string &val)
        return result;
 }
 
-void Config::load()
+bool Config::is_option(const string &name) const
+{
+       return options.count(name);
+}
+
+const Config::Option &Config::get_option(const string &name) const
 {
-       FS::Path fn = package.get_source()/".config";
+       return get_item(options, name);
+}
 
+void Config::load()
+{
+       FS::Path fn = package.get_source_directory()/".config";
        FS::Stat stat = FS::stat(fn);
        if(stat)
        {
@@ -102,11 +70,25 @@ void Config::load()
        }
 }
 
+void Config::save() const
+{
+       if(!changed)
+               return;
+
+       FS::Path fn = package.get_source_directory()/".config";
+
+       package.get_builder().get_logger().log("files", format("Writing %s", fn));
+       IO::BufferedFile out(fn.str(), IO::M_WRITE);
+
+       for(OptionMap::const_iterator i=options.begin(); i!=options.end(); ++i)
+               IO::print(out, "option \"%s\" \"%s\";\n", i->second.name, i->second.value);
+}
+
 
 Config::Option::Option(const string &n, const string &v, const string &d):
        name(n),
-       defv(v),
-       descr(d),
+       default_value(v),
+       description(d),
        value(v)
 { }