On 14 August 2018 at 18:24, Eric Engestrom <eric.engest...@intel.com> wrote:
> Signed-off-by: Eric Engestrom <eric.engest...@intel.com>
> ---
> bin/git_sha1_gen.py | 2 +-
> src/gallium/drivers/svga/svga_screen.c | 6 +-----
> src/gallium/state_trackers/clover/api/device.cpp | 6 +-----
> src/gallium/state_trackers/clover/api/platform.cpp | 6 +-----
> src/git_sha1.h.in | 2 +-
> src/mesa/main/context.c | 6 +-----
> src/mesa/main/version.c | 6 +-----
> 7 files changed, 7 insertions(+), 27 deletions(-)
>
> diff --git a/bin/git_sha1_gen.py b/bin/git_sha1_gen.py
> index bc0279ccef16d42bb88d..bcc29de1c6218ab3af79 100755
> --- a/bin/git_sha1_gen.py
> +++ b/bin/git_sha1_gen.py
> @@ -53,4 +53,4 @@ def write_if_different(contents):
> new_sha1 = git_sha1_h_in.read().replace('@VCS_TAG@', git_sha1)
> write_if_different(new_sha1)
You're missing the leading " (" and trailing ")" in the above call.
Without those the output will change as below:
from: foo (git-12345)
to: foogit-12345
> else:
> - write_if_different('')
> + write_if_different('#define MESA_GIT_SHA1 ""')
> diff --git a/src/gallium/drivers/svga/svga_screen.c
> b/src/gallium/drivers/svga/svga_screen.c
> index b8f936ee96730d97c570..355aaa38f6ba5510f3fe 100644
> --- a/src/gallium/drivers/svga/svga_screen.c
> +++ b/src/gallium/drivers/svga/svga_screen.c
> @@ -889,11 +889,7 @@ init_logging(struct pipe_screen *screen)
> svga_host_log(host_log);
>
> util_snprintf(host_log, sizeof(host_log) - strlen(log_prefix),
> - "%s%s"
> -#ifdef MESA_GIT_SHA1
> - " (" MESA_GIT_SHA1 ")"
> -#endif
> - , log_prefix, PACKAGE_VERSION);
> + "%s%s%s", log_prefix, PACKAGE_VERSION, MESA_GIT_SHA1);
Wondering why we need the explicit %s modifier for PACKAGE_VERSION
(and MESA_GIT_SHA1) here.
Should be safe to drop and alight with rest of tree. But that for another patch.
-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
