X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fbuilder.cpp;h=2ef5a709e1913d33cae79f1677ce1b21290d1cee;hb=0ac8d6708b8c758fb89d97a0049d4a329990b83b;hp=336cc8453c118e519b266c8a2ebc8c757385e9dc;hpb=555190f7eafcf3a67750fc63872b23e17371aa98;p=builder.git

diff --git a/source/builder.cpp b/source/builder.cpp
index 336cc84..2ef5a70 100644
--- a/source/builder.cpp
+++ b/source/builder.cpp
@@ -10,13 +10,15 @@ Distributed under the LGPL
 #include <set>
 #include <msp/core/error.h>
 #include <msp/core/getopt.h>
-#include <msp/parser/parser.h>
+#include <msp/datafile/parser.h>
 #include <msp/path/utils.h>
+#include <msp/strings/formatter.h>
 #include <msp/strings/utils.h>
 #include <msp/time/units.h>
 #include <msp/time/utils.h>
 #include "action.h"
 #include "analyzer.h"
+#include "binarypackage.h"
 #include "builder.h"
 #include "header.h"
 #include "install.h"
@@ -24,6 +26,7 @@ Distributed under the LGPL
 #include "package.h"
 #include "pkgconfig.h"
 #include "sharedlibrary.h"
+#include "sourcepackage.h"
 #include "systemlibrary.h"
 #include "tarball.h"
 #include "unlink.h"
@@ -33,13 +36,14 @@ using namespace std;
 using namespace Msp;
 
 Builder::Builder(int argc, char **argv):
+	main_pkg(0),
 	analyzer(0),
 	build(false),
 	clean(0),
 	dry_run(false),
 	help(false),
 	verbose(1),
-	chrome(false),
+	show_progress(false),
 	build_file("Build"),
 	jobs(1),
 	conf_all(false),
@@ -64,8 +68,8 @@ Builder::Builder(int argc, char **argv):
 	getopt.add_option('A', "conf-all",   conf_all,     GetOpt::NO_ARG);
 	getopt.add_option('B', "build-all",  build_all,    GetOpt::NO_ARG);
 	getopt.add_option('C', "chdir",      work_dir,     GetOpt::REQUIRED_ARG);
+	getopt.add_option('P', "progress",   show_progress, GetOpt::NO_ARG);
 	getopt.add_option('W', "what-if",    what_if,      GetOpt::REQUIRED_ARG);
-	getopt.add_option(     "chrome",     chrome,       GetOpt::NO_ARG);
 	getopt.add_option(     "conf-only",  conf_only,    GetOpt::NO_ARG);
 	getopt.add_option(     "full-paths", full_paths,   GetOpt::NO_ARG);
 	//getopt.add_option(     "makefile",   create_makefile, GetOpt::NO_ARG);
@@ -93,8 +97,8 @@ Builder::Builder(int argc, char **argv):
 	else if(!clean && !create_makefile)
 		build=true;
 
