From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 42F45A0577; Tue, 7 Apr 2020 21:52:27 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 781131BF90; Tue, 7 Apr 2020 21:52:26 +0200 (CEST) Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 780871BF7B for ; Tue, 7 Apr 2020 21:52:25 +0200 (CEST) Received: from 2606-a000-111b-43ee-0000-0000-0000-1bf2.inf6.spectrum.com ([2606:a000:111b:43ee::1bf2] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1jLuGQ-0001by-Al; Tue, 07 Apr 2020 15:52:09 -0400 Date: Tue, 7 Apr 2020 15:52:01 -0400 From: Neil Horman To: Thomas Monjalon Cc: David Marchand , Kevin Laatz , mdr@ashroe.eu, dev Message-ID: <20200407195201.GC128325@hmswarspite.think-freely.org> References: <20200406193430.81268-1-nhorman@tuxdriver.com> <20200407113311.GB128325@hmswarspite.think-freely.org> <35252260.WIbDr2VvFA@thomas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <35252260.WIbDr2VvFA@thomas> X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCH] Remove abi_versioning.sh from tree X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Apr 07, 2020 at 01:58:57PM +0200, Thomas Monjalon wrote: > 07/04/2020 13:33, Neil Horman: > > On Tue, Apr 07, 2020 at 09:36:17AM +0200, David Marchand wrote: > > > On Mon, Apr 6, 2020 at 9:34 PM Neil Horman wrote: > > > > > > > > Since we've moved away from our initial abi_versioning.sh script, in > > > > > > abi_versioning.sh does not exist (idem with the patch title). > > > I suppose you meant validate-abi.sh. > > > > > Crud, you're right, I was convoluting terms, sorry. Shall I repost with a > > corrected changelog? > > Not only the commit log, look below how you did not care about the basic > usage of the new tool. > > > > > > favor of check_abi.sh, which uses libabigail, remove the old script from > > > > > > check-abi.sh > > > > > > > the tree, and update the docs accordingly > [...] > > > > --- a/doc/guides/contributing/abi_versioning.rst > > > > +++ b/doc/guides/contributing/abi_versioning.rst > > The maintainer of doc/guides/contributing/abi_*.rst > is Ray Kinsella so I add him as Cc. > > [...] > > > > -The syntax of the ``validate-abi.sh`` utility is:: > > > > +The syntax of the ``check-abi.sh`` utility is:: > > > > > > > > - ./devtools/validate-abi.sh > > > > + ./devtools/check-abi.sh > > > > > > The new script is not a direct replacement. > > > It won't take git revisions, but build directories where versions of > > > dpdk have been compiled. > > > > > > devtools/test-build.sh and devtools/test-meson-builds.sh illustrate its use. > > > https://git.dpdk.org/dpdk/tree/devtools/test-meson-builds.sh#n127 > > David, I think Neil did not take time to understand what changed in > ABI tooling. > I really wonder who is the real maintainer of ABI tooling and policy. > Neil, Ray, I was expecting a better involvement in this major > policy enforcement. > > This is where we are: > - Neil asked first for ABI compatibility > - Neil created validate-abi.sh > - Ray asked for a strict policy > - Kevin worked on a new tooling > - David completed the tooling work > - David integrated ABI checks in Travis > > There are many people partly involved. > I think we need one person truly involved in ABI questions, > someone who feels responsible and will take care of details > like the documentation update requested above. > > Please don't rely on David and myself, we are already very busy > with making sure every patches are properly reviewed. > We need good help on the ABI topic in general. > We're all very busy Thomas, theres no need to get bent out of shape. You're right, I thought this was easier than it was, thinking that check-abi.sh was interface compatible, apologies. I'm going through this more carefully, and will repost soon. FWIW, I don't think I'll be able to fill this role full time. I'm not sure if anyone else is either, i'm afraid. Neil > >