Re: [patch 1/1] document Acked-by:

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

 



"John Anthony Kazos Jr." <[email protected]> writes:

> "Acked-by:" does not mean "I like this" but rather "I approve of this".

I'd say it means "I acknowledge it". If you want to express
approval, why not use some sort of "Approved-by"?

> If I put "Acked-by: John..." on a patch of any kind, even trivial, it 
> would look incredibly stupid, because I'm just some guy messing around 
> with the kernel. A tactful response to me doing that from any actual 
> kernel bigwig would be, "I appreciate your enthusiasm, but you are not 
> part of the kernel patch flow." Similarly, a tactful response to me 
> NACKing a patch would be, "I appreciate your concern, but you are in no 
> position to remove a patch from the stream. Your comments will be 
> considered and implemented or countered by an actual maintainer."
>
> This is appropriate.

You seem to know these things very well.
-- 
Krzysztof Halasa
-
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