dtc icon indicating copy to clipboard operation
dtc copied to clipboard

Fix version in libfdt/meson.build

Open anatol opened this issue 1 year ago • 4 comments

Currently meson generates file /usr/lib/libfdt.so.1.6.0 that does not match Makefile build version.

anatol avatar May 02 '23 20:05 anatol

Ugh, yes. I already knew of the duplicated version between Makefile and meson.build, but I hadn't spotted there was a third version in libfdt/meson.build in the specifics for the shared library. We really need to remove this redundancy.

I suspect there's a nice way to do that in meson, but I don't know what it is.

dgibson avatar May 03 '23 13:05 dgibson

Since the library soname is always based on the project release version, you can just use version: meson.project_version() to ensure that meson.build never gets out of sync with libfdt/meson.build.

Syncing the Makefile and meson.build is a different question. As far as I can tell, in the Makefile build you never actually use the individual integer components, so you could just store the version in a file as DTC_VERSION = $(shell cat VERSION.txt). For meson, that can be read in with

project(
    ...,
    version: files('VERSION.txt'),
    meson_version: '>=0.57.0'
)

Note your current minimum required meson is not spelled out but is 0.56.0 due to use of features introduced in that version. Bumping from 0.56.0 to 0.57.0 is a bit notable inasmuch as Debian stable ships 0.56.2 unless users enable the backports repository, in which case they receive 1.0.0.

eli-schwartz avatar May 14 '23 16:05 eli-schwartz

This is causing me issues somewhat. @anatol, do you plan to continue with this PR?

ernestask avatar Jul 28 '23 20:07 ernestask

@anatol @dgibson this was fixed awhile ago, this pull request can be closed

blmaier avatar Jun 26 '24 17:06 blmaier

@blmaier thanks for the update.

dgibson avatar Jul 01 '24 09:07 dgibson