[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