lkml.org 
[lkml]   [2010]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] perf build fixes for opensuse 10.0

* John Kacur <jkacur@redhat.com> wrote:

> On Wed, Sep 22, 2010 at 1:15 PM, Andi Kleen <andi@firstfloor.org> wrote:
> >> On Wed, Sep 22, 2010 at 11:03 AM, Andi Kleen <andi@firstfloor.org> wrote:
> >>> Fix the perf Makefile to compile on opensuse 10.0
> >>>
> >>> -Werror in shipping code is usually a bad idea. Remove that.
> >>
> >> Why not remove it then when you ship the code, but leave it in the kernel
> >> where it is useful?
> >
> > What do you mean with "you ship"?
>
> The subject of your email is "perf build fixes for opensuse 10.0"
> So, I inferred from there.
>
> > I downloaded a kernel from kernel.org, it shipped to me and didn't
> > build.
>
> Any reason you can't fix the build errors instead?

Well, it's Andi - he has a well established track record for picking the
worst possible solution for pretty much anything and then being
passive-agressive difficult about it as if there was no tomorrow. It's
typically quite time-consuming for everyone involved, with little
productive output. YMMV.

Here he could have done a dozen things to make his posting more useful:
he could have included the build warnings in the changelog (he didnt),
he could have tried to address them (he didnt), he could have argued for
some specific warnings to be disabled on specific, well-established
grounds, for example demonstrating that it _only_ produces false
positives (he didnt) - etc. etc. He did none of those things.

Thanks,

Ingo


\
 
 \ /
  Last update: 2010-09-22 20:23    [W:0.355 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site