From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH] scripts: support any legal git revisions as abi validation range Date: Thu, 3 Dec 2015 15:39:20 +0200 Message-ID: <56604608.7040503@redhat.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: "Mcnamara, John" , "dev@dpdk.org" Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 329DE569A for ; Thu, 3 Dec 2015 14:39:24 +0100 (CET) In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 12/03/2015 02:14 PM, Mcnamara, John wrote: >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, December 2, 2015 4:51 PM >> To: dev@dpdk.org >> Subject: [dpdk-dev] [PATCH] scripts: support any legal git revisions as >> abi validation range >> >> In addition to git tags, support validating abi between any legal >> gitrevisions(7) syntaxes, such as "validate-abi.sh . -1 " >> "validate-abi.sh master mybrach " etc in addition to validating >> between tags. Makes it easier to run the validator for in-development >> work. > > Hi Panu, > > +1 for this. > > You might also change the ABI validation section of the docs to go along > with this. Something like the patch below. If not I'll submit it > afterwards. Good points, including changing the usage message to REV instead of TAG. I'll send an improved version based on this, thanks. > > Also, if someone has some bandwidth it would be good to add an option > to pass -j with an optional number to "make" in the script. Can do, although I'm still waiting fo my previous, semi-related validate-abi patches from September to be applied... - Panu -