-	const list<string> &args=getopt.get_args();
-	for(list<string>::const_iterator i=args.begin(); i!=args.end(); ++i)
+	const vector<string> &args=getopt.get_args();
+	for(vector<string>::const_iterator i=args.begin(); i!=args.end(); ++i)
 	{
 		unsigned equal=i->find('=');
 		if(equal!=string::npos)
@@ -111,35 +115,16 @@ Builder::Builder(int argc, char **argv):
 
 	cwd=Path::getcwd();
 
-	archs.insert(StringMap::value_type("native", ""));
-	archs.insert(StringMap::value_type("arm", "arm-linux-gnu"));
-	archs.insert(StringMap::value_type("win32", "i586-mingw32msvc"));
+	Architecture &native_arch=archs.insert(ArchMap::value_type("native", Architecture(*this, "native"))).first->second;
+	native_arch.set_tool("CC",  "gcc");
+	native_arch.set_tool("CXX", "g++");
+	native_arch.set_tool("LD",  "gcc");
+	native_arch.set_tool("LXX", "g++");
+	native_arch.set_tool("AR",  "ar");
 
-	StringMap &native_tools=tools.insert(ToolMap::value_type("native", StringMap())).first->second;
-	native_tools.insert(StringMap::value_type("CC",   "gcc"));
-	native_tools.insert(StringMap::value_type("CXX",  "g++"));
-	native_tools.insert(StringMap::value_type("LD",   "gcc"));
-	native_tools.insert(StringMap::value_type("LDXX", "g++"));
-	native_tools.insert(StringMap::value_type("AR",   "ar"));
-
-	StringMap &release_profile=profile_tmpl.insert(ProfileTemplateMap::value_type("release", StringMap())).first->second;
-	release_profile.insert(StringMap::value_type("optimize", "3"));
-	release_profile.insert(StringMap::value_type("outdir",   "$profile"));
-
-	StringMap &debug_profile=profile_tmpl.insert(ProfileTemplateMap::value_type("debug", StringMap())).first->second;
-	debug_profile.insert(StringMap::value_type("debug",  "1"));
-	debug_profile.insert(StringMap::value_type("outdir", "$profile"));
-
-	for(StringMap::iterator i=archs.begin(); i!=archs.end(); ++i)
-	{
-		if(i->first=="native")
-			continue;
-
-		StringMap &arch_profile=profile_tmpl.insert(ProfileTemplateMap::value_type(i->first, StringMap())).first->second;
-		arch_profile.insert(StringMap::value_type("arch",   i->first));
-		arch_profile.insert(StringMap::value_type("prefix", "$HOME/local/$arch"));
-		arch_profile.insert(StringMap::value_type("outdir", "$profile"));
-	}
+	const char *home=getenv("HOME");
+	if(home)
+		load_build_file((Path::Path(home)/".builderrc").str());
 }
 
 /**
@@ -184,10 +169,12 @@ Package *Builder::get_package(const string &n)
 		}
 
 	// Package source not found - create a binary package
-	Package *pkg=Package::create(*this, n);
+	Package *pkg=BinaryPackage::from_pkgconfig(*this, n);
+
 	packages.insert(PackageMap::value_type(n, pkg));
-	if(pkg)
-		new_pkgs.push_back(pkg);
+
+	if(!pkg)
+		problem(n, "not found");
 
 	return pkg;
 }
@@ -271,7 +258,7 @@ Target *Builder::get_library(const string &lib, const string &arch, const list<s
 		syspath.push_back("/usr/lib");
 	}
 	else
-		syspath.push_back("/usr/"+get_arch_prefix(arch)+"/lib");
+		syspath.push_back("/usr/"+get_architecture(arch).get_prefix()+"/lib");
 
 	Target *tgt=0;
 	for(StringList::iterator j=syspath.begin(); (!tgt && j!=syspath.end()); ++j)
@@ -284,34 +271,15 @@ Target *Builder::get_library(const string &lib, const string &arch, const list<s
 	return tgt;
 }
 
-const string &Builder::get_arch_prefix(const string &arch) const
+const Architecture &Builder::get_architecture(const string &arch) const
 {
-	StringMap::const_iterator i=archs.find(arch);
+	ArchMap::const_iterator i=archs.find(arch);
 	if(i==archs.end())
 		throw InvalidParameterValue("Unknown architecture");
 
 	return i->second;
 }
 
-string Builder::get_tool(const std::string &tool, const std::string &arch)
-{
-	ToolMap::iterator i=tools.find(arch);
-	if(i!=tools.end())
-	{
-		StringMap::iterator j=i->second.find(tool);
-		if(j!=i->second.end())
-			return j->second;
-	}
-
-	// Either the arch, or the tool within the arch was not found
-	i=tools.find("native");
-	StringMap::iterator j=i->second.find(tool);
-	if(j==i->second.end())
-		throw InvalidParameterValue("Unknown tool");
-
-	return get_arch_prefix(arch)+"-"+j->second;
-}
-
 void Builder::apply_profile_template(Config &config, const string &pt) const
 {
 	vector<string> parts=split(pt, '-');
@@ -326,6 +294,11 @@ void Builder::apply_profile_template(Config &config, const string &pt) const
 	}
 }
 
+void Builder::problem(const string &p, const string &d)
+{
+	problems.push_back(Problem(p, d));
+}
+
 /**
 Adds a target to both the target map and the new target queue.  Called from
 Target constructor.
@@ -344,16 +317,7 @@ int Builder::main()
 		return 1;
 	}
 
-	default_pkg=packages.begin()->second;
-
-	while(!new_pkgs.empty())
-	{
-		Package *pkg=new_pkgs.front();
-		new_pkgs.erase(new_pkgs.begin());
-		pkg->resolve_refs();
-	}
-
-	default_pkg->configure(cmdline_options, conf_all?2:1);
+	main_pkg->configure(cmdline_options, conf_all?2:1);
 
 	if(help)
 	{
@@ -363,23 +327,24 @@ int Builder::main()
 		return 0;
 	}
 
-	StringMap problems;
-	for(PackageMap::iterator i=packages.begin(); i!=packages.end(); ++i)
+	if(!conf_only && create_targets())
+		return 1;
+
+	PackageList all_reqs=main_pkg->collect_requires();
+
+	for(PackageList::const_iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
 	{
+		SourcePackage *spkg=dynamic_cast<SourcePackage *>(*i);
 		string prob;
-		if(!i->second)
-			prob="missing";
-		else if(i->second->get_buildable() && i->second->get_arch()!=default_pkg->get_arch())
-			prob="wrong architecture ("+i->second->get_arch()+")";
-		if(!prob.empty())
-			problems.insert(StringMap::value_type(i->first, prob));
+		if(spkg && spkg->get_arch()!=main_pkg->get_arch())
+			problem(spkg->get_name(), format("wrong architecture (%s)", spkg->get_arch()));
 	}
 
 	if(!problems.empty())
 	{
 		cerr<<"The following problems were detected:\n";
-		for(StringMap::iterator i=problems.begin(); i!=problems.end(); ++i)
-			cerr<<"  "<<i->first<<": "<<i->second<<'\n';
+		for(ProblemList::iterator i=problems.begin(); i!=problems.end(); ++i)
+			cerr<<"  "<<i->package<<": "<<i->descr<<'\n';
 		cerr<<"Please fix them and try again.\n";
 		return 1;
 	}
@@ -387,21 +352,18 @@ int Builder::main()
 	if(conf_only)
 		return 0;
 
-	if(create_targets())
-		return 1;
-
-	cout<<packages.size()<<" packages, "<<targets.size()<<" targets\n";
+	cout<<all_reqs.size()<<" active packages, "<<targets.size()<<" targets\n";
 	if(verbose>=2)
 	{
-		for(PackageMap::iterator i=packages.begin(); i!=packages.end(); ++i)
+		for(PackageList::const_iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
 		{
-			cout<<' '<<i->second->get_name();
-			if(i->second->get_buildable())
+			cout<<' '<<(*i)->get_name();
+			if(dynamic_cast<SourcePackage *>(*i))
 				cout<<'*';
 			unsigned count=0;
 			unsigned ood_count=0;
 			for(TargetMap::iterator j=targets.begin(); j!=targets.end(); ++j)
-				if(j->second->get_package()==i->second)
+				if(j->second->get_package()==*i)
 				{
 					++count;
 					if(j->second->get_rebuild())
@@ -464,8 +426,8 @@ void Builder::usage(const char *reason, const char *argv0, bool brief)
 			"  -A, --conf-all      Apply configuration to all packages.\n"
 			"  -B, --build-all     Build all targets unconditionally.\n"
 			"  -C, --chdir DIR     Change to DIR before doing anything else.\n"
+			"  -P, --progress      Display progress while building.\n"
 			"  -W, --what-if FILE  Pretend that FILE has changed.\n"
-			"  --chrome            Use extra chrome to print status.\n"
 			"  --conf-only         Stop after configuring packages.\n"
 			"  --full-paths        Output full paths in analysis.\n"
 			//"  --makefile          Create a makefile for this package.\n"
@@ -486,7 +448,10 @@ int Builder::load_build_file(const Path::Path &fn)
 	if(!in)
 		return -1;
 
-	Parser::Parser parser(in, fn.str());
+	if(verbose>=3)
+		cout<<"Reading "<<fn<<'\n';
+
+	DataFile::Parser parser(in, fn.str());
 	Loader loader(*this, fn.subpath(0, fn.size()-1));
 	loader.load(parser);
 
@@ -512,24 +477,24 @@ int Builder::create_targets()
 	Target *tarballs=new VirtualTarget(*this, "tarballs");
 	world->add_depend(tarballs);
 
-	for(PackageMap::iterator i=packages.begin(); i!=packages.end(); ++i)
+	PackageList all_reqs=main_pkg->collect_requires();
+	for(PackageList::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
 	{
-		if(!i->second)
-			continue;
-		if(!i->second->get_buildable())
+		SourcePackage *spkg=dynamic_cast<SourcePackage *>(*i);
+		if(!spkg)
 			continue;
 
-		const ComponentList &components=i->second->get_components();
+		const ComponentList &components=spkg->get_components();
 		for(ComponentList::const_iterator j=components.begin(); j!=components.end(); ++j)
 			j->create_targets();
 
-		if(i->second->get_install_flags()&(Package::LIB|Package::INCLUDE))
+		if(spkg->get_install_flags()&(SourcePackage::LIB|SourcePackage::INCLUDE))
 		{
-			PkgConfig *pc=new PkgConfig(*this, *i->second);
-			install->add_depend(new Install(*this, *i->second, *pc));
+			PkgConfig *pc=new PkgConfig(*this, *spkg);
+			install->add_depend(new Install(*this, *spkg, *pc));
 		}
 
-		tarballs->add_depend(new TarBall(*this, *i->second));
+		tarballs->add_depend(new TarBall(*this, *spkg));
 	}
 
 	// Find dependencies until no new targets are created
@@ -583,6 +548,10 @@ int Builder::create_targets()
 		world->prepare();
 	}
 
+	for(PackageMap::iterator i=packages.begin(); i!=packages.end(); ++i)
+		if(SourcePackage *spkg=dynamic_cast<SourcePackage *>(i->second))
+			spkg->get_deps_cache().save();
+
 	return 0;
 }
 
@@ -677,8 +646,6 @@ int Builder::do_build()
 
 	vector<Action *> actions;
 
-	if(chrome)
-		cout<<"0 targets built\n";
 	unsigned count=0;
 
 	bool fail=false;
@@ -694,6 +661,12 @@ int Builder::do_build()
 				Action *action=tgt->build();
 				if(action)
 					actions.push_back(action);
+
+				if(show_progress)
+				{
+					cout<<count<<" of "<<total<<" targets built\033[1G";
+					cout.flush();
+				}
 			}
 			else if(actions.empty())
 				finish=true;
@@ -707,17 +680,7 @@ int Builder::do_build()
 			if(status>=0)
 			{
 				++count;
-				if(chrome)
-				{
-					cout<<"\e["<<actions.size()+1<<'A';
-					cout<<count<<" targets built\n";
-					if(i)
-						cout<<"\e["<<i<<"B";
-					cout<<"\e[M";
-					if(i<actions.size()-1)
-						cout<<"\e["<<actions.size()-i-1<<"B";
-					cout.flush();
-				}
+
 				delete actions[i];
 				actions.erase(actions.begin()+i);
 				if(status>0)
@@ -730,8 +693,12 @@ int Builder::do_build()
 		}
 	}
 
+	if(show_progress)
+		cout<<"\033[K";
 	if(fail)
 		cout<<"Build failed\n";
+	else if(show_progress)
+		cout<<"Build complete\n";
 
 	return fail?1:0;
 }
@@ -753,7 +720,7 @@ int Builder::do_clean()
 		Target *tgt=queue.front();
 		queue.erase(queue.begin());
 
-		if(tgt->get_buildable() && (tgt->get_package()==default_pkg || clean>=2))
+		if(tgt->get_buildable() && (tgt->get_package()==main_pkg || clean>=2))
 			clean_tgts.insert(tgt);
 
 		const TargetList &deps=tgt->get_depends();
@@ -777,16 +744,16 @@ Prints out information about the default package.
 */
 void Builder::package_help()
 {
-	const Config &config=default_pkg->get_config();
+	const Config &config=main_pkg->get_config();
 	const Config::OptionMap &options=config.get_options();
 
 	cout<<"Required packages:\n  ";
-	const PkgRefList &requires=default_pkg->get_requires();
-	for(PkgRefList::const_iterator i=requires.begin(); i!=requires.end(); ++i)
+	const PackageList &requires=main_pkg->get_requires();
+	for(PackageList::const_iterator i=requires.begin(); i!=requires.end(); ++i)
 	{
 		if(i!=requires.begin())
 			cout<<", ";
-		cout<<i->get_name();
+		cout<<(*i)->get_name();
 	}
 	cout<<"\n\n";
 	cout<<"Package configuration:\n";
@@ -799,18 +766,56 @@ void Builder::package_help()
 
 Application::RegApp<Builder> Builder::reg;
 
+
 Builder::Loader::Loader(Builder &b, const Path::Path &s):
 	bld(b),
 	src(s)
 {
+	add("architecture", &Loader::architecture);
+	add("binary_package", &Loader::binpkg);
+	add("profile", &Loader::profile);
 	add("package", &Loader::package);
 }
 
+void Builder::Loader::architecture(const string &n)
+{
+	Architecture arch(bld, n);
+	load_sub(arch);
+	bld.archs.insert(ArchMap::value_type(n, arch));
+}
+
+void Builder::Loader::binpkg(const string &n)
+{
+	BinaryPackage *pkg=new BinaryPackage(bld, n);
+	load_sub(*pkg);
+	bld.packages.insert(PackageMap::value_type(n, pkg));
+}
+
+void Builder::Loader::profile(const string &n)
+{
+	StringMap prf;
+	load_sub<ProfileLoader>(prf);
+	bld.profile_tmpl.insert(ProfileTemplateMap::value_type(n, prf));
+}
+
 void Builder::Loader::package(const string &n)
 {
-	Package *pkg=new Package(bld, n, src);
+	SourcePackage *pkg=new SourcePackage(bld, n, src);
+	if(!bld.main_pkg)
+		bld.main_pkg=pkg;
+
 	load_sub(*pkg);
 	bld.packages.insert(PackageMap::value_type(n, pkg));
-	bld.new_pkgs.push_back(pkg);
 }
 
+
+Builder::ProfileLoader::ProfileLoader(StringMap &p):
+	profile(p)
+{
+	add("option", &ProfileLoader::option);
+}
+
+void Builder::ProfileLoader::option(const string &o, const string &v)
+{
+	profile.insert(StringMap::value_type(o, v));
+}