mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "NeilBrown" <neilb@suse.de>
To: "Rob Landley" <rob@landley.net>
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 16:03:50 +1100	[thread overview]
Message-ID: <f2dbd45db6d92b291359456c98cf1f35.squirrel@neil.brown.name> (raw)
In-Reply-To: <200910052344.46809.rob@landley.net>

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-Octo
>> >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'm have to remove the perl dependency though.

NeilBrown



>    and the first perl
> removal
> patches were submitted for 2.6.26, so the real question is "why was it
> added,
> and was it worth it?"
>
> There was a long thread on an earlier submission of the patches, which
> Alan
> Cox concluded (as far as I'm concerned, unless Linus, Andrew Morton, or
> maybe
> Al Viro want to weigh in) with this post:
>
>   http://lkml.indiana.edu/hypermail/linux/kernel/0901.1/02108.html
>
> Archives are linked from there, you might have to check adjacent weeks
> too.
>
> Rob
> --
> Latency is more important than throughput. It's that simple. - Linus
> Torvalds
>


  reply	other threads:[~2009-10-06  5:04 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-05 16:01 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 [this message]
2009-10-06  6:34         ` Rob Landley
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=f2dbd45db6d92b291359456c98cf1f35.squirrel@neil.brown.name \
    --to=neilb@suse.de \
    --cc=berk@panix.com \
    --cc=dronnikov@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=rob@landley.net \
    /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

all inboxes | Powered by JetHome®