Re: [FFmpeg-devel] Patchwork attribution

2019-04-19 Thread Lou Logan
On Fri, Apr 19, 2019, at 2:15 PM, Hendrik Leppkes wrote: > > The first mail like that arrived, and handling of it of course > depends on the mail client in question. But at least Patchwork doesn't > seem to like it much, it quite simply still attributes it to the > "shared" mail. > Not sure if th

Re: [FFmpeg-devel] Patchwork attribution

2019-04-19 Thread Hendrik Leppkes
On Fri, Apr 19, 2019 at 8:23 PM Lou Logan wrote: > > On Thu, 18 Apr 2019 18:01:27 -0400 > "Lou Logan" wrote: > > > > But we can certainly give it a try if you like. I'm not sure how patchwork > > will handle it. > > (Didn't realize my webmail client wasn't actually wrapping my text > although it

Re: [FFmpeg-devel] Patchwork attribution

2019-04-19 Thread Lou Logan
On Thu, 18 Apr 2019 18:01:27 -0400 "Lou Logan" wrote: > > But we can certainly give it a try if you like. I'm not sure how patchwork > will handle it. (Didn't realize my webmail client wasn't actually wrapping my text although it was doing it in the window at the expected lengths.) I just chang

Re: [FFmpeg-devel] Patchwork attribution

2019-04-19 Thread Lou Logan
On Fri, 19 Apr 2019 00:05:00 + Andreas Rheinhardt wrote: > But now I am unsure whether I should resend my patchsets? At this point I suggest resending as it will allow the commiter to avoid having to adjust the author and will help avoid mistakes.

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Andreas Rheinhardt
Hello, Gyan: > > Patchwork can incorrectly assign ownership. See > https://patchwork.ffmpeg.org/patch/12680/ > > The author is Sam John as identified by Message ID as well as the From > field in the headers, yet Patchwork attributes this patch to "Oliver > Collyer via ffmpeg-devel", a name which

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Lou Logan
On Thu, Apr 18, 2019, at 1:25 PM, Hendrik Leppkes wrote: > > Did you try the "wrap" option? As I understand it, it would preserve > the original mail entirely, and rely on the mail client of the > recipient (ie. us) to properly unwrap the MIME container. > We've had various problems of authorship i

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Hendrik Leppkes
On Thu, Apr 18, 2019 at 9:04 PM Lou Logan wrote: > > On Thu, Apr 18, 2019, at 4:51 AM, Hendrik Leppkes wrote: > > > > Whoever setup this ML sender rewriting thing should probably look into > > options to also re-write the patch content and add a "From:" line in > > there with the original name and

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Michael Niedermayer
On Thu, Apr 18, 2019 at 03:03:44PM -0400, Lou Logan wrote: > On Thu, Apr 18, 2019, at 4:51 AM, Hendrik Leppkes wrote: > > > > Whoever setup this ML sender rewriting thing should probably look into > > options to also re-write the patch content and add a "From:" line in > > there with the original n

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Lou Logan
On Thu, Apr 18, 2019, at 4:51 AM, Hendrik Leppkes wrote: > > Whoever setup this ML sender rewriting thing should probably look into > options to also re-write the patch content and add a "From:" line in > there with the original name and email to avoid issues. I enabled this due DMARC as Timo corr

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Gyan
On 18-04-2019 07:03 PM, Hendrik Leppkes wrote: On Thu, Apr 18, 2019 at 3:31 PM Gyan wrote: On 18-04-2019 06:13 PM, Hendrik Leppkes wrote: All of those patches with re-written senders are send from the same email, ffmpeg-devel@ffmpeg.org Presumably patchwork only sends the email itself, no

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Hendrik Leppkes
On Thu, Apr 18, 2019 at 3:31 PM Gyan wrote: > > > > On 18-04-2019 06:13 PM, Hendrik Leppkes wrote: > > On Thu, Apr 18, 2019 at 12:41 PM Gyan wrote: > >> Ok. Then shouldn't it assign 'Sam John via ffmpeg-devel' as the author? > >> It did do that for 'Oliver Collyer via ffmpeg-devel', the string w

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Gyan
On 18-04-2019 06:13 PM, Hendrik Leppkes wrote: On Thu, Apr 18, 2019 at 12:41 PM Gyan wrote: Ok. Then shouldn't it assign 'Sam John via ffmpeg-devel' as the author? It did do that for 'Oliver Collyer via ffmpeg-devel', the string wrongly substituted here. This looks like a parser failure in P

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Hendrik Leppkes
On Thu, Apr 18, 2019 at 12:41 PM Gyan wrote: > > > > On 18-04-2019 03:25 PM, Timo Rothenpieler wrote: > > On 18/04/2019 09:49, Gyan wrote: > >> > >> Patchwork can incorrectly assign ownership. See > >> https://patchwork.ffmpeg.org/patch/12680/ > >> > >> The author is Sam John as identified by Mess

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Timo Rothenpieler
Ok.  Then shouldn't it assign 'Sam John via ffmpeg-devel' as the author? It did do that for 'Oliver Collyer via ffmpeg-devel', the string wrongly substituted here. This looks like a parser failure in Patchwork. Gyan If the patch is sent with git send-email, the From: header of the E-Mail is

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Gyan
On 18-04-2019 03:25 PM, Timo Rothenpieler wrote: On 18/04/2019 09:49, Gyan wrote: Patchwork can incorrectly assign ownership. See https://patchwork.ffmpeg.org/patch/12680/ The author is Sam John as identified by Message ID as well as the From field in the headers, yet Patchwork attributes

Re: [FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Timo Rothenpieler
On 18/04/2019 09:49, Gyan wrote: Patchwork can incorrectly assign ownership. See https://patchwork.ffmpeg.org/patch/12680/ The author is Sam John as identified by Message ID as well as the From field in the headers, yet Patchwork attributes this patch to "Oliver Collyer via ffmpeg-devel", a

[FFmpeg-devel] Patchwork attribution

2019-04-18 Thread Gyan
Patchwork can incorrectly assign ownership. See https://patchwork.ffmpeg.org/patch/12680/ The author is Sam John as identified by Message ID as well as the From field in the headers, yet Patchwork attributes this patch to "Oliver Collyer via ffmpeg-devel", a name which appears nowhere in the