diff options
| author | Ying Wang <wangying@google.com> | 2012-11-14 15:57:07 -0800 |
|---|---|---|
| committer | Ying Wang <wangying@google.com> | 2012-11-14 16:00:45 -0800 |
| commit | c065da230762acd1cba7714827c2d588ab32a7fb (patch) | |
| tree | 807ba1306e6a7d88fb3a4a070b69911d14b069fb /core/definitions.mk | |
| parent | 4e9b723f694b4713bdc30e8019b82b43b1217bae (diff) | |
| download | build-c065da230762acd1cba7714827c2d588ab32a7fb.zip build-c065da230762acd1cba7714827c2d588ab32a7fb.tar.gz build-c065da230762acd1cba7714827c2d588ab32a7fb.tar.bz2 | |
Move check-api to definition.mk
1) Move check-api to definition.mk, so it can be used in other places.
2) Add additional dependency to the api-stub module to consolidate the
apicheck dependency. Previously you have to run 2 "make"s in a row and
only in the 2nd run apicheck is run, due to a gnu-make optimization.
Bug: 7540265
Change-Id: Ib24ed87d9330027a9f95ed7b16dd4002cc0e3198
Diffstat (limited to 'core/definitions.mk')
| -rw-r--r-- | core/definitions.mk | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/core/definitions.mk b/core/definitions.mk index 06db960..4f4e583 100644 --- a/core/definitions.mk +++ b/core/definitions.mk @@ -2064,6 +2064,29 @@ $(if $(_erm_new_modules),$(eval $(1) += $(_erm_new_modules))\ endef ########################################################### +## API Check +########################################################### + +# eval this to define a rule that runs apicheck. +# +# Args: +# $(1) target +# $(2) stable api file +# $(3) api file to be tested +# $(4) arguments for apicheck +# $(5) command to run if apicheck failed +# $(6) target dependent on this api check +# $(7) additional dependencies +define check-api +$(TARGET_OUT_COMMON_INTERMEDIATES)/PACKAGING/$(strip $(1))-timestamp: $(2) $(3) $(APICHECK) $(7) + @echo "Checking API:" $(1) + $(hide) ( $(APICHECK_COMMAND) $(4) $(2) $(3) || ( $(5) ; exit 38 ) ) + $(hide) mkdir -p $$(dir $$@) + $(hide) touch $$@ +$(6): $(TARGET_OUT_COMMON_INTERMEDIATES)/PACKAGING/$(strip $(1))-timestamp +endef + +########################################################### ## Other includes ########################################################### |