Reorder class members

[builder.git] / source / config.cpp

diff --git a/source/config.cpp b/source/config.cpp
index 0e8e6c2c742c29ea52ee887aed7b04f5472b0427..d4f9384c5b1b05b7eb93becdcdf2a2e9a9995b18 100644 (file)
--- a/source/config.cpp
+++ b/source/config.cpp
@@ -1,13 +1,17 @@
 /* $Id$
 
 This file is part of builder
-Copyright © 2006-2007 Mikko Rasa, Mikkosoft Productions
+Copyright © 2006-2009  Mikko Rasa, Mikkosoft Productions
 Distributed under the LGPL
 */
 
-#include <fstream>
-#include <msp/core/error.h>
-#include <msp/path/utils.h>
+#include <cstdlib>
+#include <msp/core/except.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>
 #include "builder.h"
 #include "config.h"
@@ -21,22 +25,11 @@ Config::Config(SourcePackage &p):
        freeze_mtime(false)
 { }
 
-/**
-Adds a configuration option.
-
-@param   n  Option name
-@param   v  Default value
-@param   d  Description
-*/
 void Config::add_option(const string &n, const string &v, const string &d)
 {
        options.insert(OptionMap::value_type(n, Option(n, v, d)));
 }
 
-/**
-Gets the given option from the configuration.  If the option doesn't exist,
-an Exception is thrown.
-*/
 const Config::Option &Config::get_option(const string &name) const
 {
        OptionMap::const_iterator i=options.find(name);
@@ -46,27 +39,22 @@ const Config::Option &Config::get_option(const string &name) const
        return i->second;
 }
 
-/**
-Checks whether an option with the given name exists.
-*/
 bool Config::is_option(const string &name) const
 {
        return options.count(name);
 }
 
-/**
-Selects the last profile used.  If the profile cache file is not present, the
-default profile is assumed.
-*/
 void Config::select_last_profile()
 {
-       ifstream in((package.get_source()/".profile.cache").str().c_str());
-       if(in)
+       try
        {
+               IO::BufferedFile in((package.get_source()/".profile").str());
                string profile;
-               getline(in, profile);
+               in.getline(profile);
                set_option("profile", profile);
        }
+       catch(const IO::FileNotFound &)
+       { }
 
        freeze_mtime=true;
        package.get_builder().apply_profile_template(*this, get_option("profile").value);
@@ -75,19 +63,14 @@ void Config::select_last_profile()
        load();
 }
 
-/**
-Selects the given profile.  The profile cache file is updated as well, unless
---dry-run was given.
-*/
 void Config::select_profile(const string &profile)
 {
        set_option("profile", profile);
 
        if(!package.get_builder().get_dry_run())
        {
-               ofstream out((package.get_source()/".profile.cache").str().c_str());
-               if(out)
-                       out<<profile<<'\n';
+               IO::BufferedFile out((package.get_source()/".profile").str(), IO::M_WRITE);
+               IO::print(out, "%s\n", profile);
        }
 
        freeze_mtime=true;
@@ -97,15 +80,6 @@ void Config::select_profile(const string &profile)
        load();
 }
 
-/**
-Processes options from the given raw option map.  Nonexistent options are
-ignored.  If any options were changed, the mtime of the configuration is updated
-to the current time.
-
-@param   opts  A map to process options from
-
-@return  Whether any option values were changed
-*/
 bool Config::update(const StringMap &opts)
 {
        bool changed=false;
@@ -121,9 +95,6 @@ bool Config::update(const StringMap &opts)
        return changed;
 }
 
-/**
-Expands any variable references in options.
-*/
 void Config::finish()
 {
        for(unsigned n=0; n<20; ++n)
@@ -147,15 +118,17 @@ void Config::finish()
                                }
                                else
                                {
-                                       for(end=dollar+1; (isalnum(opt.value[end]) && opt.value[end]!='_'); ++end);
+                                       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
-                                       value=getenv(var.c_str());
+                               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);
 
@@ -171,17 +144,16 @@ void Config::finish()
 
 void Config::save() const
 {
-       Path::Path fn=package.get_source()/".options.cache";
+       FS::Path fn=package.get_source()/".options";
 
        OptionMap::const_iterator i=options.find("profile");
        if(i!=options.end())
-               fn=package.get_source()/(".options."+i->second.value+".cache");
+               fn=package.get_source()/(".options."+i->second.value);
 
-       ofstream out(fn.str().c_str());
-       if(!out) return;
+       IO::BufferedFile out(fn.str(), IO::M_WRITE);
 
        for(i=options.begin(); i!=options.end(); ++i)
-               out<<"option \""<<i->second.name<<"\" \""<<i->second.value<<"\";\n";
+               IO::print(out, "option \"%s\" \"%s\";\n", i->second.name, i->second.value);
 }
 
 bool Config::set_option(const string &opt, const string &val)
@@ -201,22 +173,20 @@ bool Config::set_option(const string &opt, const string &val)
 
 void Config::load()
 {
-       Path::Path fn=package.get_source()/".options.cache";
+       FS::Path fn=package.get_source()/(".options."+get_option("profile").value);
 
-       OptionMap::iterator i=options.find("profile");
-       if(i!=options.end())
-               fn=package.get_source()/(".options."+i->second.value+".cache");
-
-       ifstream in(fn.str().c_str());
-       if(!in) return;
+       try
+       {
+               IO::BufferedFile in(fn.str());
 
-       struct stat st;
-       Path::stat(fn, st);
-       mtime=Time::TimeStamp::from_unixtime(st.st_mtime);
+               mtime=Time::TimeStamp::from_unixtime(FS::stat(fn).st_mtime);
 
-       DataFile::Parser parser(in, fn.str());
-       Loader loader(*this);
-       loader.load(parser);
+               DataFile::Parser parser(in, fn.str());
+               Loader loader(*this);
+               loader.load(parser);
+       }
+       catch(const IO::FileNotFound &)
+       { }
 }