X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcomponent.cpp;h=4fddcef79de59a9ca0ca6cf05e8a5b71ec565a79;hb=1ed833343bc83b83c5f61cbfd74423bbba677a04;hp=98b6ff76590e1067709895ed1277d6535ed61e58;hpb=3938f8030b1f62802decce19777ce70fdafaff10;p=builder.git

diff --git a/source/component.cpp b/source/component.cpp
index 98b6ff7..4fddcef 100644
--- a/source/component.cpp
+++ b/source/component.cpp
@@ -1,3 +1,4 @@
+#include <deque>
 #include <msp/core/algorithm.h>
 #include <msp/fs/dir.h>
 #include <msp/fs/stat.h>
@@ -10,13 +11,6 @@
 using namespace std;
 using namespace Msp;
 
-Component::Component(SourcePackage &p, const string &n):
-	package(p),
-	name(n),
-	install(false),
-	deflt(true)
-{ }
-
 void Component::prepare()
 {
 	for(Package *r: requires)
@@ -30,18 +24,23 @@ void Component::create_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());
+	for(Package *r: direct_reqs)
+		final_build_info.update_from(r->get_exported_build_info(), BuildInfo::DEPENDENCY);
 
 	Package::Requirements all_reqs = direct_reqs;
-	for(Package *r: all_reqs)
+	deque<Package *> queue(direct_reqs.begin(), direct_reqs.end());
+	while(!queue.empty())
 	{
-		BuildInfo::UpdateLevel level = BuildInfo::CHAINED;
-		if(any_equals(direct_reqs, r))
-			level = BuildInfo::DEPENDENCY;
-		final_build_info.update_from(r->get_exported_build_info(), level);
-
-		for(Package *q: r->get_required_packages())
-			if(!any_equals(all_reqs, q))
-				all_reqs.push_back(q);
+		Package *req = queue.front();
+		queue.pop_front();
+
+		for(Package *r: req->get_required_packages())
+			if(!any_equals(all_reqs, r))
+			{
+				final_build_info.update_from(r->get_exported_build_info(), BuildInfo::CHAINED);
+				all_reqs.push_back(r);
+				queue.push_back(r);
+			}
 	}
 
 	final_build_info.update_from(package.get_build_info());
@@ -88,14 +87,15 @@ BuildInfo Component::get_build_info_for_path(const FS::Path &path) const
 	return binfo;
 }
 
-list<FS::Path> Component::collect_source_files() const
+vector<FS::Path> Component::collect_source_files() const
 {
-	list<FS::Path> files;
+	vector<FS::Path> files;
 	for(const FS::Path &p: sources)
 	{
 		if(FS::is_dir(p))
 		{
-			list<FS::Path> dirs;
+			vector<FS::Path> dirs;
+			dirs.reserve(1+overlays.size());
 			dirs.push_back(p);
 			for(const string &o: overlays)
 			{
@@ -106,7 +106,7 @@ list<FS::Path> Component::collect_source_files() const
 			set<string> overlay_files;
 			for(auto j=dirs.begin(); j!=dirs.end(); ++j)
 			{
-				package.get_builder().get_logger().log("files", format("Traversing %s", *j));
+				package.get_builder().get_logger().log("files", "Traversing %s", *j);
 				for(const string &f: list_files(*j))
 				{
 					if(j!=dirs.begin())