]> git.tdb.fi Git - builder.git/blobdiff - source/builder.cpp
Big rewrite for a more tool-centric approach
[builder.git] / source / builder.cpp
index ce82ba33351f191f1055eae0a053c915247a2053..57764b781b26a3030a9b629567c241ed1a74937c 100644 (file)
@@ -1,41 +1,37 @@
-/* $Id$
-
-This file is part of builder
-Copyright © 2006-2009  Mikko Rasa, Mikkosoft Productions
-Distributed under the LGPL
-*/
-
 #include <set>
 #include <cstdlib>
-#include <sys/utsname.h>
-#include <msp/core/except.h>
 #include <msp/core/getopt.h>
 #include <msp/datafile/parser.h>
 #include <msp/fs/dir.h>
 #include <msp/fs/stat.h>
 #include <msp/fs/utils.h>
 #include <msp/io/buffered.h>
-#include <msp/io/except.h>
 #include <msp/io/file.h>
 #include <msp/io/print.h>
-#include <msp/strings/formatter.h>
+#include <msp/strings/format.h>
 #include <msp/strings/regex.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 "copy.h"
+#include "gnuarchiver.h"
+#include "gnuccompiler.h"
+#include "gnucxxcompiler.h"
+#include "gnulinker.h"
 #include "header.h"
 #include "install.h"
 #include "misc.h"
 #include "package.h"
 #include "pkgconfig.h"
+#include "pkgconfiggenerator.h"
 #include "sharedlibrary.h"
 #include "sourcepackage.h"
 #include "systemlibrary.h"
-#include "unlink.h"
+#include "tar.h"
+#include "task.h"
 #include "virtualtarget.h"
 
 using namespace std;
@@ -46,7 +42,7 @@ namespace {
 void update_hash(string &hash, const string &value)
 {
        for(unsigned i=0; i<value.size(); ++i)
-               hash[i%hash.size()]^=value[i];
+               hash[i%hash.size()] ^= value[i];
 }
 
 }
@@ -54,6 +50,7 @@ void update_hash(string &hash, const string &value)
 
 Builder::Builder(int argc, char **argv):
        main_pkg(0),
+       native_arch(*this, string()),
        analyzer(0),
        build(false),
        clean(0),
