X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fsourcepackage.cpp;h=06ab5fe7441953a9652b9ae40b2c2df208755de9;hb=30aa066f674421268abc6c79bf828e005ac50031;hp=029d5240faecb84541ce70a54421dfbd9c95a27d;hpb=398983749391592988daca0c9b62b3d38f86e4e6;p=builder.git

diff --git a/source/sourcepackage.cpp b/source/sourcepackage.cpp
index 029d524..06ab5fe 100644
--- a/source/sourcepackage.cpp
+++ b/source/sourcepackage.cpp
@@ -27,6 +27,8 @@ SourcePackage::SourcePackage(Builder &b, const string &n, const FS::Path &s):
 	config(*this),
 	deps_cache(*this)
 {
+	config.load();
+
 	components.push_back(Component(*this, Component::TARBALL, "@src"));
 }
 
@@ -61,96 +63,6 @@ FS::Path SourcePackage::get_out_dir() const
 		return source/arch.get_name()/detail;
 }
 
-string SourcePackage::expand_string(const string &str) const
-{
-	string result = str;
-	string::size_type dollar = 0;
-	unsigned n = 0;
-	while((dollar = result.find('$'))!=string::npos)
-	{
-		if(n>1000)
-			throw bad_expansion("nested too deep");
-
-		string::size_type end;
-		string var;
-		if(dollar+1<result.size() && result[dollar+1]=='{')
-		{
-			end = result.find('}', dollar+2);
-			if(end==string::npos)
-				throw bad_expansion("unterminated variable reference");
-			var = result.substr(dollar+2, end-dollar-2);
-			++end;
-		}
-		else
-		{
-			for(end=dollar+1; (isalnum(result[end]) || result[end]=='_'); ++end) ;
-			var = result.substr(dollar+1, end-dollar-1);
-		}
-
-		string value;
-		if(config.is_option(var))
-			value = config.get_option(var).value;
-		else if(var=="arch")
-			value = builder.get_current_arch().get_name();
-		else if(var=="system")
-			value = builder.get_current_arch().get_system();
-		else if(const char *ptr = getenv(var.c_str()))
-			value = ptr;
-
-		result.replace(dollar, end-dollar, value);
-
-		++n;
-	}
-
-	return result;
-}
-
-void SourcePackage::do_configure(const StringMap &opts, unsigned flag)
-{
-	init_config();
-
-	config.load();
-
-	if(flag && config.update(opts))
-		builder.get_logger().log("configure", format("Configuration of %s changed", name));
-
-	config.finish();
-
-	for(ConditionList::iterator i=conditions.begin(); i!=conditions.end(); ++i)
-		if(i->eval())
-		{
-			const StringList &reqs = i->get_requires();
-			for(StringList::const_iterator j=reqs.begin(); j!=reqs.end(); ++j)
-				if(Package *pkg = builder.get_package_manager().find_package(*j))
-					requires.push_back(pkg);
-		}
-
-	for(PackageList::iterator i=requires.begin(); i!=requires.end(); ++i)
-	{
-		BinaryPackage *bpkg = dynamic_cast<BinaryPackage *>(*i);
-		if(bpkg && bpkg->get_need_path())
-			bpkg->set_path(config.get_option(bpkg->get_name()+"_path").value);
-	}
-
-	deps_cache.load();
-
-	for(ComponentList::iterator i=components.begin(); i!=components.end(); ++i)
-		i->configure(opts, flag);
-}
-
-void SourcePackage::init_config()
-{
-	for(FeatureList::iterator i=features.begin(); i!=features.end(); ++i)
-		config.add_option("with_"+i->name, i->def_value, i->descr);
-
-	for(PackageList::const_iterator i=requires.begin(); i!=requires.end(); ++i)
-	{
-		BinaryPackage *bpkg = dynamic_cast<BinaryPackage *>(*i);
-		if(bpkg && bpkg->get_need_path())
-			config.add_option(bpkg->get_name()+"_path", "/usr", "Path for "+bpkg->get_name());
-	}
-}
-
 void SourcePackage::create_build_info()
 {
 	if(build_type)
@@ -177,12 +89,9 @@ void SourcePackage::create_build_info()
 		if(lexical_cast<bool>(config.get_option("with_"+i->name).value))
 			build_info.defines["WITH_"+toupper(i->name)] = "1";
 
-	for(ConditionList::iterator i=conditions.begin(); i!=conditions.end(); ++i)
-		if(i->eval())
-			build_info.update_from(i->get_build_info());
-
 	for(list<Component>::iterator i=components.begin(); i!=components.end(); ++i)
 	{
+		i->prepare();
 		i->create_build_info();
 		if(i->get_type()==Component::LIBRARY)
 			export_binfo.libs.push_back(i->get_name());
@@ -191,6 +100,8 @@ void SourcePackage::create_build_info()
 
 void SourcePackage::create_targets()
 {
+	deps_cache.load();
+
 	bool pc_needed = false;
 	for(ComponentList::const_iterator i=components.begin(); i!=components.end(); ++i)
 	{
@@ -216,11 +127,25 @@ void SourcePackage::save_caches()
 SourcePackage::Loader::Loader(SourcePackage &p):
 	DataFile::DerivedObjectLoader<SourcePackage, Package>(p)
 {
+	init(0);
+}
+
+SourcePackage::Loader::Loader(SourcePackage &p, const Config::InputOptions &o):
+	DataFile::DerivedObjectLoader<SourcePackage, Package>(p)
+{
+	init(&o);
+}
+
+void SourcePackage::Loader::init(const Config::InputOptions *o)
+{
+	options = o;
 	add("version",     &SourcePackage::version);
 	add("description", &SourcePackage::description);
 	add("build_info",  &Loader::build_info);
 	add("feature",     &Loader::feature);
 	add("if",          &Loader::condition);
+	add("if_arch",     &Loader::if_arch);
+	add("if_feat",     &Loader::if_feature);
 	add("program",     &Loader::component<Component::PROGRAM>);
 	add("library",     &Loader::component<Component::LIBRARY>);
 	add("module",      &Loader::component<Component::MODULE>);
@@ -258,13 +183,22 @@ void SourcePackage::Loader::feature(const string &n, const string &d)
 	feat.def_value = "no";
 	load_sub(feat);
 	obj.features.push_back(feat);
+	string config_key = "with_"+feat.name;
+	obj.config.add_option(config_key, feat.def_value, feat.descr);
+	if(options)
+	{
+		Config::InputOptions::const_iterator i = options->find(config_key);
+		if(i!=options->end())
+			obj.config.set_option(config_key, i->second);
+	}
 }
 
 void SourcePackage::Loader::condition(const string &c)
 {
+	IO::print("%s: Note: Old-style conditions are deprecated\n", get_source());
 	Condition cond(obj, c);
-	load_sub(cond);
-	obj.conditions.push_back(cond);
+	if(cond.eval())
+		load_sub_with(*this);
 }
 
 template<Component::Type t>
@@ -290,6 +224,42 @@ void SourcePackage::Loader::headers(const string &n)
 		install_map[*i] = "include/"+comp.get_name();
 }
 
+void SourcePackage::Loader::if_arch(const string &cond)
+{
+	const Architecture &arch = obj.builder.get_current_arch();
+	bool negate = (cond[0]=='!');
+	bool match = (arch.match_name(cond.substr(negate))!=negate);
+	obj.builder.get_logger().log("configure", format("%s: arch %s %smatched", obj.name, cond, (match ? "" : "not ")));
+	if(match)
+		load_sub_with(*this);
+}
+
+void SourcePackage::Loader::if_feature(const string &cond)
+{
+	bool match = false;
+	string::size_type equals = cond.find('=');
+	if(equals!=string::npos)
+	{
+		if(equals==0)
+			error("No feature name specified");
+		bool negate = cond[equals-1]=='!';
+		string name = cond.substr(0, equals-negate);
+		string value = obj.config.get_option("with_"+name).value;
+		match = (value==cond.substr(equals+1))!=negate;
+		value = cond.substr(equals+1);
+	}
+	else
+	{
+		bool negate = (cond[0]=='!');
+		string name = cond.substr(negate);
+		string value = obj.config.get_option("with_"+name).value;
+		match = lexical_cast<bool>(value)!=negate;
+	}
+	obj.builder.get_logger().log("configure", format("%s: feature %s %smatched", obj.name, cond, (match ? "" : "not ")));
+	if(match)
+		load_sub_with(*this);
+}
+
 void SourcePackage::Loader::tarball(const string &n)
 {
 	if(n=="@src")