On 14/02/2019 14:34, Flavio Leitner wrote:
>
> Hi Ben,
>
> This is another patch with From: field altered to be
> ovs-dev@openvswitch.org.
>
> [...]
> From: Matthias May via dev <ovs-dev@openvswitch.org>
> Reply-To: Matthias May <matthias....@neratec.com>
>
> If the patch gets applied as is, the commit's Author will have the
> wrong email.
>
> Just FYI because before Sriharsha and Neal had the same issue and we
> haven't identified the root cause back then.
>
> Matthias, could you describe to where exactly you sent out the
> patch? ovs-dev@openvswitch.org or d...@openvswitch.org or something
> else?
>
> Thanks,
> fbl
>
>
> On Thu, Feb 14, 2019 at 12:18:11PM +0100, Matthias May via dev wrote:
>> On 14/02/2019 11:59, 0-day Robot wrote:
>>> Bleep bloop. Greetings Matthias May via dev, I am a robot and I have tried
>>> out your patch.
>>> Thanks for your contribution.
>>>
>>> I encountered some error that I wasn't expecting. See the details below.
>>>
>>>
>>> checkpatch:
>>> ERROR: Author should not be mailing list.
>>> Lines checked: 47, Warnings: 0, Errors: 1
>>>
>>>
>>> Please check this out. If you feel there has been an error, please email
>>> acon...@bytheb.org
>>>
>>> Thanks,
>>> 0-day Robot
>>> _______________________________________________
>>> dev mailing list
>>> d...@openvswitch.org
>>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>>
>>
>> What is this supposed to mean?
>> Is the Signed-off-by not enough?
>>
>> BR
>> Matthias
>> _______________________________________________
>> dev mailing list
>> d...@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Hi Flavio
Some info:
* git version 2.20.1
* (Reproduced) trace of what I entered:
```
maym@CHD500279:~/git/ovs$ git send-email send-email/ --no-chain-reply
send-email/0001-rstp-add-ability-to-receive-VLAN-tagged-BPDUs.patch
To whom should the emails be sent (if anyone)? ovs-dev@openvswitch.org
Message-ID to be used as In-Reply-To for the first email (if any)?
(mbox) Adding cc: Matthias May <matthias....@neratec.com> from line 'From:
Matthias May <matthias....@neratec.com>'
(body) Adding cc: Matthias May <matthias....@neratec.com> from line
'Signed-off-by: Matthias May <matthias....@neratec.com>'
From: Matthias May <matthias....@neratec.com>
To: ovs-dev@openvswitch.org
Cc: Matthias May <matthias....@neratec.com>
Subject: [PATCH] rstp: add ability to receive VLAN-tagged BPDUs
Date: Thu, 14 Feb 2019 15:01:34 +0100
Message-Id: <20190214140134.16754-1-matthias....@neratec.com>
X-Mailer: git-send-email 2.20.1
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
The Cc list above has been expanded by additional
addresses found in the patch commit message. By default
send-email prompts before sending whenever this occurs.
This behavior is controlled by the sendemail.confirm
configuration setting.
For additional information, run 'git send-email --help'.
To retain the current behavior, but squelch this message,
run 'git config --global sendemail.confirm auto'.
Send this email? ([y]es|[n]o|[e]dit|[q]uit|[a]ll): y
```
This obviously has a different Message-Id then the mail sent before.
BR
Matthias
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