@@ -70,8 +67,8 @@ Builder::Builder(int argc, char **argv):
 {
        string analyze_mode;
        string work_dir;
-       bool full_paths=false;
-       unsigned max_depth=5;
+       bool full_paths = false;
+       unsigned max_depth = 5;
        StringList cmdline_warn;
        string prfx;
        string arch;
@@ -85,6 +82,7 @@ Builder::Builder(int argc, char **argv):
        getopt.add_option('j', "jobs",       jobs,          GetOpt::REQUIRED_ARG).set_help("Run NUM commands at once, whenever possible.", "NUM");
        getopt.add_option('n', "dry-run",    dry_run,       GetOpt::NO_ARG).set_help("Don't actually do anything, only show what would be done.");
        getopt.add_option('v', "verbose",    verbose,       GetOpt::NO_ARG).set_help("Print more information about what's going on.");
+       getopt.add_option('x', "no-externals",  no_externals, GetOpt::NO_ARG).set_help("Do not load external source packages.");
        getopt.add_option('A', "conf-all",   conf_all,      GetOpt::NO_ARG).set_help("Apply configuration to all packages.");
        getopt.add_option('B', "build-all",  build_all,     GetOpt::NO_ARG).set_help("Build all targets unconditionally.");
        getopt.add_option('C', "chdir",      work_dir,      GetOpt::REQUIRED_ARG).set_help("Change to DIR before doing anything else.", "DIR");
@@ -96,13 +94,13 @@ Builder::Builder(int argc, char **argv):
        getopt.add_option(     "max-depth",  max_depth,     GetOpt::REQUIRED_ARG).set_help("Maximum depth to show in analysis.", "NUM");
        getopt.add_option(     "prefix",     prfx,          GetOpt::REQUIRED_ARG).set_help("Directory to install things to.", "DIR");
        getopt.add_option(     "warnings",   cmdline_warn,  GetOpt::REQUIRED_ARG).set_help("Compiler warnings to use.", "LIST");
-       usagemsg=getopt.generate_usage(argv[0])+" [<target> ...]";
-       helpmsg=getopt.generate_help();
+       usagemsg = getopt.generate_usage(argv[0])+" [<target> ...]";
+       helpmsg = getopt.generate_help();
        getopt(argc, argv);
 
        if(!analyze_mode.empty())
        {
-               analyzer=new Analyzer(*this);
+               analyzer = new Analyzer(*this);
 
                if(analyze_mode=="deps")
                        analyzer->set_mode(Analyzer::DEPS);
@@ -113,18 +111,18 @@ Builder::Builder(int argc, char **argv):
                else if(analyze_mode=="rdeps")
                        analyzer->set_mode(Analyzer::RDEPS);
                else
-                       throw UsageError("Invalid analyze mode");
+                       throw usage_error("Invalid analyze mode");
 
                analyzer->set_max_depth(max_depth);
                analyzer->set_full_paths(full_paths);
        }
        else if(!clean && !create_makefile)
-               build=true;
+               build = true;
 
-       const vector<string> &args=getopt.get_args();
+       const vector<string> &args = getopt.get_args();
        for(vector<string>::const_iterator i=args.begin(); i!=args.end(); ++i)
        {
-               string::size_type equal=i->find('=');
+               string::size_type equal = i->find('=');
                if(equal!=string::npos)
                        cmdline_options.insert(StringMap::value_type(i->substr(0, equal), i->substr(equal+1)));
                else
@@ -137,37 +135,43 @@ Builder::Builder(int argc, char **argv):
        if(!work_dir.empty())
                FS::chdir(work_dir);
 
-       cwd=FS::getcwd();
+       cwd = FS::getcwd();
 
-       utsname un;
-       string sysname="native";
-       if(uname(&un)==0)
-               sysname=tolower(un.sysname);
-
-       native_arch=&archs.insert(ArchMap::value_type(sysname, Architecture(*this, sysname, true))).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");
+       toolchain.add_tool(new GnuCCompiler(*this));
+       toolchain.add_tool(new GnuCxxCompiler(*this));
+       toolchain.add_tool(new GnuLinker(*this));
+       toolchain.add_tool(new GnuArchiver(*this));
+       toolchain.add_tool(new Copy(*this));
+       toolchain.add_tool(new Tar(*this));
+       toolchain.add_tool(new PkgConfigGenerator(*this));
 
        load_build_file((FS::get_sys_data_dir(argv[0], "builder")/"builderrc").str());
        load_build_file((FS::get_user_data_dir("builder")/"rc").str());
 
        if(arch.empty())
-               current_arch=native_arch;
+               current_arch = &native_arch;
        else
-               current_arch=&get_architecture(arch);
+               current_arch = new Architecture(*this, arch);
+
+       if(!current_arch->is_native())
+       {
+               for(StringMap::const_iterator i=cross_prefixes.begin(); i!=cross_prefixes.end(); ++i)
+                       if(current_arch->match_name(i->first))
+                       {
+                               current_arch->set_cross_prefix(i->second);
+                               break;
+                       }
+       }
 
        if(prfx.empty())
        {
                if(current_arch->is_native())
-                       prefix=(FS::get_home_dir()/"local").str();
+                       prefix = (FS::get_home_dir()/"local").str();
                else
-                       prefix=(FS::get_home_dir()/"local"/current_arch->get_name()).str();
+                       prefix = (FS::get_home_dir()/"local"/current_arch->get_name()).str();
        }
        else
-               prefix=FS::getcwd()/prfx;
+               prefix = FS::getcwd()/prfx;
 
        warnings.push_back("all");
        warnings.push_back("extra");
@@ -176,7 +180,7 @@ Builder::Builder(int argc, char **argv):
        warnings.push_back("error");
        for(StringList::iterator i=cmdline_warn.begin(); i!=cmdline_warn.end(); ++i)
        {
-               vector<string> warns=split(*i, ',');
+               vector<string> warns = split(*i, ',');
                warnings.insert(warnings.end(), warns.begin(), warns.end());
        }
 
@@ -197,13 +201,13 @@ int Builder::main()
 {
        if(prefix.str()!="/usr")
        {
-               FS::Path pcdir=prefix/"lib"/"pkgconfig";
-               if(const char *pcp=getenv("PKG_CONFIG_PATH"))
+               FS::Path pcdir = prefix/"lib"/"pkgconfig";
+               if(const char *pcp = getenv("PKG_CONFIG_PATH"))
                {
-                       vector<string> path=split(pcp, ':');
-                       bool found=false;
+                       vector<string> path = split(pcp, ':');
+                       bool found = false;
                        for(vector<string>::const_iterator i=path.begin(); (!found && i!=path.end()); ++i)
-                               found=(*i==pcdir.str());
+                               found = (*i==pcdir.str());
                        if(!found)
                        {
                                path.push_back(pcdir.str());
@@ -238,29 +242,42 @@ int Builder::main()
                return 0;
        }
 
-       if(!conf_only && create_targets())
-               return 1;
-
-       PackageList all_reqs=main_pkg->collect_requires();
-
        if(conf_only)
                return 0;
 
+       if(create_targets())
+               return 1;
+
        if(verbose>=2)
-               IO::print("Building on %s, for %s\n", native_arch->get_name(), current_arch->get_name());
+       {
+               IO::print("Building on %s, for %s%s\n", native_arch.get_name(),
+                       current_arch->get_name(), (current_arch->is_native() ? " (native)" : ""));
+               IO::print("Prefix is %s\n", prefix);
+       }
+
        if(verbose>=1)
-               IO::print("%d active packages, %d targets\n", all_reqs.size(), targets.size());
+       {
+               unsigned n_packages = 0;
+               for(PackageMap::const_iterator i=packages.begin(); i!=packages.end(); ++i)
+                       if(i->second && i->second->is_configured())
+                               ++n_packages;
+               IO::print("%d active packages, %d targets\n", n_packages, targets.size());
+       }
+
        if(verbose>=2)
        {
-               for(PackageList::const_iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
+               for(PackageMap::const_iterator i=packages.begin(); i!=packages.end(); ++i)
                {
-                       IO::print(" %s", (*i)->get_name());
-                       if(dynamic_cast<SourcePackage *>(*i))
+                       if(!i->second->is_configured())
+                               continue;
+
+                       IO::print(" %s", i->second->get_name());
+                       if(dynamic_cast<SourcePackage *>(i->second))
                                IO::print("*");
-                       unsigned count=0;
-                       unsigned to_be_built=0;
+                       unsigned count = 0;
+                       unsigned to_be_built = 0;
                        for(TargetMap::iterator j=targets.begin(); j!=targets.end(); ++j)
-                               if(j->second->get_package()==*i)
+                               if(j->second->get_package()==i->second)
                                {
                                        ++count;
                                        if(j->second->get_rebuild())
@@ -291,9 +308,9 @@ int Builder::main()
        }
 
        if(clean)
-               exit_code=do_clean();
+               exit_code = do_clean();
        else if(build)
-               exit_code=do_build();
+               exit_code = do_build();
 
        return exit_code;
 }
@@ -317,34 +334,37 @@ string Builder::run_pkgconfig(const string &pkg, const string &what)
                IO::print("Running %s\n", join(argv.begin(), argv.end()));
 
        int status;
-       string res=run_command(argv, &status);
+       string res = run_command(argv, &status);
        if(status)
-               throw Exception(format("pkg-config for package %s failed", pkg));
+               throw runtime_error(format("pkg-config for package %s failed", pkg));
 
        return res;
 }
 
 Package *Builder::get_package(const string &name)
 {
-       PackageMap::iterator i=packages.find(format("%s/%s", name, current_arch->get_name()));
+       PackageMap::iterator i = packages.find(format("%s/%s", name, current_arch->get_system()));
        if(i==packages.end())
-               i=packages.find(name);
+               i = packages.find(name);
        if(i!=packages.end())
                return i->second;
 
-       FS::Path path=get_package_location(name);
-       if(!path.empty() && !load_build_file(path/"Build"))
+       if(!no_externals)
        {
-               i=packages.find(name);
-               if(i!=packages.end())
-                       return i->second;
+               FS::Path path = get_package_location(name);
+               if(!path.empty() && !load_build_file(path/"Build"))
+               {
+                       i = packages.find(name);
+                       if(i!=packages.end())
+                               return i->second;
+               }
        }
 
-       Package *pkg=0;
+       Package *pkg = 0;
        try
        {
                // Package source not found - create a binary package
-               pkg=BinaryPackage::from_pkgconfig(*this, name);
+               pkg = BinaryPackage::from_pkgconfig(*this, name);
        }
        catch(...)
        {
@@ -358,13 +378,20 @@ Package *Builder::get_package(const string &name)
 
 Target *Builder::get_target(const string &n) const
 {
-       // XXX Used for getting targets by path.  get_target(const FS::Path &)?
-       TargetMap::const_iterator i=targets.find(n);
+       TargetMap::const_iterator i = targets.find(n);
        if(i!=targets.end())
                return i->second;
        return 0;
 }
 
+FileTarget *Builder::get_target_by_path(const FS::Path &p) const
+{
+       TargetMap::const_iterator i = targets_by_path.find(p.str());
+       if(i!=targets_by_path.end())
+               return static_cast<FileTarget *>(i->second);
+       return 0;
+}
+
 Target *Builder::get_header(const string &include, const FS::Path &from, const list<string> &path)
 {
        string hash(8, 0);
@@ -373,23 +400,24 @@ Target *Builder::get_header(const string &include, const FS::Path &from, const l
        for(list<string>::const_iterator i=path.begin(); i!=path.end(); ++i)
                update_hash(hash, *i);
 
-       string id=hash+include;
-       TargetMap::iterator i=includes.find(id);
+       string id = hash+include;
+       TargetMap::iterator i = includes.find(id);
        if(i!=includes.end())
                return i->second;
 
        static string cxx_ver;
        if(cxx_ver.empty())
        {
-               StringList argv;
+               // XXX This needs to go elsewhere
+               /*StringList argv;
                argv.push_back(current_arch->get_tool("CXX"));
                argv.push_back("--version");
-               if(RegMatch m=Regex("[0-9]\\.[0-9.]+").match(run_command(argv)))
+               if(RegMatch m = Regex("[0-9]\\.[0-9.]+").match(run_command(argv)))
                {
-                       cxx_ver=m[0].str;
+                       cxx_ver = m[0].str;
                        while(!cxx_ver.empty() && !FS::is_dir(FS::Path("/usr/include/c++")/cxx_ver))
                        {
-                               string::size_type dot=cxx_ver.rfind('.');
+                               string::size_type dot = cxx_ver.rfind('.');
                                if(dot==string::npos)
                                        break;
                                cxx_ver.erase(dot);
@@ -397,11 +425,11 @@ Target *Builder::get_header(const string &include, const FS::Path &from, const l
                        if(verbose>=5)
                                IO::print("C++ version is %s\n", cxx_ver);
                }
-               else
-                       cxx_ver="-";
+               else*/
+                       cxx_ver = "-";
        }
 
-       string fn=include.substr(1);
+       string fn = include.substr(1);
        if(verbose>=5)
                IO::print("Looking for include %s from %s with path %s\n", fn, from, join(path.begin(), path.end()));
 
@@ -409,17 +437,17 @@ Target *Builder::get_header(const string &include, const FS::Path &from, const l
        if(current_arch->is_native())
                syspath.push_back("/usr/include");
        else
-               syspath.push_back("/usr/"+current_arch->get_prefix()+"/include");
+               syspath.push_back("/usr/"+current_arch->get_cross_prefix()+"/include");
        if(cxx_ver!="-")
                syspath.push_back((FS::Path("/usr/include/c++/")/cxx_ver).str());
 
-       Target *tgt=0;
+       Target *tgt = 0;
        if(include[0]=='\"')
-               tgt=get_header(FS::Path(from)/fn);
+               tgt = get_header(FS::Path(from)/fn);
        for(list<string>::const_iterator j=path.begin(); (!tgt && j!=path.end()); ++j)
-               tgt=get_header(cwd/ *j/fn);
+               tgt = get_header(cwd/ *j/fn);
        for(list<string>::const_iterator j=syspath.begin(); (!tgt && j!=syspath.end()); ++j)
-               tgt=get_header(FS::Path(*j)/fn);
+               tgt = get_header(FS::Path(*j)/fn);
 
        includes.insert(TargetMap::value_type(id, tgt));
 
@@ -432,8 +460,8 @@ Target *Builder::get_library(const string &lib, const list<string> &path, LibMod
        for(list<string>::const_iterator i=path.begin(); i!=path.end(); ++i)
                update_hash(hash, *i);
 
-       string id=hash+string(1, mode)+lib;
-       TargetMap::iterator i=libraries.find(id);
+       string id = hash+string(1, mode)+lib;
+       TargetMap::iterator i = libraries.find(id);
        if(i!=libraries.end())
                return i->second;
 
@@ -442,40 +470,35 @@ Target *Builder::get_library(const string &lib, const list<string> &path, LibMod
        {
                syspath.push_back("/lib");
                syspath.push_back("/usr/lib");
+               if(current_arch->match_name("pc-32-linux"))
+                       syspath.push_back("/usr/lib/i386-linux-gnu");
+               else if(current_arch->match_name("pc-64-linux"))
+                       syspath.push_back("/usr/lib/x86_64-linux-gnu");
        }
        else
-               syspath.push_back("/usr/"+current_arch->get_prefix()+"/lib");
+               syspath.push_back("/usr/"+current_arch->get_cross_prefix()+"/lib");
 
        if(verbose>=5)
                IO::print("Looking for library %s with path %s\n", lib, join(path.begin(), path.end()));
 
-       Target *tgt=0;
+       Target *tgt = 0;
        for(StringList::const_iterator j=path.begin(); (!tgt && j!=path.end()); ++j)
-               tgt=get_library(lib, cwd/ *j, mode);
+               tgt = get_library(lib, cwd/ *j, mode);
        for(StringList::iterator j=syspath.begin(); (!tgt && j!=syspath.end()); ++j)
-               tgt=get_library(lib, *j, mode);
+               tgt = get_library(lib, *j, mode);
 
        libraries.insert(TargetMap::value_type(id, tgt));
 
        return tgt;
 }
 
-const Architecture &Builder::get_architecture(const string &arch) const
-{
-       ArchMap::const_iterator i=archs.find(arch);
-       if(i==archs.end())
-               throw KeyError("Unknown architecture", arch);
-
-       return i->second;
-}
-
 void Builder::apply_profile_template(Config &config, const string &pt) const
 {
-       vector<string> parts=split(pt, '-');
+       vector<string> parts = split(pt, '-');
 
        for(vector<string>::iterator i=parts.begin(); i!=parts.end(); ++i)
        {
-               ProfileTemplateMap::const_iterator j=profile_tmpl.find(*i);
+               ProfileTemplateMap::const_iterator j = profile_tmpl.find(*i);
                if(j==profile_tmpl.end())
                        continue;
 
@@ -494,6 +517,11 @@ void Builder::add_target(Target *t)
        new_tgts.push_back(t);
 }
 
+void Builder::register_path(const FS::Path &path, FileTarget *t)
+{
+       targets_by_path.insert(TargetMap::value_type(path.str(), t));
+}
+
 void Builder::usage(const char *reason, const char *argv0, bool brief)
 {
        if(reason)
@@ -518,7 +546,7 @@ FS::Path Builder::get_package_location(const string &name)
        try
        {
                // Try to get source directory with pkgconfig
-               string srcdir=strip(run_pkgconfig(name, "source"));
+               string srcdir = strip(run_pkgconfig(name, "source"));
                if(!srcdir.empty())
                        return srcdir;
        }
@@ -529,10 +557,10 @@ FS::Path Builder::get_package_location(const string &name)
        {
                for(list<FS::Path>::const_iterator i=pkg_path.begin(); i!=pkg_path.end(); ++i)
                {
-                       list<string> files=list_files(*i);
+                       list<string> files = list_files(*i);
                        for(list<string>::const_iterator j=files.begin(); j!=files.end(); ++j)
                        {
-                               FS::Path full=*i / *j;
+                               FS::Path full = *i / *j;
                                if(FS::exists(full/"Build"))
                                        pkg_dirs.push_back(full);
                        }
@@ -541,11 +569,11 @@ FS::Path Builder::get_package_location(const string &name)
                        IO::print("%d packages found in path\n", pkg_dirs.size());
        }
 
-       bool msp=!name.compare(0, 3, "msp");
+       bool msp = !name.compare(0, 3, "msp");
        for(list<FS::Path>::const_iterator i=pkg_dirs.begin(); i!=pkg_dirs.end(); ++i)
        {
-               string base=basename(*i);
-               unsigned dash=base.rfind('-');
+               string base = basename(*i);
+               unsigned dash = base.rfind('-');
 
                if(!base.compare(0, dash, name))
                        return *i;
@@ -558,52 +586,50 @@ FS::Path Builder::get_package_location(const string &name)
 
 int Builder::load_build_file(const FS::Path &fn)
 {
-       try
-       {
-               IO::BufferedFile in(fn.str());
+       if(!FS::exists(fn))
+               return -1;
 
-               if(verbose>=3)
-                       IO::print("Reading %s\n", fn);
+       IO::BufferedFile in(fn.str());
 
-               DataFile::Parser parser(in, fn.str());
-               Loader loader(*this, fn.subpath(0, fn.size()-1));
-               loader.load(parser);
-       }
-       catch(const IO::FileNotFound &)
-       {
-               return -1;
-       }
+       if(verbose>=3)
+               IO::print("Reading %s\n", fn);
+
+       DataFile::Parser parser(in, fn.str());
+       Loader loader(*this, fn.subpath(0, fn.size()-1));
+       loader.load(parser);
 
        return 0;
 }
 
 int Builder::create_targets()
 {
-       Target *world=new VirtualTarget(*this, "world");
+       Target *world = new VirtualTarget(*this, "world");
 
-       Target *def_tgt=new VirtualTarget(*this, "default");
+       Target *def_tgt = new VirtualTarget(*this, "default");
        world->add_depend(def_tgt);
 
-       Target *install=new VirtualTarget(*this, "install");
+       Target *install = new VirtualTarget(*this, "install");
        world->add_depend(install);
 
-       Target *tarballs=new VirtualTarget(*this, "tarballs");
+       Target *tarballs = new VirtualTarget(*this, "tarballs");
        world->add_depend(tarballs);
 
-       PackageList all_reqs=main_pkg->collect_requires();
-       for(PackageList::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
+       for(PackageMap::const_iterator i=packages.begin(); i!=packages.end(); ++i)
        {
-               SourcePackage *spkg=dynamic_cast<SourcePackage *>(*i);
+               if(!i->second || !i->second->is_configured())
+                       continue;
+
+               SourcePackage *spkg = dynamic_cast<SourcePackage *>(i->second);
                if(!spkg)
                        continue;
 
-               const ComponentList &components=spkg->get_components();
+               const ComponentList &components = spkg->get_components();
                for(ComponentList::const_iterator j=components.begin(); j!=components.end(); ++j)
                        j->create_targets();
 
                if(spkg->get_install_flags()&(SourcePackage::LIB|SourcePackage::INCLUDE))
                {
-                       PkgConfig *pc=new PkgConfig(*this, *spkg);
+                       PkgConfig *pc = new PkgConfig(*this, *spkg);
                        install->add_depend(new Install(*this, *spkg, *pc));
                }
        }
@@ -611,7 +637,7 @@ int Builder::create_targets()
        // Find dependencies until no new targets are created
        while(!new_tgts.empty())
        {
-               Target *tgt=new_tgts.front();
+               Target *tgt = new_tgts.front();
                new_tgts.erase(new_tgts.begin());
                tgt->find_depends();
                if(!tgt->get_depends_ready())
@@ -621,7 +647,7 @@ int Builder::create_targets()
        // Apply what-ifs
        for(StringList::iterator i=what_if.begin(); i!=what_if.end(); ++i)
        {
-               Target *tgt=get_target((cwd/ *i).str());
+               FileTarget *tgt = get_target_by_path(cwd/ *i);
                if(!tgt)
                {
                        IO::print(IO::cerr, "Unknown what-if target %s\n", *i);
@@ -631,27 +657,27 @@ int Builder::create_targets()
        }
 
        // Make the cmdline target depend on all targets mentioned on the command line
-       Target *cmdline=new VirtualTarget(*this, "cmdline");
-       bool build_world=false;
+       Target *cmdline = new VirtualTarget(*this, "cmdline");
        for(list<string>::iterator i=cmdline_targets.begin(); i!=cmdline_targets.end(); ++i)
        {
-               Target *tgt=get_target(*i);
+               Target *tgt = get_target(*i);
                if(!tgt)
-                       tgt=get_target((cwd/ *i).str());
+                       tgt = get_target_by_path(*i);
+               if(!tgt)
+                       tgt = get_target_by_path(cwd/ *i);
                if(!tgt)
                {
                        IO::print("I don't know anything about %s\n", *i);
                        return -1;
                }
-               if(tgt==world)
-                       build_world=true;
+
                cmdline->add_depend(tgt);
        }
 
        cmdline->prepare();
 
        for(PackageMap::iterator i=packages.begin(); i!=packages.end(); ++i)
-               if(SourcePackage *spkg=dynamic_cast<SourcePackage *>(i->second))
+               if(SourcePackage *spkg = dynamic_cast<SourcePackage *>(i->second))
                        spkg->get_deps_cache().save();
 
        return 0;
@@ -659,12 +685,12 @@ int Builder::create_targets()
 
 Target *Builder::get_header(const FS::Path &fn)
 {
-       Target *tgt=get_target(fn.str());
+       Target *tgt = get_target_by_path(fn);
        if(tgt) return tgt;
 
        if(FS::is_reg(fn))
        {
-               tgt=new SystemHeader(*this, fn.str());
+               tgt = new SystemHeader(*this, fn.str());
                return tgt;
        }
        return 0;
@@ -677,7 +703,8 @@ Target *Builder::get_library(const string &lib, const FS::Path &path, LibMode mo
 
        if(mode!=ALL_STATIC)
        {
-               if(current_arch->get_name()=="win32")
+               // XXX Should probably let the Architecture populate the list
+               if(current_arch->get_system()=="windows")
                {
                        candidates.push_back("lib"+lib+".dll");
                        candidates.push_back(lib+".dll");
@@ -689,19 +716,17 @@ Target *Builder::get_library(const string &lib, const FS::Path &path, LibMode mo
        /* Static libraries are always considered, since sometimes shared versions
        may not be available */
        candidates.push_back("lib"+lib+".a");
-       if(current_arch->get_name()=="win32")
+       if(current_arch->get_system()=="windows")
                candidates.push_back("lib"+lib+".dll.a");
 
        for(StringList::iterator i=candidates.begin(); i!=candidates.end(); ++i)
        {
-               string full=(path/ *i).str();
-               Target *tgt=get_target(full);
+               FS::Path full = path/ *i;
+               Target *tgt = get_target_by_path(full);
 
                if(tgt)
                {
-                       Target *real_tgt=tgt;
-                       if(Install *inst=dynamic_cast<Install *>(tgt))
-                               real_tgt=&inst->get_source();
+                       Target *real_tgt = tgt->get_real_target();
 
                        /* Ignore dynamic libraries from local packages unless library mode is
                        DYNAMIC */
@@ -712,7 +737,7 @@ Target *Builder::get_library(const string &lib, const FS::Path &path, LibMode mo
                }
                else if(FS::is_reg(full))
                {
-                       tgt=new SystemLibrary(*this, full);
+                       tgt = new SystemLibrary(*this, full.str());
                        return tgt;
                }
        }
@@ -722,9 +747,9 @@ Target *Builder::get_library(const string &lib, const FS::Path &path, LibMode mo
 
 int Builder::do_build()
 {
-       Target *cmdline=get_target("cmdline");
+       Target *cmdline = get_target("cmdline");
 
-       unsigned total=0;
+       unsigned total = 0;
        for(map<string, Target *>::const_iterator i=targets.begin(); i!=targets.end(); ++i)
                if(i->second->is_buildable() && i->second->get_rebuild())
                        ++total;
@@ -737,46 +762,48 @@ int Builder::do_build()
        if(verbose>=1)
                IO::print("Will build %d target%s\n", total, (total!=1 ? "s" : ""));
 
-       vector<Action *> actions;
+       vector<Task *> tasks;
 
-       unsigned count=0;
+       unsigned count = 0;
 
-       bool fail=false;
-       bool finish=false;
+       bool fail = false;
+       bool finish = false;
 
        while(!finish)
        {
-               if(actions.size()<jobs && !fail)
+               if(tasks.size()<jobs && !fail)
                {
-                       Target *tgt=cmdline->get_buildable_target();
+                       Target *tgt = cmdline->get_buildable_target();
                        if(tgt)
                        {
-                               Action *action=tgt->build();
-                               if(action)
-                                       actions.push_back(action);
+                               if(tgt->get_tool())
+                                       IO::print("[%-10s] [%-4s] %s\n", tgt->get_package()->get_name(), tgt->get_tool()->get_tag(), tgt->get_name());
+                               Task *task = tgt->build();
+                               if(task)
+                                       tasks.push_back(task);
 
                                if(show_progress)
                                        IO::print("%d of %d target%s built\033[1G", count, total, (total!=1 ? "s" : ""));
                        }
-                       else if(actions.empty())
-                               finish=true;
+                       else if(tasks.empty())
+                               finish = true;
                }
                else
                        Time::sleep(10*Time::msec);
 
-               for(unsigned i=0; i<actions.size();)
+               for(unsigned i=0; i<tasks.size();)
                {
-                       int status=actions[i]->check();
-                       if(status>=0)
+                       Task::Status status = tasks[i]->check();
+                       if(status!=Task::RUNNING)
                        {
                                ++count;
 
-                               delete actions[i];
-                               actions.erase(actions.begin()+i);
-                               if(status>0)
-                                       fail=true;
-                               if(actions.empty() && fail)
-                                       finish=true;
+                               delete tasks[i];
+                               tasks.erase(tasks.begin()+i);
+                               if(status==Task::ERROR)
+                                       fail = true;
+                               if(tasks.empty() && fail)
+                                       finish = true;
                        }
                        else
                                ++i;
@@ -803,36 +830,32 @@ int Builder::do_clean()
 
        while(!queue.empty())
        {
-               Target *tgt=queue.front();
+               Target *tgt = queue.front();
                queue.erase(queue.begin());
 
                if(tgt->is_buildable() && (tgt->get_package()==main_pkg || clean>=2))
                        clean_tgts.insert(tgt);
 
-               const TargetList &deps=tgt->get_depends();
+               const TargetList &deps = tgt->get_depends();
                for(TargetList::const_iterator i=deps.begin(); i!=deps.end(); ++i)
                        if(!clean_tgts.count(*i))
                                queue.push_back(*i);
        }
 
        for(set<Target *>::iterator i=clean_tgts.begin(); i!=clean_tgts.end(); ++i)
-               if(FileTarget *ft=dynamic_cast<FileTarget *>(*i))
-               {
-                       Action *action=new Unlink(*this, *ft);
-                       while(action->check()<0) ;
-                       delete action;
-               }
+               if(FileTarget *ft = dynamic_cast<FileTarget *>(*i))
+                       unlink(ft->get_path());
 
        return 0;
 }
 
 void Builder::package_help()
 {
-       const Config &config=main_pkg->get_config();
-       const Config::OptionMap &options=config.get_options();
+       const Config &config = main_pkg->get_config();
+       const Config::OptionMap &options = config.get_options();
 
        IO::print("Required packages:\n  ");
-       const PackageList &requires=main_pkg->get_requires();
+       const PackageList &requires = main_pkg->get_requires();
        for(PackageList::const_iterator i=requires.begin(); i!=requires.end(); ++i)
        {
                if(i!=requires.begin())
@@ -842,7 +865,7 @@ void Builder::package_help()
        IO::print("\n\nPackage configuration:\n");
        for(Config::OptionMap::const_iterator i=options.begin(); i!=options.end(); ++i)
        {
-               const Config::Option &opt=i->second;
+               const Config::Option &opt = i->second;
                IO::print("  %s: %s (%s)", opt.name, opt.descr, opt.value);
                if(opt.value!=opt.defv)
                        IO::print(" [%s]", opt.defv);
@@ -850,7 +873,6 @@ void Builder::package_help()
        }
 }
 
-Application::RegApp<Builder> Builder::reg;
 string Builder::usagemsg;
 string Builder::helpmsg;
 
@@ -859,26 +881,24 @@ Builder::Loader::Loader(Builder &b, const FS::Path &s):
        bld(b),
        src(s)
 {
-       add("architecture", &Loader::architecture);
        add("binary_package", &Loader::binpkg);
+       add("cross_prefix", &Loader::cross_prefix);
        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);
+       BinaryPackage *pkg = new BinaryPackage(bld, n);
        load_sub(*pkg);
        bld.packages.insert(PackageMap::value_type(n, pkg));
 }
 
+void Builder::Loader::cross_prefix(const string &a, const string &p)
+{
+       bld.cross_prefixes[a] = p;
+}
+
 void Builder::Loader::profile(const string &n)
 {
        StringMap prf;
@@ -889,9 +909,9 @@ void Builder::Loader::profile(const string &n)
 
 void Builder::Loader::package(const string &n)
 {
-       SourcePackage *pkg=new SourcePackage(bld, n, src);
+       SourcePackage *pkg = new SourcePackage(bld, n, src);
        if(!bld.main_pkg)
-               bld.main_pkg=pkg;
+               bld.main_pkg = pkg;
 
        load_sub(*pkg);
        bld.packages.insert(PackageMap::value_type(n, pkg));