On Wed, Feb 24, 2016 at 7:31 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > Hi Rob, > > On 24 February 2016 at 18:56, Rob Herring <r...@kernel.org> 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 <emil.veli...@collabora.com> >> Signed-off-by: Rob Herring <r...@kernel.org> >> --- >> 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 mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
