Further changes for library compatibility

[builder.git] / source / config.cpp

diff --git a/source/config.cpp b/source/config.cpp
index d4f9384c5b1b05b7eb93becdcdf2a2e9a9995b18..87573e22360d5a765a12e5412616364b6f0ffa18 100644 (file)
--- a/source/config.cpp
+++ b/source/config.cpp
@@ -5,11 +5,9 @@ Copyright © 2006-2009  Mikko Rasa, Mikkosoft Productions
 Distributed under the LGPL
 */
 
-#include <cstdlib>
-#include <msp/core/except.h>
+#include <msp/core/maputils.h>
 #include <msp/fs/stat.h>
 #include <msp/fs/utils.h>
-#include <msp/io/except.h>
 #include <msp/io/file.h>
 #include <msp/io/print.h>
 #include <msp/time/utils.h>
@@ -32,11 +30,7 @@ void Config::add_option(const string &n, const string &v, const string &d)
 
 const Config::Option &Config::get_option(const string &name) const
 {
-       OptionMap::const_iterator i=options.find(name);
-       if(i==options.end())
-               throw Exception("Tried to access nonexistent option "+name);
-
-       return i->second;
+       return get_item(options, name);
 }
 
 bool Config::is_option(const string &name) const
@@ -53,12 +47,12 @@ void Config::select_last_profile()
                in.getline(profile);
                set_option("profile", profile);
        }
-       catch(const IO::FileNotFound &)
+       catch(const IO::file_not_found &)
        { }
 
-       freeze_mtime=true;
+       freeze_mtime = true;
        package.get_builder().apply_profile_template(*this, get_option("profile").value);
-       freeze_mtime=false;
+       freeze_mtime = false;
 
        load();
 }
@@ -73,82 +67,41 @@ void Config::select_profile(const string &profile)
                IO::print(out, "%s\n", profile);
        }
 
-       freeze_mtime=true;
+       freeze_mtime = true;
        package.get_builder().apply_profile_template(*this, profile);
-       freeze_mtime=false;
+       freeze_mtime = false;
 
        load();
 }
 
 bool Config::update(const StringMap &opts)
 {
-       bool changed=false;
+       bool changed = false;
        for(StringMap::const_iterator i=opts.begin(); i!=opts.end(); ++i)
        {
                if(set_option(i->first, i->second) && i->first!="profile")
-                       changed=true;
+                       changed = true;
        }
 
        if(changed && !freeze_mtime)
-               mtime=Time::now();
+               mtime = Time::now();
 
        return changed;
 }
 
 void Config::finish()
 {
-       for(unsigned n=0; n<20; ++n)
-       {
-               bool changed=false;
-               for(OptionMap::iterator i=options.begin(); i!=options.end(); ++i)
-               {
-                       Option &opt=i->second;
-                       unsigned dollar=0;
-                       while((dollar=opt.value.find('$', dollar))!=string::npos)
-                       {
-                               unsigned end;
-                               string var;
-                               if(opt.value[dollar+1]=='{')
-                               {
-                                       end=opt.value.find('}', dollar+2);
-                                       if(end==string::npos)
-                                               throw Exception("Unterminated variable reference");
-                                       var=opt.value.substr(dollar+2, end-dollar-2);
-                                       ++end;
-                               }
-                               else
-                               {
-                                       for(end=dollar+1; (isalnum(opt.value[end]) && opt.value[end]!='_'); ++end) ;
-                                       var=opt.value.substr(dollar+1, end-dollar-1);
-                               }
-
-                               string value;
-                               if(is_option(var))
-                                       value=get_option(var).value;
-                               else if(var=="arch")
-                                       value=package.get_builder().get_current_arch().get_name();
-                               else if(const char *ptr=getenv(var.c_str()))
-                                       value=ptr;
-
-                               opt.value.replace(dollar, end-dollar, value);
-
-                               dollar+=value.size();
-                               changed=true;
-                       }
-               }
-
-               if(!changed)
-                       break;
-       }
+       for(OptionMap::iterator i=options.begin(); i!=options.end(); ++i)
+               i->second.value = package.expand_string(i->second.value);
 }
 
 void Config::save() const
 {
-       FS::Path fn=package.get_source()/".options";
+       FS::Path fn = package.get_source()/".options";
 
-       OptionMap::const_iterator i=options.find("profile");
+       OptionMap::const_iterator i = options.find("profile");
        if(i!=options.end())
-               fn=package.get_source()/(".options."+i->second.value);
+               fn = package.get_source()/(".options."+i->second.value);
 
        IO::BufferedFile out(fn.str(), IO::M_WRITE);
 
@@ -158,14 +111,14 @@ void Config::save() const
 
 bool Config::set_option(const string &opt, const string &val)
 {
-       bool result=false;
+       bool result = false;
 
-       OptionMap::iterator i=options.find(opt);
+       OptionMap::iterator i = options.find(opt);
        if(i!=options.end())
        {
                if(i->second.value!=val)
-                       result=true;
-               i->second.value=val;
+                       result = true;
+               i->second.value = val;
        }
 
        return result;
@@ -173,19 +126,19 @@ bool Config::set_option(const string &opt, const string &val)
 
 void Config::load()
 {
-       FS::Path fn=package.get_source()/(".options."+get_option("profile").value);
+       FS::Path fn = package.get_source()/(".options."+get_option("profile").value);
 
        try
        {
                IO::BufferedFile in(fn.str());
 
-               mtime=Time::TimeStamp::from_unixtime(FS::stat(fn).st_mtime);
+               mtime = FS::stat(fn).get_modify_time();
 
                DataFile::Parser parser(in, fn.str());
                Loader loader(*this);
                loader.load(parser);
        }
-       catch(const IO::FileNotFound &)
+       catch(const IO::file_not_found &)
        { }
 }