[Mesa-dev] [PATCH] android: fix outdir for gen_enum_to_str files
Eric Engestrom
eric.engestrom at imgtec.com
Fri Mar 3 13:26:39 UTC 2017
On Friday, 2017-03-03 12:52:56 +0200, Tapani Pälli wrote:
> when files are being generated the value of $intermediates var content can be
> completely random, this makes sure that outdir is the wanted one.
>
> Fixes: 3f2cb699 ("android: vulkan: add support for libmesa_vulkan_util")
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
This can probably be applied to a lot of other places; even though it
might not be fixing bugs there, it's still cleaner to reuse the var than
copying the path.
> ---
> src/vulkan/Android.mk | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/vulkan/Android.mk b/src/vulkan/Android.mk
> index 9f71d8f..7653f34 100644
> --- a/src/vulkan/Android.mk
> +++ b/src/vulkan/Android.mk
> @@ -1,4 +1,5 @@
> # Copyright © 2017 Mauro Rossi <issor.oruam at gmail.com>
> +# Copyright © 2017 Intel Corporation
> #
> # Permission is hereby granted, free of charge, to any person obtaining a
> # copy of this software and associated documentation files (the "Software"),
> @@ -45,7 +46,7 @@ vulkan_api_xml = $(MESA_TOP)/src/vulkan/registry/vk.xml
> $(LOCAL_GENERATED_SOURCES): $(MESA_TOP)/src/vulkan/util/gen_enum_to_str.py $(vulkan_api_xml)
> @echo "target Generated: $(PRIVATE_MODULE) <= $(notdir $(@))"
> @mkdir -p $(dir $@)
> - $(hide) $(MESA_PYTHON2) $(MESA_TOP)/src/vulkan/util/gen_enum_to_str.py --xml $(vulkan_api_xml) --outdir $(intermediates)/util
> + $(hide) $(MESA_PYTHON2) $(MESA_TOP)/src/vulkan/util/gen_enum_to_str.py --xml $(vulkan_api_xml) --outdir $(dir $@)
>
> LOCAL_EXPORT_C_INCLUDE_DIRS := \
> $(intermediates)
> --
> 2.9.3
>
More information about the mesa-dev
mailing list