]> git.tdb.fi Git - builder.git/blobdiff - source/component.cpp
Add a way for components to use libraries from the same package
[builder.git] / source / component.cpp
index 88fe71443bc2b4497c51a590a2a262b0fcbe6223..feb5094b27a071fa8823eabd4e09c2fae466a11d 100644 (file)
@@ -23,54 +23,60 @@ using namespace std;
 using namespace Msp;
 
 Component::Component(SourcePackage &p, Type t, const string &n):
-       pkg(p),
+       package(p),
        type(t),
        name(n),
        install(false),
        deflt(true)
 { }
 
-void Component::configure(const StringMap &opts, unsigned flag)
+void Component::prepare()
 {
-       for(StringList::iterator i=sources.begin(); i!=sources.end(); ++i)
-               *i = (pkg.get_source()/pkg.expand_string(*i)).str();
-
-       for(PackageList::const_iterator i=requires.begin(); i!=requires.end(); ++i)
-               (*i)->configure(opts, flag&2);
+       for(Package::Requirements::const_iterator i=requires.begin(); i!=requires.end(); ++i)
+               (*i)->prepare();
 }
 
 void Component::create_build_info()
 {
-       const PackageList &pkg_reqs = pkg.get_requires();
-       PackageList direct_reqs = requires;
+       BuildInfo final_build_info;
+
+       const Package::Requirements &pkg_reqs = package.get_required_packages();
+       Package::Requirements direct_reqs = requires;
        direct_reqs.insert(direct_reqs.end(), pkg_reqs.begin(), pkg_reqs.end());
 
-       PackageList all_reqs = direct_reqs;
-       for(PackageList::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
+       Package::Requirements all_reqs = direct_reqs;
+       for(Package::Requirements::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
        {
                BuildInfo::UpdateLevel level = BuildInfo::CHAINED;
                if(find(direct_reqs.begin(), direct_reqs.end(), *i)!=direct_reqs.end())
                        level = BuildInfo::DEPENDENCY;
-               build_info.update_from((*i)->get_exported_binfo(), level);
+               final_build_info.update_from((*i)->get_exported_build_info(), level);
 
-               const PackageList &reqs = (*i)->get_requires();
-               for(PackageList::const_iterator j=reqs.begin(); j!=reqs.end(); ++j)
+               const Package::Requirements &reqs = (*i)->get_required_packages();
+               for(Package::Requirements::const_iterator j=reqs.begin(); j!=reqs.end(); ++j)
                        if(find(all_reqs.begin(), all_reqs.end(), *j)==all_reqs.end())
                                all_reqs.push_back(*j);
        }
 
-       build_info.update_from(pkg.get_build_info());
+       final_build_info.update_from(package.get_build_info());
 
        for(BuildInfo::PathList::iterator i=build_info.incpath.begin(); i!=build_info.incpath.end(); ++i)
-               *i = (pkg.get_source() / *i).str();
+               *i = (package.get_source_directory() / *i).str();
        for(BuildInfo::PathList::iterator i=build_info.libpath.begin(); i!=build_info.libpath.end(); ++i)
-               *i = (pkg.get_source() / *i).str();
+               *i = (package.get_source_directory() / *i).str();
+
+       final_build_info.update_from(build_info);
+       build_info = final_build_info;
 
-       if(pkg.get_library_mode()<DYNAMIC)
+       if(type==LIBRARY || type==MODULE)
+               if(build_info.libmode<BuildInfo::DYNAMIC)
+                       build_info.libmode = BuildInfo::DYNAMIC;
+
+       if(build_info.libmode<BuildInfo::DYNAMIC)
        {
-               for(PackageList::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
+               for(Package::Requirements::iterator i=all_reqs.begin(); i!=all_reqs.end(); ++i)
                {
-                       const BuildInfo &ebi = (*i)->get_exported_binfo();
+                       const BuildInfo &ebi = (*i)->get_exported_build_info();
                        build_info.libpath.insert(build_info.libpath.end(), ebi.libpath.begin(), ebi.libpath.end());
                }
        }
@@ -78,62 +84,55 @@ void Component::create_build_info()
 
 void Component::create_targets() const
 {
-       Builder &builder = pkg.get_builder();
+       Builder &builder = package.get_builder();
        const Toolchain &toolchain = builder.get_toolchain();
-       Target *world = builder.get_target("world");
-       Target *def_tgt = builder.get_target("default");
 
-       PathList source_filenames = collect_source_files();
+       SourceList source_filenames = collect_source_files();
        list<Target *> inst_list;
 
        string inst_loc;
        if(type==TARBALL)
        {
-               //const Tool &tar = toolchain.get_tool("TAR");
-
-               string tarname = name;
-               if(name=="@src")
-               {
-                       tarname = pkg.get_name()+"-"+pkg.get_version();
-                       source_filenames.push_back(pkg.get_source()/"Build");
-               }
+               const Tool &tar = toolchain.get_tool("TAR");
 
                list<Target *> files;
-               for(PathList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
+               for(SourceList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
                {
                        FileTarget *file = builder.get_vfs().get_target(*i);
                        if(!file)
-                               file = new File(builder, *i);
+                               file = new File(builder, package, *i);
                        files.push_back(file);
                }
 
+               string tarname = name;
                if(name=="@src")
                {
+                       tarname = package.get_name()+"-"+package.get_version();
+                       files.insert(files.begin(), &package.get_build_file());
+
                        const Builder::TargetMap &targets = builder.get_targets();
                        for(Builder::TargetMap::const_iterator i=targets.begin(); i!=targets.end(); ++i)
-                               if(i->second->get_package()==&pkg && !i->second->is_buildable())
-                                       files.push_back(i->second);
+                               if(i->second->get_package()==&package && !i->second->is_buildable())
+                                       if(find(files.begin(), files.end(), i->second)==files.end())
+                                               files.push_back(i->second);
                }
 
-               /* XXX The source files don't have a package at the moment, so we can't
-               create the tarball target until things get fixed up a bit */
-               /*Target *result = tar.create_target(files, tarname);
+               Target *result = tar.create_target(files, tarname);
 
-               Target *tarballs_tgt = builder.get_target("tarballs");
-               tarballs_tgt->add_depend(*result);*/
+               builder.get_target("tarballs")->add_dependency(*result);
 
                return;
        }
        else if(type==INSTALL)
        {
                inst_loc = name;
-               for(PathList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
+               for(SourceList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
                {
                        FileTarget *ft;
                        if(Target *tgt = builder.get_vfs().get_target(*i))
                                ft = dynamic_cast<FileTarget *>(tgt);
                        else
-                               ft = new File(builder, pkg, *i);
+                               ft = new File(builder, package, *i);
                        inst_list.push_back(ft);
                }
        }
@@ -145,13 +144,10 @@ void Component::create_targets() const
                if(Target *tgt = builder.get_vfs().get_target(source_filenames.front()))
                        source = dynamic_cast<File *>(tgt);
                else
-                       source = new File(builder, pkg, source_filenames.front());
+                       source = new File(builder, package, source_filenames.front());
                Target *result = dcomp.create_target(*source);
 
-               if(&pkg==builder.get_main_package() && deflt)
-                       def_tgt->add_depend(*result);
-               else
-                       world->add_depend(*result);
+               builder.add_primary_target(*result);
                if(install)
                        inst_list.push_back(result);
        }
@@ -159,20 +155,23 @@ void Component::create_targets() const
        if(type==PROGRAM || type==LIBRARY || type==MODULE)
        {
                list<Target *> objs;
-               for(PathList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
+               for(SourceList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
                {
                        string ext = FS::extpart(FS::basename(*i));
                        const Tool *tool = toolchain.get_tool_for_suffix(ext, true);
                        if(tool)
                        {
                                Target *src = tool->create_source(*this, *i);
+                               if(!src)
+                                       continue;
+
                                if(tool->accepts_suffix(ext))
                                {
                                        Target *obj = tool->create_target(*src);
                                        objs.push_back(obj);
                                }
 
-                               if(type==LIBRARY && install && !dynamic_cast<FileTarget *>(src)->get_install_location().empty())
+                               if(type==LIBRARY && install && dynamic_cast<FileTarget *>(src)->is_installable())
                                        inst_list.push_back(src);
                        }
                }
@@ -191,12 +190,27 @@ void Component::create_targets() const
                else
                        results.push_back(linker.create_target(objs));
 
+               const Target::Dependencies &world_deps = builder.get_target("world")->get_dependencies();
+               for(UseList::const_iterator i=uses.begin(); i!=uses.end(); ++i)
+               {
+                       /* The world target depends on all primary targets; look for the
+                       static library belonging to the component.  This is a bit roundabout
+                       way but gets the job done. */
+                       bool found = false;
+                       for(Target::Dependencies::const_iterator j=world_deps.begin(); j!=world_deps.end(); ++j)
+                               if((*j)->get_component()==*i && dynamic_cast<StaticLibrary *>(*j))
+                               {
+                                       results.front()->add_dependency(**j);
+                                       found = true;
+                                       break;
+                               }
+                       if(!found)
+                               builder.problem(package.get_name(), format("Can't find static library %s for component %s", (*i)->get_name(), name));
+               }
+
                for(list<Target *>::const_iterator i=results.begin(); i!=results.end(); ++i)
                {
-                       if(&pkg==builder.get_main_package() && deflt)
-                               def_tgt->add_depend(**i);
-                       else
-                               world->add_depend(**i);
+                       builder.add_primary_target(**i);
                        if(install)
                                inst_list.push_back(*i);
                }
@@ -207,18 +221,19 @@ void Component::create_targets() const
        for(list<Target *>::const_iterator i=inst_list.begin(); i!=inst_list.end(); ++i)
        {
                Target *inst = copy.create_target(**i, inst_loc);
-               inst_tgt->add_depend(*inst);
+               inst_tgt->add_dependency(*inst);
        }
 }
 
-PathList Component::collect_source_files() const
+Component::SourceList Component::collect_source_files() const
 {
-       PathList files;
-       for(StringList::const_iterator i=sources.begin(); i!=sources.end(); ++i)
+       SourceList files;
+       for(SourceList::const_iterator i=sources.begin(); i!=sources.end(); ++i)
        {
                FS::Path path(*i);
                if(FS::is_dir(path))
                {
+                       package.get_builder().get_logger().log("files", format("Traversing %s", path));
                        list<string> sfiles = list_files(path);
                        for(list<string>::iterator j=sfiles.begin(); j!=sfiles.end(); ++j)
                                files.push_back(path / *j);
@@ -240,16 +255,17 @@ Component::Loader::Loader(Component &c):
        add("build_info",      &Loader::build_info);
        add("require",         &Loader::require);
        add("default",         &Component::deflt);
+       add("use",             &Loader::use);
 }
 
 void Component::Loader::source(const string &s)
 {
-       obj.sources.push_back(s);
+       obj.sources.push_back((obj.package.get_source_directory()/s).str());
 }
 
 void Component::Loader::require(const string &n)
 {
-       Package *req = obj.pkg.get_builder().get_package_manager().find_package(n);
+       Package *req = obj.package.get_builder().get_package_manager().find_package(n);
        if(req)
                obj.requires.push_back(req);
 }
@@ -261,5 +277,17 @@ void Component::Loader::build_info()
 
 void Component::Loader::install_map()
 {
-       load_sub(obj.install_map, obj.pkg.get_source());
+       load_sub(obj.install_map, obj.package.get_source_directory());
+}
+
+void Component::Loader::use(const string &n)
+{
+       const SourcePackage::ComponentList &components = obj.package.get_components();
+       for(SourcePackage::ComponentList::const_iterator i=components.begin(); i!=components.end(); ++i)
+               if(i->get_name()==n && i->get_type()==LIBRARY)
+               {
+                       obj.uses.push_back(&*i);
+                       return;
+               }
+       error(format("Unknown library component '%s'", n));
 }