Improve shared object versioning

[kmilos]

Currently because of

https://github.com/aous72/OpenJPH/blob/77aae494a6a221d1af403a4cdea8b458c6c1caa1/src/core/CMakeLists.txt#L139

it is necessary to rebuild all clients on every minor release, even if API/ABI has not changed, which is somewhat inconvenient (and wasteful).

Either

1. Have soverison as major only (0, i.e. "unstable" for now) and let clients deal w/ potential breakage on a case by case basis depending on each release until you reach stable 1.x, 2.x etc., or
2. Introduce a separate library version that's bumped only when needed (e.g. you could even fix it at 0.17 now and increase manually until you reach 1.x)

[aous72]

Thank you Milos. I will look into this.

0.17 has changes to both library API and apps.

[kmilos]

Thanks. There was e.g. no such change between 0.15 and 0.16, it was "just" performance dynamically linked apps would get for free w/o a real need to rebuild... So, could've been a patch update only, but I get why the performance bump might be seen as a more significant one. Anyhow, I guess a separate API/ABI versioning might provide a way to manage releases in a more flexible fashion. Or just be more strict on the patch vs minor...

[hmaarrfk]

One of the hiccups I faced regarding this is that the "version" is included in the .lib extension on windows.

This is the first time I have seen a "Version identifier" in a lib file.

It would be nice if the .lib didn't have any version identifier.

For example, ffmpeg does:

Library/lib/avcodec.lib
Library/lib/avdevice.lib
Library/lib/avfilter.lib
Library/lib/avformat.lib
Library/lib/avutil.lib

but installs

Library/bin/avcodec-61.dll
Library/bin/avdevice-61.dll
Library/bin/avfilter-10.dll
Library/bin/avformat-61.dll
Library/bin/avutil-59.dll

which has the 'so version' in the dll, but not in the lib file.

https://conda-metadata-app.streamlit.app/?q=conda-forge%2Fwin-64%2Fffmpeg-7.1.0-gpl_hb26d62f_701.conda

In my release of openjph i'm patching out this behavior with https://github.com/conda-forge/openjph-feedstock/pull/1 but I don't know if what you would prefer.

[kmilos]

I don't see this problem building w/ MinGW - e.g. in the MSYS2 package one gets w/o any patching

bin/libopenjph-0.17.dll
lib/libopenjph.a
lib/libopenjph.dll.a

(.a is the true static lib built w/ -DBUILD_SHARED_LIBS=OFF and .dll.a is the implib that goes w/ versioned libopenjph-0.17.dll produced by -DBUILD_SHARED_LIBS=ON -DCMAKE_DLL_NAME_WITH_SOVERSION=ON)

Edit: Ah, there is a different MSVC path that could indeed be improved (or simply removed so people can handle it w/ CMake options/properties?)...

[VVD]

If you change ABI then change (increase) major version number too: 0.16.0 => 1.0.0 instead of 0.17.0 and SONAME libopenjph.0 => libopenjph.1.

[aous72]

Thank you for your suggestion. I think you agree with Milos.

Yes, my worry is that I might need to change the ABI often (I know this is a bad idea, because it breaks compatibility with existing apps) -- then I will end be with a high number for the major.

I feel I am still far away from version 1.0 -- still a lot of work needs to be done.
I am also planning a major restructure starting around Dec/Jan, which will change the whole ABI.

I still need to think it through. Suggestions are welcome.

[hmaarrfk]

Generally for projects with version "0", the minor version is treated at the "major" lol!!!!

Great work on this, do what feels right for you tbh.