Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(252)

Unified Diff: SConstruct

Issue 2848018: Adding dependencies for side effect files. (Closed) Base URL: http://src.chromium.org/git/update_engine.git
Patch Set: Created 10 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: SConstruct
diff --git a/SConstruct b/SConstruct
index 53004abca45d7f54bb91a0914a0653d098dbc703..13faa8666aaa33e6881af087ed60d4c347c7f98c 100644
--- a/SConstruct
+++ b/SConstruct
@@ -173,6 +173,20 @@ env['BUILDERS']['ProtocolBuffer'] = proto_builder
env['BUILDERS']['DbusBindings'] = dbus_bindings_builder
env['BUILDERS']['GlibMarshal'] = glib_marshal_builder
+# Hack to fix dependencies from auto generated headers.
+# Some files indirectly included update_metatadata.pb.h
+# which is built as a side effect of building update_metadata.pb.cc
+env.Depends('bzip_extent_writer_unittest.cc', 'update_metadata.pb.cc');
+env.Depends('bzip_extent_writer.cc', 'update_metadata.pb.cc');
+
+# Some files indirectly include marshal.glibmarshal.h
+# which is built as a side effect of the .c file
+env.Depends('dbus_service.cc', 'marshal.glibmarshal.c');
+
+# Some files indirectly include update_engine.dbusserver.h
+# which is built as a side effect of the dbusclient.h file
+env.Depends('mock_http_fetcher.cc', 'update_engine.dbusclient.h');
+
# Fix issue with scons not passing pkg-config vars through the environment.
for key in Split('PKG_CONFIG_LIBDIR PKG_CONFIG_PATH'):
if os.environ.has_key(key):
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698