* get_maintainer.pl and git send-email: 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid
@ 2016-09-14 10:45 Rafał Miłecki
2016-09-14 14:13 ` Joe Perches
0 siblings, 1 reply; 3+ messages in thread
From: Rafał Miłecki @ 2016-09-14 10:45 UTC (permalink / raw)
To: Joe Perches, Linux Kernel Mailing List
Hi Joe,
Some time ago I started using git send-email with --cc-cmd and
get_maintainer.pl. I liked it and it worked fine until I tried to send
ubifs patch patch. I got this:
Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid
5.1.2 RFC-5321 address. j11sm4705544lfe.27 - gsmtp
I suspect there may be some problem with MAINTAINERS entries using braces.
Could you take a look at following try?
https://patchwork.ozlabs.org/patch/669853/
wget -O mtd.patch https://patchwork.ozlabs.org/patch/669853/mbox/
git send-email --cc-cmd="scripts/get_maintainer.pl" mtd.patch
It results in:
(cc-cmd) Adding cc: linux-mtd@lists.infradead.org (open list:UBI FILE
SYSTEM (UBIFS)) from: 'scripts/get_maintainer.pl'
and Cc-ing:
linux-mtd@lists.infradead.org) (open list:UBI FILE SYSTEM (UBIFS)
Is this something to fix at get_maintainer.pl level?
--
Rafał
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: get_maintainer.pl and git send-email: 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid
2016-09-14 10:45 get_maintainer.pl and git send-email: 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid Rafał Miłecki
@ 2016-09-14 14:13 ` Joe Perches
2016-09-14 14:32 ` Joe Perches
0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2016-09-14 14:13 UTC (permalink / raw)
To: Rafał Miłecki, Linux Kernel Mailing List
On Wed, 2016-09-14 at 12:45 +0200, Rafał Miłecki wrote:
> Hi Joe,
Hello Rafał
> Some time ago I started using git send-email with --cc-cmd and
> get_maintainer.pl. I liked it and it worked fine until I tried to send
> ubifs patch patch. I got this:
> Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
> > 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid
> 5.1.2 RFC-5321 address. j11sm4705544lfe.27 - gsmtp
>
> I suspect there may be some problem with MAINTAINERS entries using braces.
>
> Could you take a look at following try?
> https://patchwork.ozlabs.org/patch/669853/
>
> wget -O mtd.patch https://patchwork.ozlabs.org/patch/669853/mbox/
> git send-email --cc-cmd="scripts/get_maintainer.pl" mtd.patch
>
> It results in:
> > (cc-cmd) Adding cc: linux-mtd@lists.infradead.org (open list:UBI FILE
> SYSTEM (UBIFS)) from: 'scripts/get_maintainer.pl'
> and Cc-ing:
> linux-mtd@lists.infradead.org) (open list:UBI FILE SYSTEM (UBIFS)
>
> Is this something to fix at get_maintainer.pl level?
Not really.
At a minimum you need to add "--norolestats" to the cc-cmd invocation
--cc-cmd="./scripts/get_maintainer.pl --norolestats"
I use scripts to do this along with .gitconfig entries.
[sendemail]
chainreplyto = false
thread = false
suppresscc = self
cccmd = ~/bin/cc.sh
tocmd = ~/bin/to.sh
The scripts use --to-cmd to address the direct maintainers
and cc-cmd to address indirect maintainers and mailing lists.
$ cat ~/bin/to.sh
#!/bin/bash
opts="--nogit --nogit-fallback --norolestats --pattern-depth=1"
if [[ $(basename $1) =~ ^0000- ]] ; then
./scripts/get_maintainer.pl --nom $opts $(dirname $1)/*
else
maint=$(./scripts/get_maintainer.pl --nol $opts $1)
if [ "$maint" == "" ] ; then
echo "linux-kernel@vger.kernel.org"
else
echo "$maint"
fi
fi
$ cat ~/bin/cc.sh
#!/bin/bash
opts="--nogit --nogit-fallback --norolestats"
if [[ $(basename $1) =~ ^0000- ]] ; then
./scripts/get_maintainer.pl --nom $opts $(dirname $1)/*
else
./scripts/get_maintainer.pl $opts $1
fi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: get_maintainer.pl and git send-email: 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid
2016-09-14 14:13 ` Joe Perches
@ 2016-09-14 14:32 ` Joe Perches
0 siblings, 0 replies; 3+ messages in thread
From: Joe Perches @ 2016-09-14 14:32 UTC (permalink / raw)
To: Rafał Miłecki, Linux Kernel Mailing List
On Wed, 2016-09-14 at 07:13 -0700, Joe Perches wrote:
> The scripts use --to-cmd to address the direct maintainers
> and cc-cmd to address indirect maintainers and mailing lists.
I neglected to mention I use a separate directory for each patch series
so there are no other files in the directory that could be scanned with
the wildcard use of '$(dirname $1)/*'
$ git format-patch -o <new_directory> ...
> $ cat ~/bin/to.sh
> #!/bin/bash
>
> opts="--nogit --nogit-fallback --norolestats --pattern-depth=1"
>
> if [[ $(basename $1) =~ ^0000- ]] ; then
> ./scripts/get_maintainer.pl --nom $opts $(dirname $1)/*
> else
> maint=$(./scripts/get_maintainer.pl --nol $opts $1)
> if [ "$maint" == "" ] ; then
> > echo "linux-kernel@vger.kernel.org"
> else
> echo "$maint"
> fi
> fi
>
> $ cat ~/bin/cc.sh
> #!/bin/bash
>
> opts="--nogit --nogit-fallback --norolestats"
>
> if [[ $(basename $1) =~ ^0000- ]] ; then
> ./scripts/get_maintainer.pl --nom $opts $(dirname $1)/*
> else
> ./scripts/get_maintainer.pl $opts $1
> fi
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-14 14:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-14 10:45 get_maintainer.pl and git send-email: 5.1.2 The recipient address <linux-mtd@lists.infradead.org)> is not a valid Rafał Miłecki
2016-09-14 14:13 ` Joe Perches
2016-09-14 14:32 ` Joe Perches
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.