From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 06 Nov 2023 14:44:04 +0100 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1qzztb-000D1w-2Y for lore@lore.pengutronix.de; Mon, 06 Nov 2023 14:44:04 +0100 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1qzzta-0003nS-Rh; Mon, 06 Nov 2023 14:44:02 +0100 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1qzztZ-0003n8-OH; Mon, 06 Nov 2023 14:44:01 +0100 Message-ID: <16005265-1560-721a-b208-e98df874f3ba@pengutronix.de> Date: Mon, 6 Nov 2023 14:44:01 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Content-Language: en-US To: =?UTF-8?Q?Enrico_J=c3=b6rns?= , Roland Hieber References: <20231106115822.268672-1-a.fatoum@pengutronix.de> <20231106120426.qa7s6j7bktnxsp77@pengutronix.de> <3f9dc13e7710d87ad6a022d07c0c3e2314c0e5a1.camel@pengutronix.de> From: Ahmad Fatoum In-Reply-To: <3f9dc13e7710d87ad6a022d07c0c3e2314c0e5a1.camel@pengutronix.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [OSS-Tools] [PATCH dt-utils] meson: align libdt-utils version with autotools' X-BeenThere: oss-tools@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: Pengutronix Public Open-Source-Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: oss-tools@pengutronix.de, =?UTF-8?Q?Enrico_J=c3=b6rns?= Sender: "OSS-Tools" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: oss-tools-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false Hello Enrico, On 06.11.23 13:12, Enrico Jörns wrote: > Am Montag, dem 06.11.2023 um 13:04 +0100 schrieb Roland Hieber: >> On Mon, Nov 06, 2023 at 12:58:22PM +0100, Ahmad Fatoum wrote: >>> Makefile.am has: >>> >>>   LIBDT_CURRENT=6 >>>   LIBDT_REVISION=0 >>>   LIBDT_AGE=0 >>> >>> along with a comment on how the values were chosen. Copy the comment and >>> the values into the meson.build as well, so appropriate symlinks >>> pointing at the versioned library are created. We forego the extra >>> complexity of having a common file that's read from both build systems >>> as we are intent on phasing out autotools anyway, once wrinkles such as >>> what's fixed here are ironed out. >>> >>> Reported-by: Enrico Jörns >>> Signed-off-by: Ahmad Fatoum >> >> Reviewed-by: Roland Hieber >> >>> --- >>>  meson.build | 9 +++++++++ >>>  1 file changed, 9 insertions(+) >>> >>> diff --git a/meson.build b/meson.build >>> index 9579e712dfea..9894d7311bb3 100644 >>> --- a/meson.build >>> +++ b/meson.build >>> @@ -133,6 +133,15 @@ libdt = shared_library('dt-utils', >>>    c_args : ['-include', meson.current_build_dir() / 'version.h'], >>>    dependencies : [udevdep, versiondep], >>>    gnu_symbol_visibility : 'default', >>> +# If the library source code has changed at all since the last release, >>> +#   then increment revision (‘c:r:a’ becomes ‘c:r+1:a’). >>> +# If any interfaces have been added/removed/changed since the last release, >>> +#   then increment current, and set revision to 0. > > Does this make sense? It was copied into Makefile.am verbatim from the libtool documentation: https://www.gnu.org/software/libtool/manual/html_node/Updating-version-info.html > Isn't "current" meant to mark *incompatible* public API changes? libtool numbering is new to me as well, but apparently "Current" is the most recent interface number that this library implements. The major version number should thus be current - age, not just current. I will fix that in v2. See https://www.gnu.org/software/libtool/manual/html_node/Libtool-versioning.html https://gitlab.gnome.org/GNOME/gcab/-/merge_requests/6/diffs?commit_id=2c8048f74ec8c088397d47730aa47c574526918f > This might be valid for "removed" or "changed", but not for "added". > > How does "interfaces" differ from "public interfaces" below? No difference AFAIK. Cheers, Ahmad > > Regards, Enrico > >>> +# If any public interfaces have been added since the last public release, >>> +#   then increment age. >>> +# If any interfaces have been removed or changed since the last release, >>> +#   then set age to 0. >>> +  version: '6.0.0', >>>    install : true) >>>   >>>  executable('barebox-state', >>> -- >>> 2.39.2 >>> >>> >>> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |