- if(!needs_rebuild() && package && package->get_config().get_mtime()>mtime)
- mark_rebuild("Package options changed");
+ if(!needs_rebuild())
+ {
+ // Some side effects might not exist
+ for(Dependencies::iterator i=side_effects.begin(); (i!=side_effects.end() && !needs_rebuild()); ++i)
+ if((*i)->needs_rebuild())
+ mark_rebuild((*i)->get_name()+" needs rebuilding");
+ }
+
+ if(!needs_rebuild() && package)
+ {
+ if(package->get_config().get_mtime()>mtime)
+ mark_rebuild("Package options changed");
+
+ if(tool->get_executable())
+ {
+ string build_sig = create_build_signature();
+ if(package->get_cache().has_key(this, "build_sig"))
+ {
+ if(package->get_cache().get_value(this, "build_sig")!=build_sig)
+ mark_rebuild("Build signature changed");
+ }
+ }
+ }