[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100714161704.GE23436@shisha.kicks-ass.net>
Date: Wed, 14 Jul 2010 19:17:04 +0300
From: Alexander Shishkin <virtuoso@...nd.org>
To: Sam Ravnborg <sam@...nborg.org>
Cc: linux-kbuild@...r.kernel.org, Sebastian Dalfu? <sd@...f.de>,
Oliver Hartkopp <oliver@...tkopp.net>,
Michael Guntsche <mike@...loops.com>,
Jonathan Nieder <jrnieder@...il.com>,
Michael Tokarev <mjt@....msk.ru>, Alek Du <alek.du@...el.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Michal Marek <mmarek@...e.cz>,
Russell King <rmk@....linux.org.uk>,
"H. Peter Anvin" <hpa@...or.com>,
Albin Tonnerre <albin.tonnerre@...e-electrons.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] [RFC] implement size_append in perl
On Wed, Jul 14, 2010 at 05:23:37 +0200, Sam Ravnborg wrote:
> On Wed, Jul 14, 2010 at 05:01:03PM +0300, Alexander Shishkin wrote:
> > The current implementation has been several times fixed to work around
> > different shells and build environments and it remains a somewhat tricky
> > shell script. This patch replaces all the dependencies by a single line
> > of perl. Build dependency on perl is not new, so that shouldn't be a
> > problem.
> >
> > The new implementation does exactly what it's supposed to: it outputs
> > a little-endian 32-bit unsigned integer of a file's length.
> >
> > This might simplify matters with different build environments and
> > different shells and is also considerably shorter and hopefully more
> > readable.
>
> Reminds me that I had a patch that moved this to mkpiggy.c.
> Goggle gave this hit:
>
> http://lists.openwall.net/linux-kernel/2009/12/20/92
Ok, thanks for the reference.
> I will try to update this to cover arm also and check
> if other archs needs this.
>
> In other word - move thes size stuff to mkpiggy so we
> skip this extra step.
Hm, maybe this mkpiggy can be made arch-independent and moved to scripts/
as well?
Regards,
--
Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists