Set version, soversion from cmake if available

[julianneswinoga]

This is a pretty basic thing that we should be pulling in from cmake

[jpakkane]

mesonbuild.interpreterbase.exceptions.InterpreterException: Assert failed: Shared library version not picked up correctly

Maybe because setting version/soversion changes the output file name?

[julianneswinoga]

I added that assert to double check that the cmake-meson interop is setting the version correctly. On Linux the shared library version is just appended to the filename (libcmModLib.so.1.0.1) so I can just check the target path ends with the version. On other OSs that seems to not be the case. I can either make the assert for Linux-only builders, or do some research to see how shared libraries are version tagged on other OSs.

This is only because I don't see a way to get a version from a target object, if there's a nicer way I'd love to use that 😁

[julianneswinoga]

Changed the test case assertion to be dependent on the host system.

Interestingly I also found that on Windows-Cygwin meson will name the library cygcmModLib-1.dll but cmake will name it cygcmModLib.dll 🤔

[julianneswinoga]

I just squashed the fixups, should be good to go now! 🙂

[dcbaker]

This still looks good to me, I've added it the milestone for the next release so we can get it in once the merge window opens.

[eli-schwartz]

Merge window is open, in it goes.

[mmomtchev]

This breaks with static libraries that export VERSION or SO_VERSION because the static_library target does not accept these kwargs. Now, should static libraries export these properties is a good question.

Many projects support being built as both shared or static library and I guess that very few filter out their target properties based on the library type.

[thesamesam]

The fix is in #15297.