[Mesa-dev] [PATCH mesa] autotools: don't ship the git_sha1.h generated in git in the tarballs
Emil Velikov
emil.l.velikov at gmail.com
Tue Aug 21 13:09:31 UTC 2018
On 21 August 2018 at 13:04, Eric Engestrom <eric.engestrom at intel.com> wrote:
> This file is regenerated at build time anyway, so this would just get
> overwritten anyway. No reason to ship it in the tarball.
>
> Fixes: 44df06211cf2c301f6ef "autotools: include git_sha1.h in dist tarball"
> Fixes: 471f708ed6f4787813d0 "git_sha1: simplify logic"
> Cc: Juan A. Suarez Romero <jasuarez at igalia.com>
> Signed-off-by: Eric Engestrom <eric.engestrom at intel.com>
> ---
> I double-checked; as is, the git hash of the current HEAD would be
> stored in the tarball, and the file would get overwritten with an empty
> hash when mesa is built from the tarball; with this patch, the incorrect
> file is not shipped, and is still generated like normal at build time.
> ---
> src/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 412510f435b19a309532..c4fcd8a92488bf89d6a8 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -26,7 +26,7 @@ git_sha1.h:
>
> BUILT_SOURCES = git_sha1.h
> CLEANFILES = $(BUILT_SOURCES)
> -EXTRA_DIST = git_sha1.h meson.build
> +EXTRA_DIST = meson.build
>
Was going to mention that in Juan's patch but I was late.
Thanks Eric.
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
-Emil
More information about the mesa-dev
mailing list