[U-Boot] exclude email by patman
Simon,
Is there a way to exclude specific emails from the CC list when using patman? The no-longer-existed email address got rejected by SMTP server.
York
Dear York,
In message AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com you wrote:
Is there a way to exclude specific emails from the CC list when using patman? The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Best regards,
Wolfgang Denk
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com you wrote:
Is there a way to exclude specific emails from the CC list when using patman? The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email address, it is not the format of email, but the server rejects the address because the account has been disabled/deleted. It is not the knowledge patman for git send-email can acquire. My current workaround is to send patches manually using git send-email.
York
On Tue, Feb 09, 2016 at 05:08:35PM +0000, york sun wrote:
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com you wrote:
Is there a way to exclude specific emails from the CC list when using patman? The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email address, it is not the format of email, but the server rejects the address because the account has been disabled/deleted. It is not the knowledge patman for git send-email can acquire. My current workaround is to send patches manually using git send-email.
No, I think you are on the same page. Wolfgang is saying we shouldn't have dead email addresses on file (... where patman picks them up, ie MAINTAINERS). In your case, it should only fail on bad freescale/nxp emails, yes? Maybe there's more stuff to poke people about and/or drop?
On 02/09/2016 09:13 AM, Tom Rini wrote:
On Tue, Feb 09, 2016 at 05:08:35PM +0000, york sun wrote:
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com you wrote:
Is there a way to exclude specific emails from the CC list when using patman? The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email address, it is not the format of email, but the server rejects the address because the account has been disabled/deleted. It is not the knowledge patman for git send-email can acquire. My current workaround is to send patches manually using git send-email.
No, I think you are on the same page. Wolfgang is saying we shouldn't have dead email addresses on file (... where patman picks them up, ie MAINTAINERS). In your case, it should only fail on bad freescale/nxp emails, yes? Maybe there's more stuff to poke people about and/or drop?
Good we are on the same page. Let me be specific on this issue I am facing. For the RFC patch I posted yesterday http://patchwork.ozlabs.org/patch/580577/, if I run script/get_maintainer.pl,
$ perl scripts/get_maintainer.pl 0001-arm-ls1021aqds-Convert-to-driver-model-and-enable-se.patch Albert Aribaud albert.u.boot@aribaud.net (maintainer:ARM) Alison Wang alison.wang@freescale.com (maintainer:LS1021AQDS BOARD) Hans de Goede hdegoede@redhat.com (commit_signer:34/85=40%,authored:21/85=25%,added_lines:97/202=48%,removed_lines:14/47=30%) Simon Glass sjg@chromium.org (commit_signer:18/85=21%,authored:5/85=6%,commit_signer:1/2=50%,commit_signer:5/6=83%) Ian Campbell ijc@hellion.org.uk (commit_signer:18/85=21%) Stefan Roese sr@denx.de (commit_signer:9/85=11%,authored:5/85=6%,added_lines:13/202=6%,removed_lines:3/47=6%) York Sun york.sun@nxp.com (commit_signer:8/85=9%,commit_signer:1/2=50%,authored:1/2=50%,commit_signer:1/6=17%,authored:1/6=17%,removed_lines:216/221=98%) Masahiro Yamada yamada.masahiro@socionext.com (authored:5/85=6%,removed_lines:5/47=11%) Lokesh Vutla lokeshvutla@ti.com (removed_lines:3/47=6%) Bin Meng bmeng.cn@gmail.com (commit_signer:1/2=50%,authored:1/2=50%) Haikun Wang Haikun.Wang@freescale.com (commit_signer:5/6=83%,authored:5/6=83%,added_lines:221/221=100%) Jagan Teki jteki@openedev.com (commit_signer:1/6=17%) u-boot@lists.denx.de (open list)
In this list, Haikun Wang is no longer with Freescale/NXP. Since he is not a maintainer, there is nothing to fix. His address is in the commit history and nothing can/should be done to change that.
I understand this issue is related to my SMTP server which checks local recipients before accepting emails. I can't be the only one having this trouble.
It is not a huge deal if this can't be fixed. I asked to see if there is a quick fix.
York
On Tue, Feb 9, 2016 at 10:08 AM, york sun york.sun@nxp.com wrote:
On 02/09/2016 09:13 AM, Tom Rini wrote:
On Tue, Feb 09, 2016 at 05:08:35PM +0000, york sun wrote:
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message <
AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com> you wrote:
Is there a way to exclude specific emails from the CC list when using
patman?
The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email
address, it
is not the format of email, but the server rejects the address because
the
account has been disabled/deleted. It is not the knowledge patman for
git
send-email can acquire. My current workaround is to send patches
manually using
git send-email.
No, I think you are on the same page. Wolfgang is saying we shouldn't have dead email addresses on file (... where patman picks them up, ie MAINTAINERS). In your case, it should only fail on bad freescale/nxp emails, yes? Maybe there's more stuff to poke people about and/or drop?
Good we are on the same page. Let me be specific on this issue I am facing. For the RFC patch I posted yesterday http://patchwork.ozlabs.org/patch/580577/, if I run script/get_maintainer.pl,
$ perl scripts/get_maintainer.pl 0001-arm-ls1021aqds-Convert-to-driver-model-and-enable-se.patch Albert Aribaud albert.u.boot@aribaud.net (maintainer:ARM) Alison Wang alison.wang@freescale.com (maintainer:LS1021AQDS BOARD) Hans de Goede hdegoede@redhat.com
(commit_signer:34/85=40%,authored:21/85=25%,added_lines:97/202=48%,removed_lines:14/47=30%) Simon Glass sjg@chromium.org
(commit_signer:18/85=21%,authored:5/85=6%,commit_signer:1/2=50%,commit_signer:5/6=83%) Ian Campbell ijc@hellion.org.uk (commit_signer:18/85=21%) Stefan Roese sr@denx.de
(commit_signer:9/85=11%,authored:5/85=6%,added_lines:13/202=6%,removed_lines:3/47=6%) York Sun york.sun@nxp.com
(commit_signer:8/85=9%,commit_signer:1/2=50%,authored:1/2=50%,commit_signer:1/6=17%,authored:1/6=17%,removed_lines:216/221=98%) Masahiro Yamada yamada.masahiro@socionext.com (authored:5/85=6%,removed_lines:5/47=11%) Lokesh Vutla lokeshvutla@ti.com (removed_lines:3/47=6%) Bin Meng bmeng.cn@gmail.com (commit_signer:1/2=50%,authored:1/2=50%) Haikun Wang Haikun.Wang@freescale.com (commit_signer:5/6=83%,authored:5/6=83%,added_lines:221/221=100%) Jagan Teki jteki@openedev.com (commit_signer:1/6=17%) u-boot@lists.denx.de (open list)
In this list, Haikun Wang is no longer with Freescale/NXP. Since he is not a maintainer, there is nothing to fix. His address is in the commit history and nothing can/should be done to change that.
I understand this issue is related to my SMTP server which checks local recipients before accepting emails. I can't be the only one having this trouble.
It is not a huge deal if this can't be fixed. I asked to see if there is a quick fix.
York
yes - I am having this issue too; and I have also complaints about me cc'ing too many people on my patches! So, my work-around is: - when patman executes "git send-email --annotate ...", it gives me the chance to review my patches (in my editor 'vi') - while this is occurring, from my editor ('vi'), I run ":e /tmp/patman.*" (there is always only one file...) and I manually update the list of emails.... - then after saving that file, the "git send-email" sends to the appropriate people.
Ugly -- but it works! Thanks, Steve
_______________________________________________
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Hi,
On 9 February 2016 at 11:56, Steve Rae steve.rae@broadcom.com wrote:
On Tue, Feb 9, 2016 at 10:08 AM, york sun york.sun@nxp.com wrote:
On 02/09/2016 09:13 AM, Tom Rini wrote:
On Tue, Feb 09, 2016 at 05:08:35PM +0000, york sun wrote:
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message <
AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com> you wrote:
Is there a way to exclude specific emails from the CC list when using
patman?
The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email
address, it
is not the format of email, but the server rejects the address because
the
account has been disabled/deleted. It is not the knowledge patman for
git
send-email can acquire. My current workaround is to send patches
manually using
git send-email.
No, I think you are on the same page. Wolfgang is saying we shouldn't have dead email addresses on file (... where patman picks them up, ie MAINTAINERS). In your case, it should only fail on bad freescale/nxp emails, yes? Maybe there's more stuff to poke people about and/or drop?
Good we are on the same page. Let me be specific on this issue I am facing. For the RFC patch I posted yesterday http://patchwork.ozlabs.org/patch/580577/, if I run script/get_maintainer.pl,
$ perl scripts/get_maintainer.pl 0001-arm-ls1021aqds-Convert-to-driver-model-and-enable-se.patch Albert Aribaud albert.u.boot@aribaud.net (maintainer:ARM) Alison Wang alison.wang@freescale.com (maintainer:LS1021AQDS BOARD) Hans de Goede hdegoede@redhat.com
(commit_signer:34/85=40%,authored:21/85=25%,added_lines:97/202=48%,removed_lines:14/47=30%) Simon Glass sjg@chromium.org
(commit_signer:18/85=21%,authored:5/85=6%,commit_signer:1/2=50%,commit_signer:5/6=83%) Ian Campbell ijc@hellion.org.uk (commit_signer:18/85=21%) Stefan Roese sr@denx.de
(commit_signer:9/85=11%,authored:5/85=6%,added_lines:13/202=6%,removed_lines:3/47=6%) York Sun york.sun@nxp.com
(commit_signer:8/85=9%,commit_signer:1/2=50%,authored:1/2=50%,commit_signer:1/6=17%,authored:1/6=17%,removed_lines:216/221=98%) Masahiro Yamada yamada.masahiro@socionext.com (authored:5/85=6%,removed_lines:5/47=11%) Lokesh Vutla lokeshvutla@ti.com (removed_lines:3/47=6%) Bin Meng bmeng.cn@gmail.com (commit_signer:1/2=50%,authored:1/2=50%) Haikun Wang Haikun.Wang@freescale.com (commit_signer:5/6=83%,authored:5/6=83%,added_lines:221/221=100%) Jagan Teki jteki@openedev.com (commit_signer:1/6=17%) u-boot@lists.denx.de (open list)
In this list, Haikun Wang is no longer with Freescale/NXP. Since he is not a maintainer, there is nothing to fix. His address is in the commit history and nothing can/should be done to change that.
I understand this issue is related to my SMTP server which checks local recipients before accepting emails. I can't be the only one having this trouble.
It is not a huge deal if this can't be fixed. I asked to see if there is a quick fix.
York
yes - I am having this issue too; and I have also complaints about me cc'ing too many people on my patches! So, my work-around is:
- when patman executes "git send-email --annotate ...", it gives me the
chance to review my patches (in my editor 'vi')
- while this is occurring, from my editor ('vi'), I run ":e /tmp/patman.*"
(there is always only one file...) and I manually update the list of emails....
- then after saving that file, the "git send-email" sends to the
appropriate people.
Ugly -- but it works! Thanks, Steve
Also -m disables the get_maintainer feature altogether.
Do we need to add an email blacklist to work around this problem? I get it a lot but it doesn't really bother me.
Regards, Simon
On 02/12/2016 07:54 AM, Simon Glass wrote:
Hi,
On 9 February 2016 at 11:56, Steve Rae steve.rae@broadcom.com wrote:
On Tue, Feb 9, 2016 at 10:08 AM, york sun york.sun@nxp.com wrote:
On 02/09/2016 09:13 AM, Tom Rini wrote:
On Tue, Feb 09, 2016 at 05:08:35PM +0000, york sun wrote:
On 02/09/2016 12:00 AM, Wolfgang Denk wrote:
Dear York,
In message <
AM4PR0401MB1732EC48981EED9B70CE23859AD50@AM4PR0401MB1732.eurprd04.prod.outlook.com> you wrote:
> > Is there a way to exclude specific emails from the CC list when using
patman?
> The no-longer-existed email address got rejected by SMTP server.
That';s just papering over the problems, and thus the wrong approach.
The correct thing to do is having the wrong addresses fixed or removed.
Dear Wolfgang,
I don't think we are on the same page. When I mention invalid email
address, it
is not the format of email, but the server rejects the address because
the
account has been disabled/deleted. It is not the knowledge patman for
git
send-email can acquire. My current workaround is to send patches
manually using
git send-email.
No, I think you are on the same page. Wolfgang is saying we shouldn't have dead email addresses on file (... where patman picks them up, ie MAINTAINERS). In your case, it should only fail on bad freescale/nxp emails, yes? Maybe there's more stuff to poke people about and/or drop?
Good we are on the same page. Let me be specific on this issue I am facing. For the RFC patch I posted yesterday http://patchwork.ozlabs.org/patch/580577/, if I run script/get_maintainer.pl,
$ perl scripts/get_maintainer.pl 0001-arm-ls1021aqds-Convert-to-driver-model-and-enable-se.patch Albert Aribaud albert.u.boot@aribaud.net (maintainer:ARM) Alison Wang alison.wang@freescale.com (maintainer:LS1021AQDS BOARD) Hans de Goede hdegoede@redhat.com
(commit_signer:34/85=40%,authored:21/85=25%,added_lines:97/202=48%,removed_lines:14/47=30%) Simon Glass sjg@chromium.org
(commit_signer:18/85=21%,authored:5/85=6%,commit_signer:1/2=50%,commit_signer:5/6=83%) Ian Campbell ijc@hellion.org.uk (commit_signer:18/85=21%) Stefan Roese sr@denx.de
(commit_signer:9/85=11%,authored:5/85=6%,added_lines:13/202=6%,removed_lines:3/47=6%) York Sun york.sun@nxp.com
(commit_signer:8/85=9%,commit_signer:1/2=50%,authored:1/2=50%,commit_signer:1/6=17%,authored:1/6=17%,removed_lines:216/221=98%) Masahiro Yamada yamada.masahiro@socionext.com (authored:5/85=6%,removed_lines:5/47=11%) Lokesh Vutla lokeshvutla@ti.com (removed_lines:3/47=6%) Bin Meng bmeng.cn@gmail.com (commit_signer:1/2=50%,authored:1/2=50%) Haikun Wang Haikun.Wang@freescale.com (commit_signer:5/6=83%,authored:5/6=83%,added_lines:221/221=100%) Jagan Teki jteki@openedev.com (commit_signer:1/6=17%) u-boot@lists.denx.de (open list)
In this list, Haikun Wang is no longer with Freescale/NXP. Since he is not a maintainer, there is nothing to fix. His address is in the commit history and nothing can/should be done to change that.
I understand this issue is related to my SMTP server which checks local recipients before accepting emails. I can't be the only one having this trouble.
It is not a huge deal if this can't be fixed. I asked to see if there is a quick fix.
York
yes - I am having this issue too; and I have also complaints about me cc'ing too many people on my patches! So, my work-around is:
- when patman executes "git send-email --annotate ...", it gives me the
chance to review my patches (in my editor 'vi')
- while this is occurring, from my editor ('vi'), I run ":e /tmp/patman.*"
(there is always only one file...) and I manually update the list of emails....
- then after saving that file, the "git send-email" sends to the
appropriate people.
Ugly -- but it works! Thanks, Steve
Also -m disables the get_maintainer feature altogether.
Do we need to add an email blacklist to work around this problem? I get it a lot but it doesn't really bother me.
Simon,
I consider get_maintainer is one of the good feature in patman. It helps when generating large patchset. I'd rather not to disable it.
A locally maintained black list works. An exclusion flag also works. My 2 cents.
York
participants (5)
-
Simon Glass -
Steve Rae -
Tom Rini -
Wolfgang Denk -
york sun