Unbreak "Avidemux vADM_VERSION (ADM_SUBVERSION)"

Started by mm0359, March 22, 2017, 09:18:54 AM

Previous topic - Next topic

mm0359

win32/avidemux_2.6.18_r170322_win32.exe

admlog.txt header has been broken for a few releases (iirc):
QuoteAvidemux vADM_VERSION (ADM_SUBVERSION) .

Would be nice to unbreak it...

Fwiw,
ADM_VERSION search
And possibly sync main.cpp to be (more) like Q_about.cpp.

eumagga0x2a

I've noticed this too, but would like to investigate and hopefully solve the issue that my own Avidemux builds for windows currently don't run at all (access violation somewhere in Qt).

eumagga0x2a


mm0359

I checked ADD_SOURCE_CFLAGS search.

It looks like 3 lines were missed in [build/vs] Change how the strings are passed from cmake to compiler.
Or were they left as is on purpose?

cli/CMakeLists.txt

ADD_SOURCE_CFLAGS(../common/main.cpp "-DADM_VERSION='\"${AVIDEMUX_VERSION}\"'")


gtk/CMakeLists.txt

ADD_SOURCE_CFLAGS(../common/main.cpp "-DADM_SUBVERSION='\"${ADM_SUBVERSION}\"'")
ADD_SOURCE_CFLAGS(../common/main.cpp "-DADM_VERSION='\"${AVIDEMUX_VERSION}\"'")


eumagga0x2a

Quote from: mm0359 on April 01, 2017, 12:06:07 PM
It looks like 3 lines were missed in [build/vs] Change how the strings are passed from cmake to compiler.
Or were they left as is on purpose?

No, thank you for pointing out. The GTK part is not built, so it doesn't matter there.

mm0359

#5
Quote from: eumagga0x2a on March 29, 2017, 08:34:04 AM
[main] Fix version string generation would fix this

Commited as [cmake/qt/cli] Unset ADM_SUBVERSION for releases, fix version string generation for jobs
then [cmake] Revert unsetting ADM_SUBVERSION.

Don't we miss a few more IF (ADM_SUBVERSION)? (See ADM_SUBVERSION search.)
In
avidemux_core/corePackageTarGz.cmake (PS: Ah, it seems unused since v2.6.0!?)
avidemux/gtk/ADM_userInterfaces/ADM_dialog/CMakeLists.txt
avidemux/gtk/gtkPackageRpm.cmake
avidemux/gtk/CMakeLists.txt

mm0359


mm0359

#7
Quote from: mm0359 on March 22, 2017, 09:18:54 AM
ADM_VERSION search
And possibly sync main.cpp to be (more) like Q_about.cpp.

Could you remove the extra ending " ." in main.cpp?
And maybe the starting "  " too in main.cpp and ADM_jobs.cpp?

Moreover, could the latters use the same pattern(s) as Q_about.cpp?
Maybe all 3 like "vADM_VERSION (rADM_SUBVERSION, fflibs ADM_FFMPEG_VERSION)"?
(And "vADM_VERSION (Release, fflibs ADM_FFMPEG_VERSION)",
or even "vADM_VERSION - Release (rADM_SUBVERSION, fflibs ADM_FFMPEG_VERSION)".)

eumagga0x2a

This all is possible, but I don't dive into such style questions because they don't improve the substance of the application while binding ressources and it was not me who put the period here.

mm0359

Quote from: eumagga0x2a on April 01, 2017, 01:41:51 PM
I don't dive into such style questions

Noted.
Ftr, the data I miss most is ADM_FFMPEG_VERSION in log file.