[Fixed]version build commit reference n° is no longer updated in the build

Started by Jan Gruuthuse, August 16, 2016, 05:57:54 AM

Previous topic - Next topic

Jan Gruuthuse

The revision commit number is stuck on commit from 2 days ago:  295a8f3    [appImage] more fixes

I've rebuild this commit: twice on 16.04.1 and 14.04.5 with the same result.

mean


Jan Gruuthuse

git show
Quotecommit 4af7da548db2ec0aeeb2ed66b1db55ca0db0936b
Author: mean <fixounet@free.fr>
Date:   Mon Aug 15 17:22:08 2016 +0200

    [plugin] remove avsfilter for the moment, ppwrite is missing

diff --git a/avidemux_plugins/ADM_videoFilters6/CMakeLists.txt b/avidemux_plugins/ADM_videoFilters6/CMakeLists.txt
index 68dd839..b62ec2e 100644
--- a/avidemux_plugins/ADM_videoFilters6/CMakeLists.txt
+++ b/avidemux_plugins/ADM_videoFilters6/CMakeLists.txt
@@ -1,7 +1,7 @@
ADD_SUBDIRECTORY(addBorder)
ADD_SUBDIRECTORY(asharp)
ADD_SUBDIRECTORY(ass)
-ADD_SUBDIRECTORY(avsfilter)
+#ADD_SUBDIRECTORY(avsfilter)^M
ADD_SUBDIRECTORY(black)
ADD_SUBDIRECTORY(blackenBorder)
ADD_SUBDIRECTORY(changeFps)
or
git rev-parse HEAD
Quote4af7da548db2ec0aeeb2ed66b1db55ca0db0936b
could well be, took a while before I did notice the small issue.

eumagga0x2a

This is by design, isn't it? The git revision shown in the "About" dialog should be the latest revision which touched the particular module (in this case https://github.com/mean00/avidemux2/tree/master/avidemux/qt4), not the repository in general.

(An approach which restored the previous logic http://avidemux.org/smif/index.php/topic,16988.msg76162.html#msg76162 was rejected.)

Jan Gruuthuse

Can't we use the output generated by: git rev-parse HEAD
What produces this output:
Quote4af7da548db2ec0aeeb2ed66b1db55ca0db0936b
and just take the first 7 on the left of that output (
4af7da5) ? And redirect this where it can be used while we are building from git?

mean


eumagga0x2a

Oops, I must have misread your intentions then. In this case

diff --git a/cmake/admMainChecks.cmake b/cmake/admMainChecks.cmake
index 58f6291..2c8c604 100644
--- a/cmake/admMainChecks.cmake
+++ b/cmake/admMainChecks.cmake
@@ -131,7 +131,7 @@ MESSAGE(STATUS "****************")
IF(RELEASE OR AVIDEMUX_EXTERNAL_BUILD)
         SET( ADM_SUBVERSION 0)
ELSE()
-        admGetRevision( ${CMAKE_SOURCE_DIR} ADM_SUBVERSION)
+        admGetRevision( ${AVIDEMUX_TOP_SOURCE_DIR} ADM_SUBVERSION)
ENDIF()
MESSAGE("")
include(avidemuxVersion)


was right, just setting ${AVIDEMUX_TOP_SOURCE_DIR} from respective CMakeLists.txt was wrong.

The way suggested by Jan would be location independent.

mean

It was actually  unclear  in my head, because the aim was to do 2 things at the same time

1- exporting the top source dir is not ok, the module should be buildable independently
2- having a per component tag is both ok and not ok. It should be a "common" marker from packages build from the same source tree

Until now, the solutions failed 1/ or 2/

the rev-parse git trick solves it all


eumagga0x2a

Yes, confirmed, thanks!

By the way, why do the cmake scripts still take svn into account? Old cruft which doesn't hurt but is to cumbersome to remove? Or is there a real use for it?