Re: [discuss] Re: [PATCH] [6/6] i386: Move CONFIG_DOUBLEFAULT into arch/i386 where it belongs.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 20, 2006 at 03:26:21PM +0200, Andi Kleen wrote:
> On Thursday 20 April 2006 13:49, Adrian Bunk wrote:
> 
> > My main problem with his patch is still the way he did it - sending a 
> > patch reverting a recently included patch with neither discussion before 
> > the patch nor mentioning in the patch that it's a revert nor a Cc to the 
> > people involved with the patch.
> 
> I just noticed a problem (bogus symbols in my x86-64 config) and fixed it. I normally 
> don't look at which patch it introduced for such trivial changes.

First of all, the "problem" of an unset config variable in the .config 
is at most a cosmetical issue.

And even if it had been a real problem, discussing such an issue and 
convincing people why you consider it to be bad is always better, since 
whoever gets conviced in the situation will have learned for similar 
future cases.

There are ways how things should work and we should follow them.

As an example, there is one trivial x86_64 specific patch removing the 
small bloat caused by an unused export I made in -mm Andrew has 
forwarded at least twice to you.

Until now, I'd have considered it an unfriendly act to forward such a 
patch to Linus through my trivial tree bypassing you as the maintainer, 
but looking at your statement it seems to be OK for you if I send such 
trivial patches directly to Linus without bothering to go through you as 
the maintainer.

> -Andi

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux