linux-raid.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rob Landley <rob@landley.net>
To: NeilBrown <neilb@suse.de>
Cc: berk walker <berk@panix.com>,
	Vladimir Dronnikov <dronnikov@gmail.com>,
	linux-kernel@vger.kernel.org, linux-raid@vger.kernel.org
Subject: Re: [PATCH 1/1] md: drivers/md/unroll.pl replaced with awk analog
Date: Tue, 6 Oct 2009 01:34:56 -0500	[thread overview]
Message-ID: <200910060134.57457.rob@landley.net> (raw)
In-Reply-To: <f2dbd45db6d92b291359456c98cf1f35.squirrel@neil.brown.name>

On Tuesday 06 October 2009 00:03:50 NeilBrown wrote:
> On Tue, October 6, 2009 3:44 pm, Rob Landley wrote:
> > On Monday 05 October 2009 18:57:14 berk walker wrote:
> >> Rob Landley wrote:
> >> > On Monday 05 October 2009 11:01:39 Vladimir Dronnikov wrote:
> >> >> From: Vladimir Dronnikov <dronnikov@gmail.com>
> >> >>
> >> >> drivers/md/unroll.pl replaced by awk script to drop build-time
> >> >> dependency on perl
> >> >>
> >> >> Signed-off-by: Vladimir Dronnikov <dronnikov@gmail.com>
> >> >
> >> > Acked-by: Rob Landley <rob@landley.net>
> >> >
> >> > We already discussed this on another mailing list, thread starts at:
> >> >
> >> > http://lists.impactlinux.com/pipermail/firmware-impactlinux.com/2009-O
> >> >cto ber/000328.html
> >> >
> >> > I've added this as patch #4 in the perl removal series I've submitted
> >> > during the last few merge windows.
> >> >
> >> > Thanks,
> >> >
> >> > Rob
> >>
> >> Why is perl being removed? [I know that I have missed a lot]
> >> berk-
> >
> > Before 2.6.25 the kernel build had never used perl,
>
> Uhhhmmmm.  md has used perl for creating some C files since
> RAID6 was added, which is before the dawn of git.
>
> So I don't think this statement is true.

I was wrong, my mistake.  (The kernel builds I was doing had never needed 
perl, but I wasn't building raid for any target systems.  The systems that had 
raid were generally big suckers using distro kernels.  Vladimir _is_ building 
with raid support, so he addressed the issue.)

My current perl removal patch series started when timeconst.pl came in, 
meaning you couldn't build any kernel without perl anymore because 
_everything_ uses time constants.  My current patch to remove that is 
something like the fifth version I've had to do to keep up with various changes 
to the kernel since then.  Most recently reposted here:

  http://lkml.indiana.edu/hypermail/linux/kernel/0909.2/01661.html

Before that, I noticed (and removed) perl from the User Mode Linux build in 
2005:

  http://lkml.indiana.edu/hypermail/linux/kernel/0503.1/0806.html

It's not _just_ perl, it's gratuitous build dependencies in general, which 
make a cross-compiler's life unpleasant.  For example, there was a window 
during which you needed curses development headers to run "make oldconfig" (and 
that leaked into uClibc's copy of kconfig when they synced with upstream), but 
it was pretty easy to patch out and Sam Ravnborg had already fixed it upstream 
before I got around to pushing that.

I hit this sort of thing and have to fix it locally, life is easier for me if 
it's fixed upstream so I don't have to maintain out-of-tree patches...

Thanks,

Rob
-- 
Latency is more important than throughput. It's that simple. - Linus Torvalds

  reply	other threads:[~2009-10-06  6:34 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-05 16:01 [PATCH 1/1] md: drivers/md/unroll.pl replaced with awk analog Vladimir Dronnikov
2009-10-05 21:30 ` Rob Landley
2009-10-05 23:57   ` berk walker
2009-10-06  4:44     ` Rob Landley
2009-10-06  5:03       ` NeilBrown
2009-10-06  6:34         ` Rob Landley [this message]
2009-10-05 22:54 ` Neil Brown
2009-10-06  5:34   ` Vladimir Dronnikov
2009-10-12  6:01     ` Neil Brown
2009-10-12  6:46       ` Vladimir Dronnikov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200910060134.57457.rob@landley.net \
    --to=rob@landley.net \
    --cc=berk@panix.com \
    --cc=dronnikov@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=neilb@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).