From 4578a490ce5a5a6325b4ff2b8f44468464de2d94 Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Thu, 22 Sep 2005 15:43:22 +0000 Subject: * Parse multi-valued options. --- src/libstore/globals.cc | 47 +++++++++++++++++++++++++++++++++-------------- src/libstore/globals.hh | 2 +- 2 files changed, 34 insertions(+), 15 deletions(-) (limited to 'src/libstore') diff --git a/src/libstore/globals.cc b/src/libstore/globals.cc index 8cbae54e20ca..9a3ac6981384 100644 --- a/src/libstore/globals.cc +++ b/src/libstore/globals.cc @@ -1,6 +1,7 @@ #include "globals.hh" #include +#include string nixStore = "/UNINIT"; @@ -22,7 +23,15 @@ list buildUsers; static bool settingsRead = false; -static map settings; +static map settings; + + +template A & genericAt(T & container, unsigned int n) +{ + class T::iterator i = container.begin(); + advance(i, n); + return *i; +} static void readSettings() @@ -43,34 +52,44 @@ static void readSettings() if (hash != string::npos) line = string(line, 0, hash); - if (line.find_first_not_of(" ") == string::npos) continue; + Strings tokens = tokenizeString(line); + if (tokens.empty()) continue; - istringstream is(line); - string name, sep, value; - is >> name >> sep >> value; - if (sep != "=" || !is) + if (tokens.size() < 2 || genericAt(tokens, 1) != "=") throw Error(format("illegal configuration line `%1%' in `%2%'") % line % settingsFile); - - settings[name] = value; + + string name = genericAt(tokens, 0); + + Strings::iterator i = tokens.begin(); + advance(i, 2); + settings[name] = Strings(i, tokens.end()); }; settingsRead = true; } -string querySetting(const string & name, const string & def) +Strings querySetting(const string & name, const Strings & def) { if (!settingsRead) readSettings(); - map::iterator i = settings.find(name); + map::iterator i = settings.find(name); return i == settings.end() ? def : i->second; } bool queryBoolSetting(const string & name, bool def) { - string value = querySetting(name, def ? "true" : "false"); - if (value == "true") return true; - else if (value == "false") return false; + Strings defs; + if (def) defs.push_back("true"); else defs.push_back("false"); + + Strings value = querySetting(name, defs); + if (value.size() != 1) + throw Error(format("configuration option `%1%' should be either `true' or `false', not a list") + % name); + + string v = value.front(); + if (v == "true") return true; + else if (v == "false") return false; else throw Error(format("configuration option `%1%' should be either `true' or `false', not `%2%'") - % name % value); + % name % v); } diff --git a/src/libstore/globals.hh b/src/libstore/globals.hh index 327b1bbc3de9..8ba0a030041e 100644 --- a/src/libstore/globals.hh +++ b/src/libstore/globals.hh @@ -63,7 +63,7 @@ extern bool buildAllowRoot; extern list buildUsers; -string querySetting(const string & name, const string & def); +Strings querySetting(const string & name, const Strings & def); bool queryBoolSetting(const string & name, bool def); -- cgit 1.4.1