diff options
| author | Alex Deymo <deymo@chromium.org> | 2015-08-14 10:54:02 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-08-14 21:04:31 +0000 |
| commit | f7feb23fbecae9daf269bd9a5b6b3415b75aae08 (patch) | |
| tree | 8651e7a0a9c137747921e051a2f472f992aa59c8 /payload_generator/graph_utils_unittest.cc | |
| parent | 04ef2078a1c17733b363971394dacad5f5f5b524 (diff) | |
update_engine: libupdate_engine depends on dbus-proxies action.
The generated ninja file didn't have a dependency between the source
files using the generated dbus_proxies.h and dbus_mocks.h because the
included .gypi file defines an "action" and not a "rule", as in the
adaptor side.
This patch wraps the action generating the dbus proxies in its own
target, making the libupdate_engine target depend on it.
With this change, the generated .ninja files include a dependency
between obj/update_engine/libupdate_engine.actions_depends.stamp file
and obj/update_engine/update_engine-dbus-proxies.actions_rules_copies.stamp
forcing it to generate the headers before using them.
BUG=chromium:520824
TEST=rm /build/link/var/cache/portage/chromeos-base/update_engine/out/Default/gen/include/update_engine/dbus_*.h; \
FEATURES=test cros_workon_make update_engine --reconf # works
Change-Id: If66f76e105de446333ea2e24bf0c0ee7a55dc594
Reviewed-on: https://chromium-review.googlesource.com/293830
Commit-Queue: Alex Deymo <deymo@chromium.org>
Trybot-Ready: Alex Deymo <deymo@chromium.org>
Tested-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Diffstat (limited to 'payload_generator/graph_utils_unittest.cc')
0 files changed, 0 insertions, 0 deletions