On Mon, Jun 28, 2021 at 07:10:01AM -0700, Tyler Retzlaff wrote:

On Mon, Jun 28, 2021 at 01:35:09PM +0300, Andrew Rybchenko wrote:

quoted

On 6/28/21 1:01 PM, Andrew Rybchenko wrote:

quoted

On 6/24/21 1:34 AM, Jie Zhou wrote:

quoted

Enable building libraries that testpmd depends on for Windows

Signed-off-by: Jie Zhou <redacted>
Signed-off-by: Jie Zhou <redacted>

Why are two a bit difference Signed-off-by used in these
patches? Can we drop the first one and keep the second one
which matches E-mail From?

In fact looking at git log I see both E-mail addresses
present, but it looks like the first one (w/o linux.) is
used more often.

yes, the CI system rejects the patch without the first because it
doesn't match the original commit. it also rejects without the second
because it needs to match the dev email list subscriber mail so both
have been provided to pass CI.

i think future patches submitted won't be an issue because git local
config for the dpdk clone has been changed to match the second which is
also the subscription mail but changing the config doesn't re-write
the history on the commits (and it shouldn't).

Yes, the first one was originally manually added due to some CI warning. 
Double checked git config on local dpdk clone and ensured the user.email 
matches the second one. Will remove the first one in V15 and investigate 
if CI still complains.

Thanks.