On 2 July 2015 at 00:21, Damien Lespiau <damien.lespiau@xxxxxxxxx> wrote: > When developing, it's quite annoying that the version changes every > commit, causing the library to be rebuild and everything single binary everything → every > re-linked. > > Add a config option to skip that. > > I remember Ville asking for this "feature" as well. > > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > --- > configure.ac | 7 +++++++ > lib/Makefile.sources | 5 +++++ > 2 files changed, 12 insertions(+) > > diff --git a/configure.ac b/configure.ac > index 4208f00..caa3f50 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -212,6 +212,13 @@ if test "x$enable_debug" = xyes; then > AC_SUBST([DEBUG_CFLAGS]) > fi > > +# prevent relinking the world on every commit for developpers "developers" > +AC_ARG_ENABLE(skip-version, I wonder whether "git-hash" would be more obvious here, so "--disable-git-hash" would turn off the git hash in version output. > + AS_HELP_STRING([--enable-skip-version], > + [Do not use git hash in version]), > + [skip_version=$enableval], [skip_version=no]) > +AM_CONDITIONAL(SKIP_VERSION, [test "x$skip_version" = xyes]) > + > # ----------------------------------------------------------------------------- > > # To build multithread code, gcc uses -pthread, Solaris Studio cc uses -mt > diff --git a/lib/Makefile.sources b/lib/Makefile.sources > index f8a1b92..2148684 100644 > --- a/lib/Makefile.sources > +++ b/lib/Makefile.sources > @@ -60,6 +60,10 @@ libintel_tools_la_SOURCES = \ > > .PHONY: version.h.tmp > > +if SKIP_VERSION > +$(IGT_LIB_PATH)/version.h.tmp: > + @echo '#define IGT_GIT_SHA1 "SKIP"' >> $@ Perhaps just "git", so the version output appears as (for example) "1.11-git", rather than "1.11-SKIP". > +else > $(IGT_LIB_PATH)/version.h.tmp: > @touch $@ > @if test -d $(GPU_TOOLS_PATH)/.git; then \ > @@ -73,6 +77,7 @@ $(IGT_LIB_PATH)/version.h.tmp: > else \ > echo '#define IGT_GIT_SHA1 "NOT-GIT"' ; \ > fi >> $@ > +endif # SKIP_VERSION > > > $(IGT_LIB_PATH)/version.h: $(IGT_LIB_PATH)/version.h.tmp > -- > 2.1.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
