From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v5] ABI: Add abi checking utility Date: Tue, 17 Mar 2015 22:17:45 +0100 Message-ID: <1846221.pQ0YdT9c48@xps13> References: <1422652596-12777-1-git-send-email-nhorman@tuxdriver.com> <1426615721-32119-1-git-send-email-nhorman@tuxdriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Neil Horman Return-path: In-Reply-To: <1426615721-32119-1-git-send-email-nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2015-03-17 14:08, Neil Horman: > There was a request for an abi validation utilty for the ongoing ABI stability > work. As it turns out there is a abi compliance checker in development that > seems to be under active development and provides fairly detailed ABI compliance > reports. Its not yet intellegent enough to understand symbol versioning, but it > does provide the ability to identify symbols which have changed between > releases, along with details of the change, and offers developers the > opportunity to identify which symbols then need versioning and validation for a > given update via manual testing. > > This script automates the use of the compliance checker between two arbitrarily > specified tags within the dpdk tree. To execute enter the $RTE_SDK directory > and run: > > ./scripts/validate_abi.sh $GIT_TAG1 $GIT_TAG2 $CONFIG > > where $GIT_TAG1 and 2 are git tags and $CONFIG is a config specification > suitable for passing as the T= variable in the make config command. > > Note the upstream source for the abi compliance checker is here: > http://ispras.linuxbase.org/index.php/ABI_compliance_checker > > It generates a report for each DSO built from the requested tags that developers > can review to find ABI compliance issues. > > Signed-off-by: Neil Horman > --- > > Change Notes: > > v2) Fixed some typos as requested by Thomas > > v3) Fixed some additional typos Thomas requested > Improved script to work from detached state > Added some documentation to the changelog > Added some comments to the scripts > > v4) Remove duplicate exports. > Move restoration of starting branch/comit to cleanup_and_exit > > v5) Fixed exit cleanup > Added MAINTAINERS entry Acked-by: Thomas Monjalon Applied, thanks