<div dir="ltr">Hi Pekka and thanks for the review, <br><div><br>Hmm strange, wasn't what I observed when I submitted the patch, will look at that and let you know.<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
2014-08-15 15:33 GMT+02:00 Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Wed, 30 Jul 2014 00:28:52 +0200<br>
Manuel Bachmann <<a href="mailto:manuel.bachmann@open.eurogiciel.org">manuel.bachmann@open.eurogiciel.org</a>> wrote:<br>
<br>
> When building from a git repository, in Makefile.am, the<br>
> macro creating the "src/git-version.h" file will fail<br>
> if the last commit subject contains a special character<br>
> such as ". Escape characters so that the output string<br>
> is, at worst, empty.<br>
><br>
> Signed-off-by: Manuel Bachmann <<a href="mailto:manuel.bachmann@open.eurogiciel.org">manuel.bachmann@open.eurogiciel.org</a>><br>
> ---<br>
>  Makefile.am |    2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/Makefile.am b/Makefile.am<br>
> index 191dcc9..68f4d18 100644<br>
> --- a/Makefile.am<br>
> +++ b/Makefile.am<br>
> @@ -112,7 +112,7 @@ endif<br>
><br>
>  if HAVE_GIT_REPO<br>
>  src/git-version.h : $(top_srcdir)/.git/logs/HEAD<br>
> -     $(AM_V_GEN)echo "#define BUILD_ID \"$(shell git --git-dir=$(top_srcdir)/.git describe --always --dirty) $(shell git --git-dir=$(top_srcdir)/.git log -1 --format='%s (%ci)')\"" > $@<br>

> +     $(AM_V_GEN)echo "#define BUILD_ID \"${(shell git --git-dir=$(top_srcdir)/.git describe --always --dirty)} ${(shell git --git-dir=$(top_srcdir)/.git log -1 --format='%s (%ci)')}\"" > $@<br>

>  else<br>
>  src/git-version.h :<br>
>       $(AM_V_GEN)echo "#define BUILD_ID \"unknown (not built from git or tarball)\"" > $@<br>
<br>
</div></div>For me, this just makes the BUILD_ID always empty, even without any<br>
funny characters.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><font>Regards,<br>
<br>
<i><b>Manuel BACHMANN</b><br>
Tizen Project<br>
VANNES-FR</i><br>
</font></div>
</div>