Re: [PATCH] Documentation: process: Fix grammar errors in management-style.rst

[Jonathan Corbet <corbet@lwn.net>]

I Viswanath <viswanathiyyappan@gmail.com> writes:

> Fix various grammatical errors in management-style.rst
>
> Signed-off-by: I Viswanath <viswanathiyyappan@gmail.com>

Thank you for working to improve our documentation...I don't think I can
apply this patch, though.

Your signoff should include your full name, please.

But more importantly...

> ---
>  Documentation/process/management-style.rst | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/process/management-style.rst b/Documentation/process/management-style.rst
> index dfbc69bf49d4..4ae05a89950a 100644
> --- a/Documentation/process/management-style.rst
> +++ b/Documentation/process/management-style.rst
> @@ -15,8 +15,8 @@ to do with reality.  It started as a lark, but that doesn't mean that it
>  might not actually be true. You'll have to decide for yourself.
>  
>  Btw, when talking about "kernel manager", it's all about the technical
> -lead persons, not the people who do traditional management inside
> -companies.  If you sign purchase orders or you have any clue about the
> +leads, not those who do traditional management inside
> +companies.  If you sign purchase orders or have any clue about the

This change is OK, but what was there was OK too

>  budget of your group, you're almost certainly not a kernel manager.
>  These suggestions may or may not apply to you.
>  
> @@ -42,9 +42,9 @@ actually true.
>  The name of the game is to **avoid** having to make a decision.  In
>  particular, if somebody tells you "choose (a) or (b), we really need you
>  to decide on this", you're in trouble as a manager.  The people you
> -manage had better know the details better than you, so if they come to
> +manage better know the details better than you; if they come to

This one changes the meaning of the sentence

>  you for a technical decision, you're screwed.  You're clearly not
> -competent to make that decision for them.
> +competent enough to make that decision for them.

...and this one was fine as it was.

>  (Corollary:if the people you manage don't know the details better than
>  you, you're also screwed, although for a totally different reason.
> -- 

In general, this kind of playing with the wording for text that is not
actually wrong is not hugely helpful; I would encourage you to look
elsewhere for places to contribute to the documentation.

Thanks,

jon