[vlc-commits] contribs: gcrypt: Make sure that a git repo is available during reconfigure
Martin Storsjö
git at videolan.org
Sat Jun 20 21:29:35 CEST 2020
vlc/vlc-3.0 | branch: master | Martin Storsjö <martin at martin.st> | Thu Dec 21 11:48:16 2017 +0200| [fa7f03627ac8986003902ba3c1b7fbcf0b6b1ffb] | committer: Martin Storsjö
contribs: gcrypt: Make sure that a git repo is available during reconfigure
Without this, when reconfiguring, BUILD_FILEVERSION will end up with
a trailing comma with no revision number, which means that
src/versioninfo.rc will have a syntax error on build.
To reproduce/understand, download the plain gcrypt tarball;
configure contains a line like this:
BUILD_FILEVERSION="${BUILD_FILEVERSION}45409"
After running configure, src/versioninfo.rc contains
"FILEVERSION 1,7,8,45409".
If running autoreconfigure with no enclosing git repo anywhere,
the same line in configure will end up as this:
BUILD_FILEVERSION="${BUILD_FILEVERSION}"
After configuring, src/versioninfo.rc will now contain
"FILEVERSION 1,7,8,", which results in syntax errors.
As long as an enclosing git repo is available somewhere (if
the surrounding vlc tree contains a .git directory), this isn't
an issue though.
Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>
(cherry picked from commit 8be62dbc7af4bb53437383d1cb01c7ef0fb36a09)
> http://git.videolan.org/gitweb.cgi/vlc/vlc-3.0.git/?a=commit;h=fa7f03627ac8986003902ba3c1b7fbcf0b6b1ffb
---
contrib/src/gcrypt/rules.mak | 3 +++
1 file changed, 3 insertions(+)
diff --git a/contrib/src/gcrypt/rules.mak b/contrib/src/gcrypt/rules.mak
index 3406b2cdb3..9e5e851530 100644
--- a/contrib/src/gcrypt/rules.mak
+++ b/contrib/src/gcrypt/rules.mak
@@ -74,6 +74,9 @@ endif
endif
.gcrypt: gcrypt
+ # Reconfiguring this requires a git repo to be available, to
+ # successfully produce a nonempty mym4_revision_dec.
+ git init && git commit --allow-empty -m "dummy commit"
$(RECONF)
cd $< && $(HOSTVARS) CFLAGS="$(CFLAGS) $(GCRYPT_EXTRA_CFLAGS)" ./configure $(HOSTCONF) $(GCRYPT_CONF)
cd $< && $(MAKE) install
More information about the vlc-commits
mailing list