From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ie0-f175.google.com ([209.85.223.175]:61534 "EHLO mail-ie0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754567Ab3DKQZP convert rfc822-to-8bit (ORCPT ); Thu, 11 Apr 2013 12:25:15 -0400 Received: by mail-ie0-f175.google.com with SMTP id c12so2272041ieb.34 for ; Thu, 11 Apr 2013 09:25:14 -0700 (PDT) Date: Thu, 11 Apr 2013 11:25:10 -0500 From: Rob Landley Subject: Re: [PATCH 3/3] convert headers_install.pl to headers_install.sh In-Reply-To: <5163F2F0.1070405@suse.cz> (from mmarek@suse.cz on Tue Apr 9 05:52:32 2013) Message-Id: <1365697510.18069.78@driftwood> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; DelSp=Yes; Format=Flowed Content-Disposition: inline Content-Transfer-Encoding: 8BIT Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Michal Marek Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, Thomas Gleixner , Josh Boyer , "Paul E. McKenney" , Andrew Morton , David Howells , Sam Ravnborg On 04/09/2013 05:52:32 AM, Michal Marek wrote: > On 28.2.2013 09:19, Michal Marek wrote: > > On 27.2.2013 06:58, Rob Landley wrote: > >> From: Rob Landley > >> > >> Remove perl from make headers_install by replacing a perl script > (doing > >> a simple regex search and replace) with a smaller, faster, simpler, > >> POSIX-2008 shell script implementation. The new shell script is a > single > >> for loop calling sed and piping its output through unifdef to > produce the > >> target file. > >> > >> Same as last time except for minor tweak to deal with code review > from here: > >> http://lkml.indiana.edu/hypermail/linux/kernel/1302.3/00078.html > >> > >> (Note that this drops the "arch" argument, which isn't used. Kbuild > >> already points to the right input files on the command line.) > >> > >> Signed-off-by: Rob Landley > > > > Looks good, I will apply it after v3.9-rc1 is out. > > I forgot to do it right after rc1, but I took the patch from the mm > tree > now. I also added Sam's Ack from the earlier submission. > > Michal Yay! Is this for 3.10? (Not seeing it in Linus's tree now, and -rc7 seems a bit late for anything to go in...) Thanks, Rob