From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id D81315B3E for ; Fri, 2 Nov 2018 12:51:04 +0100 (CET) Received: from [107.15.85.130] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1gII5F-0002Kr-DH; Thu, 01 Nov 2018 14:52:57 -0400 Date: Fri, 2 Nov 2018 07:50:45 -0400 From: Neil Horman To: Thomas Monjalon Cc: dev@dpdk.org, doucette@bu.edu Message-ID: <20181102115045.GA31972@hmswarspite.think-freely.org> References: <20181101135410.15945-1-nhorman@tuxdriver.com> <3649242.bmXl2ZTEod@xps> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3649242.bmXl2ZTEod@xps> User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Score: 0.3 (/) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCH] check-symbol-change: fix regex to match on end of map file 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: , X-List-Received-Date: Fri, 02 Nov 2018 11:51:05 -0000 On Thu, Nov 01, 2018 at 11:53:00PM +0100, Thomas Monjalon wrote: > 01/11/2018 14:54, Neil Horman: > > the regex to determine the end of the map file chunk in a patch seems to > > be wrong, It was using perl regex syntax, which awk doesn't appear to > > support (I'm still not sure how it was working previously). Regardless, > > it wasn't triggering and as a result symbols were getting added to the > > mapdb that shouldn't be there. > > > > Fix it by converting the regex to use traditional posix syntax, matching > > only on the negation of the character class [^map] > > > > Tested and shown to be working on the ip_frag patch set provided by > > doucette@bu.edu > > > > Signed-off-by: Neil Horman > > CC: thomas@monjalon.net > > CC: doucette@bu.edu > > Reported-by: doucette@bu.edu > > You could use these lines: > > Fixes: 4bec48184e33 ("devtools: add checks for ABI symbol addition") > > Reported-by: Cody Doucette > I'm fine with the second line, and the first is fine I guess, but I'm not sure there is an exact correlation > > --- a/devtools/check-symbol-change.sh > > +++ b/devtools/check-symbol-change.sh > > - /[-+] a\/.*\.^(map)/ {in_map=0} > > + /[-+] a\/.*\.[^map]/ {in_map=0} > > Not sure this is what you intend: > [^map] means any character except "m", "a" and "p". > Its not 100%, but its pretty close. The regex for exact matching on not a specific string is pretty large and complex. Since we have no files that that end in .m .a or .p, this should give us what we want for the forseeable future. > I don't know whether awk supports this syntax: (?!foo) > It unfortunately doesn't, thats perl syntax, and while grep I think supports it, awk is more strictly posix compliant. Neil > >