Hi Hans, On 2021-04-19 23:10:37 +0200, Hans Verkuil wrote: > On 19/04/2021 20:28, Niklas Söderlund wrote: > > If the v4l-utils repository is a git submodule the $(top_srcdir)/.git is > > a file and not a directory. Update the shell check to allow the test to > > pass in both cases. > > > > Signed-off-by: Niklas Söderlund <niklas.soderlund@xxxxxxxxxxxx> > > --- > > configure.ac | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/configure.ac b/configure.ac > > index f144a50d034fbda0..4ac4195cd6e4747e 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -588,7 +588,7 @@ fi > > CPPFLAGS="-I\$(top_srcdir)/lib/include -Wall -Wpointer-arith -D_GNU_SOURCE $CPPFLAGS" > > > > # Obtain git SHA of HEAD > > -AC_SUBST(GIT_SHA, ["-DGIT_SHA=\$(shell if test -d \$(top_srcdir)/.git ; then git -C \$(top_srcdir) rev-parse --short=12 HEAD ; fi)"]) > > +AC_SUBST(GIT_SHA, ["-DGIT_SHA=\$(shell if test -e \$(top_srcdir)/.git ; then git -C \$(top_srcdir) rev-parse --short=12 HEAD ; fi)"]) > > > > # Obtain git commit count of HEAD > > AC_SUBST(GIT_COMMIT_CNT, ["-DGIT_COMMIT_CNT=\$(shell if test -d \$(top_srcdir)/.git ; then printf '-'; git -C \$(top_srcdir) rev-list --count HEAD ; fi)"]) > > Isn't the same change needed here and in GIT_COMMIT_DATE as well? Wops, you are correct. I noticed the problem in an older branch where version.h was generated in different utilities and this is a bad conflict fix of that. Do you want me to resend? > > Regards, > > Hans > > > > -- Regards, Niklas Söderlund
