split .so generation into compile and link stage

1 files changed, 80 insertions, 34 deletions

diff --git a/lib/component_factory.cc b/lib/component_factory.cc
index bcf19f2..f8666dc 100644
--- a/lib/component_factory.cc
+++ b/lib/component_factory.cc
@@ -17,12 +17,13 @@
  #include "sitecing/sitecing_exception.h"
 #endif
  
 namespace sitecing {
 
     static const char *pp_targets[] = { ".cc", ".h", ".imports", ".classname", ".baseclassname", ".ancestors" };
+    static const char *cc_targets[] = { ".o", ".d" };
 
     component_factory::component_factory(configuration& c)
 	: config(c),
     root_source(normalize_path(c.root_source,strip_trailing_slash)+'/'),
     root_intermediate(normalize_path(c.root_intermediate,strip_trailing_slash)+'/'),
     root_so(normalize_path(c.root_so,strip_trailing_slash)+'/') {
@@ -38,25 +39,13 @@ namespace sitecing {
 	}catch(utility_no_affix& una) {
 	}
 	// .so binaries
 	try {
 	    string noso = strip_suffix(dp,".so");
 	    string noro = strip_prefix(noso,root_so);
-	    deps.push_back(root_intermediate+noro+".cc");
-	    config_options *co_cpp_deps = config.lookup_config(noro,config_options::flag_cpp_deps);
-	    if( (!co_cpp_deps) || co_cpp_deps->cpp_deps) {
-		ifstream df((root_intermediate+noro+".d").c_str(),ios::in);
-		if(df.good()) {
-		    string str;
-		    while(!df.eof()) {
-			df >> str;
-			if(str.find_first_of("\\:")==string::npos)
-			    deps.push_back(combine_path(config.root_source+noro,str));
-		    }
-		}
-	    }
+	    deps.push_back(root_intermediate+noro+".o");
 	    config_options *co_so_deps = config.lookup_config(noro,config_options::flag_so_deps);
 	    if(co_so_deps) {
 		for(list<string>::const_iterator i=co_so_deps->so_deps.begin();i!=co_so_deps->so_deps.end();++i)
 		    deps.push_back(*i);
 	    }
 	    return;
@@ -95,12 +84,35 @@ namespace sitecing {
 		}
 		return;
 	    }catch(utility_no_affix& una) {
 		// do nothing. must be a cpp dependency.
 	    }
 	}
+	// compiler targets
+	for(int cct=0;cct<sizeof(cc_targets)/sizeof(*cc_targets);cct++) {
+	    try {
+		string nos = strip_suffix(dp,cc_targets[cct]);
+		string noro = strip_prefix(nos,root_intermediate);
+		deps.push_back(root_intermediate+noro+".cc");
+		config_options *co_cpp_deps = config.lookup_config(noro,config_options::flag_cpp_deps);
+		if( (!co_cpp_deps) || co_cpp_deps->cpp_deps) {
+		    ifstream df((root_intermediate+noro+".d").c_str(),ios::in);
+		    if(df.good()) {
+			string str;
+			while(!df.eof()) {
+			    df >> str;
+			    if(str.find_first_of("\\:")==string::npos)
+				deps.push_back(combine_path(config.root_source+noro,str));
+			}
+		    }
+		}
+		// XXX: extra deps like IntermediateDeps?
+	    }catch(utility_no_affix& una) {
+		// do nothing.
+	    }
+	}
     }
 
     bool component_factory::is_uptodate(const string& dst,file_list_t *deps) {
 	string dp = normalize_path(dst,strip_trailing_slash);
 	// XXX: or just compare it off, instead of throwing things around.
 	try {
@@ -126,60 +138,94 @@ namespace sitecing {
 	}catch(utility_no_prefix& unp) {
 	}
 	// .so files
 	try {
 	    string noso = strip_suffix(dp,".so");
 	    string noro = strip_prefix(noso,root_so);
-	    string cc = root_intermediate+noro+".cc";
-	    if(access(cc.c_str(),R_OK))
-		throw konforka::exception(CODEPOINT,string("can't access preprocessed component code (")+cc+")");
+	    string o = root_intermediate+noro+".o";
+	    if(access(o.c_str(),R_OK))
+		throw konforka::exception(CODEPOINT,string("can't access compiled component code (")+o+")");
 	    make_path(dir_name(root_so+noro),0755);
-	    string pwd = dir_name(root_source+noro);
-	    auto_chdir dir_changer(pwd);
-	    file_lock lock_source(root_intermediate+noro+".lock");
+	    file_lock lock_cc(root_intermediate+noro+".o.lock");
 	    file_lock lock_so(root_so+noro+".so.lock");
-	    int stdO = open((root_intermediate+noro+".stdout").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
+	    int stdO = open((root_intermediate+noro+".ld.stdout").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
 	    if(stdO<0)
-		throw konforka::exception(CODEPOINT,"failed to open/create compiler stdout");
-	    int stdE = open((root_intermediate+noro+".stderr").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
+		throw konforka::exception(CODEPOINT,"failed to open/create linker stdout");
+	    int stdE = open((root_intermediate+noro+".ld.stderr").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
 	    if(stdE<0) {
 		close(stdO);
-		throw konforka::exception(CODEPOINT,"failed to open/create compiler's stderr");
+		throw konforka::exception(CODEPOINT,"failed to open/create linker stderr");
 	    }
 	    list<string> args;
-	    config_options *co_cpp_flags = config.lookup_config(noro,config_options::flag_cpp_flags);
-	    if(co_cpp_flags) {
-		args.insert(args.end(),co_cpp_flags->cpp_flags.begin(),co_cpp_flags->cpp_flags.end());
-	    }
 	    config_options *co_ld_flags = config.lookup_config(noro,config_options::flag_ld_flags);
 	    if(co_ld_flags) {
 		args.insert(args.end(),co_ld_flags->ld_flags.begin(),co_ld_flags->ld_flags.end());
 	    }
-	    // TODO: maybe move it to separare config option like CoreCPPFLags?
-	    args.push_back("-I"+root_intermediate);
-	    args.push_back("-I"+root_source);
-	    args.push_back("-MD"); args.push_back("-MF"); args.push_back(root_intermediate+noro+".d");
 	    args.push_back("-shared");
-	    args.push_back("-o"); args.push_back(dp);
-	    args.push_back(cc);
+	    args.push_back(o);
 	    file_list_t ancestors;
 	    get_ancestors(noro,ancestors);
 	    for(file_list_t::const_iterator i=ancestors.begin();i!=ancestors.end();++i) {
 		string aso=root_so+*i+".so";
 		make(aso);
 		args.push_back(aso);
 	    }
+	    args.push_back("-o"); args.push_back(dp);
 	    // TODO: "g++" configurable
 	    int rv = execute("g++",args,stdO,stdE);
 	    if(! (WIFEXITED(rv) && !WEXITSTATUS(rv)) )
-		throw compile_error(CODEPOINT,"failed to compile component",noro);
+		// TODO:TODO: linker_error
+		throw compile_error(CODEPOINT,"failed to link component",noro);
 	    return;
 	}catch(utility_no_prefix& unp) {
 	    throw konforka::exception(CODEPOINT,"component is outside of component root");
 	}catch(utility_no_suffix& uns) {
 	}
+	// compiler targets
+	for(int cct=0;cct<sizeof(cc_targets)/sizeof(*cc_targets);cct++) {
+	    try {
+		string nos = strip_suffix(dp,cc_targets[cct]);
+		string noro = strip_prefix(nos,root_intermediate);
+		string cc = root_intermediate+noro+".cc";
+		string o = root_intermediate+noro+".o";
+		if(access(cc.c_str(),R_OK))
+		    throw konforka::exception(CODEPOINT,string("can't access preprocessed component code (")+cc+")");
+		make_path(dir_name(cc),0755);
+		string pwd = dir_name(root_source+noro);
+		auto_chdir dir_changer(pwd);
+		file_lock lock_source(root_intermediate+noro+".lock");
+		file_lock lock_cc(root_intermediate+noro+".o.lock");
+		int stdO = open((root_intermediate+noro+".stdout").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
+		if(stdO<0)
+		    throw konforka::exception(CODEPOINT,"failed to open/create compiler stdout");
+		int stdE = open((root_intermediate+noro+".stderr").c_str(),O_CREAT|O_TRUNC|O_WRONLY,0664);
+		if(stdE<0) {
+		    close(stdO);
+		    throw konforka::exception(CODEPOINT,"failed to open/create compiler's stderr");
+		}
+		list<string> args;
+		config_options *co_cpp_flags = config.lookup_config(noro,config_options::flag_cpp_flags);
+		if(co_cpp_flags) {
+		    args.insert(args.end(),co_cpp_flags->cpp_flags.begin(),co_cpp_flags->cpp_flags.end());
+		}
+		// TODO: maybe move it to separare config option like CoreCPPFLags?
+		args.push_back("-I"+root_intermediate);
+		args.push_back("-I"+root_source);
+		args.push_back("-MD"); args.push_back("-MF"); args.push_back(root_intermediate+noro+".d");
+		args.push_back("-c");
+		args.push_back(cc);
+		args.push_back("-o"); args.push_back(o);
+		// TODO: "g++" configurable
+		int rv = execute("g++",args,stdO,stdE);
+		if(! (WIFEXITED(rv) && !WEXITSTATUS(rv)) )
+		    throw compile_error(CODEPOINT,"failed to compile component",noro);
+		return;
+	    }catch(utility_no_affix& una) {
+		// do nothing, not a compiler target
+	    }
+	}
 	// preprocessor targets
 	for(int ppt=0;ppt<sizeof(pp_targets)/sizeof(*pp_targets);ppt++) {
 	    try {
 		string nos = strip_suffix(dp,pp_targets[ppt]);
 		string noro = strip_prefix(nos,root_intermediate);
 		string src = root_source+noro;