netdev - Re: [PATCH] Staging: qlge: Remove parenthesis around single condition

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 12 Mar 2023 11:39:17 +0100
From:   "Fabio M. De Francesco" <fmdefrancesco@...il.com>
To:     Sumitra Sharma <sumitraartsy@...il.com>
Cc:     GR-Linux-NIC-Dev@...vell.com, coiby.xu@...il.com,
        error27@...il.com, gregkh@...uxfoundation.org,
        linux-kernel@...r.kernel.org, linux-staging@...ts.linux.dev,
        manishc@...vell.com, netdev@...r.kernel.org,
        outreachy@...ts.linux.dev, julia.lawall@...ia.fr
Subject: Re: [PATCH] Staging: qlge: Remove parenthesis around single condition

On domenica 12 marzo 2023 08:17:00 CET Sumitra Sharma wrote:
> Hi Fabio,
> 
> Thank you for the insights.

Hi Sumitra,

Please pay attention to the suggestions that Julia kindly provides to you.

A moment ago I read these other words from her to you: "Sorry to press the 
point, but you should be doing it now, not promising to do it in the future.".

She is sorry to press because of her special attitudes. I've been helped 
uncountable times by Julia who, despite her major role and commitments, is so 
kind to spend time here in a Sunday morning. 

I think that it's you who should be sorry for pushing her that way... 

You are doing it again and again: in your last message you did not provide the 
necessary context. You are asking me to recall which "insights" you are 
talking about. I know for sure that here there is someone who read hundreds of 
emails per day.

> I went to the .rst files because they are directly
> linked in the first-patch document. I also noticed the difference between a
> ".rst" file and its counter human readable ".html" file. You were right that
> many information is being missed when anyone will read the .rst instead of
> .html/.pdf.

Don't summarize my own words in order to make me understand what question you 
are answering. Just put my question/objection/comment inline while replying.

Most mentors here read several tens of emails per day. You should not expect 
them to skim the whole thread. Just reply inline, so they find all the 
necessary context in one place and so there is no need to traverse the thread 
to understand what you are referring to. 

You have been told so several times. You risk that someone starts to ignore 
your messages in order to save time for other applicants.

> I would like to suggest that the links that redirect to the .rst
> source files in the first-patch document must be changed to the links that
> redirect to there corresponding human readable format. Let me know if I 
could
> do it under the [KERNEL NEWBIEs ACCESS].

This is a great idea and I fully agree with you.
Please ask Alison to grant access to change that documentation (it's not 
something I can do myself).

> Apart from this I will be happy to patch the style guide after this
> contribution period.

Best applicants go on with further contributions after the end of these 
period. This fact has been observed several times in the other rounds.
I'd like to see more people who are seriously willing to be part of this 
Community, regardless of the Outreachy project's deadlines. 

> Regards,
> 
> Sumitra

Thanks,

Fabio

P.S.: I probably went a bit too far in the first part of this email. I am 
sorry about it. However, please demonstrate that you finally got it, not with 
further promises but with actual changes in the way you communicate :-)