On Wed, Mar 11, 2020 at 4:54 PM Igor Opaniuk igor.opaniuk@gmail.com wrote:
On Wed, Mar 11, 2020 at 3:04 PM Andy Shevchenko andriy.shevchenko@linux.intel.com wrote:
On Wed, Mar 11, 2020 at 06:24:07AM -0600, Simon Glass wrote:
On Wed, 11 Mar 2020 at 06:09, Andy Shevchenko andriy.shevchenko@linux.intel.com wrote:
On Tue, Mar 10, 2020 at 08:28:29PM -0600, Simon Glass wrote:
On Tue, 10 Mar 2020 at 08:51, Andy Shevchenko andriy.shevchenko@linux.intel.com wrote:
On Sat, Mar 07, 2020 at 04:22:13PM -0700, Simon Glass wrote:
...
It looks like it went to the mailing list...
are you subscribed?
It doesn't matter if you want people to pay an attention (by putting them to Cc list), so, since it was one patch I can't see the rest and know if they affect or not the one, you Cc'ed me to.
I don't fully understand this, but are you saying you did not get the patches in your email? If you subscribe to the U-Boot mailing list you should receive all patches even if not cc'd to you.
Probably I got to another email, not this one.
I did not specifically cc you on this series but I can do that if I send it again. But patman has added you to one patch (number 3) and therefore you are also cc'd on the cover letter.
Yes, and this one I got without seeing full picture.
Can you check whether you are subscribed to the mailing list?
As I said, I am not subscribed by email patman uses to Cc me.
Basically it's a bug in patman. Either it shouldn't tell me, or should Cc to the entire series. Of course there are cases, when patches are independent and no need to spam everybody with tons of emails, but this should be clarified in cover letter. Did I miss that?
AFAIR, last time you were not happy about "irrelevant" patches that were CC-ed to you "by mistake" [1]. This time, as I see, is vice-versa.
Nope, it's not. It maybe that I missed cover letter to tell that the rest is irrelevant to me. Otherwise Cc list is not filled correctly.
The algo is simple - if the change touches a file where you're marked as one of commiters/maintainers - it will add your email to CC, otherwise it won't (however you can still find the full list of patches in the ML, it takes a couple of seconds only).
Yes, but here is obvious flaw. If the change is done in the file dependent to the preparatory patch(es), then I need to be Cc'ed in all relevant.
For example, patch 1 does introduce nice helper function foo(), patches 2-(N-1) does change some irrelevant files, patch N touches my stuff, I have to be Cc'ed to the 1) cover letter, 2) patch 1, 3) patch N.
Unfortunately current state of tooling in U-Boot is not too good to be able to read thoughts of people and guess their wishes.
See above. *Algo is simple*.
But if you know how to improve that, patches are always welcome!