On Wed, Feb 24, 2016 at 7:31 PM, Emil Velikov <[email protected]> wrote: > Hi Rob, > > On 24 February 2016 at 18:56, Rob Herring <[email protected]> wrote: >> With the Android build system changes to ninja/kati, the use of >> .SECONDEXPANSION is no longer supported. Fix this by avoiding rule specific >> variables and using $(transform-generated-source). >> >> Cc: Emil Velikov <[email protected]> >> Signed-off-by: Rob Herring <[email protected]> >> --- >> src/mesa/drivers/dri/common/Android.mk | 22 +++++++++++++--------- >> 1 file changed, 13 insertions(+), 9 deletions(-) >> >> diff --git a/src/mesa/drivers/dri/common/Android.mk >> b/src/mesa/drivers/dri/common/Android.mk >> index f1a7330..8a21e63 100644 >> --- a/src/mesa/drivers/dri/common/Android.mk >> +++ b/src/mesa/drivers/dri/common/Android.mk >> @@ -74,20 +74,24 @@ $(intermediates)/xmlpool/%.po: >> $(LOCAL_PATH)/xmlpool/%.po $(POT) >> sed -i -e 's/charset=.*\\n/charset=UTF-8\\n/' $@; \ >> fi >> >> -$(intermediates)/xmlpool/%/LC_MESSAGES/options.mo: >> $(intermediates)/xmlpool/%.po >> +PRIVATE_SCRIPT := $(LOCAL_PATH)/xmlpool/gen_xmlpool.py >> +PRIVATE_LOCALEDIR := $(intermediates)/xmlpool >> +PRIVATE_TEMPLATE_HEADER := $(LOCAL_PATH)/xmlpool/t_options.h >> +PRIVATE_MO_FILES := >> $(MESA_DRI_OPTIONS_LANGS:%=$(intermediates)/xmlpool/%/LC_MESSAGES/options.mo) >> + >> +LOCAL_GENERATED_SOURCES += $(PRIVATE_MO_FILES) >> + >> +$(PRIVATE_MO_FILES): $(intermediates)/xmlpool/%/LC_MESSAGES/options.mo: >> $(intermediates)/xmlpool/%.po >> mkdir -p $(dir $@) >> msgfmt -o $@ $< >> > I sort of mentioned it before, but never got around to testing it: > Do you foresee any issues if we move the generation rules into a > common place, and reuse them in both builds ?
It would work, but I think some of the rules could actually be dropped and use common Android rules instead. Though, that may not work in earlier Android versions and trying to test these changes across master, M, L, K, etc. is a PIA. I've been wondering if there is a better way to handle Android builds other that scattering Android.mk files all over the tree. Rob _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
