2012/9/3 Junio C Hamano gits...@pobox.com:
Marco Stornelli marco.storne...@gmail.com writes:
I tried the Johannes's script, but it seems it doesn't work well with
the pattern of format-patch (To: mail1,\n mail2,\n
mailN). The multilines are not well managed.
I am guessing that the reason
Il 04/09/2012 17:49, Junio C Hamano ha scritto:
Marco Stornelli marco.storne...@gmail.com writes:
2012/9/4 Junio C Hamano gits...@pobox.com:
I would expect, at least when you are responding to an existing
message, some of them are filled already (and if so, I think appp.sh
wants to know
2012/9/2 Junio C Hamano gits...@pobox.com:
Marco Stornelli marco.storne...@gmail.com writes:
Il 01/09/2012 15:59, Johannes Sixt ha scritto:
Look how you write:
perl -e '... $ENV{'PATCHTMP'} ...'
That is, perl actually sees this script:
... $ENV{PATCHTMP} ...
(no quotes around
Il 02/09/2012 22:42, Junio C Hamano ha scritto:
Marco Stornelli marco.storne...@gmail.com writes:
Il 01/09/2012 15:59, Johannes Sixt ha scritto:
Look how you write:
perl -e '... $ENV{'PATCHTMP'} ...'
That is, perl actually sees this script:
... $ENV{PATCHTMP} ...
(no quotes
in the email
header.
Added comments for point 1), added parsing of To: for point 2) and
added parsing of Cc: in the email header for point 3), removing the
Cc: parsing from commit message.
Signed-off-by: Marco Stornelli marco.storne...@gmail.com
---
v6: replaced single quotas with double quotas
Il 31/08/2012 23:35, Johannes Sixt ha scritto:
Am 31.08.2012 16:09, schrieb Marco Stornelli:
+CCS=`perl -e 'local $/=undef; open FILE, $ENV{'PATCHTMP'}; $text=FILE;
+close FILE; $addr = $1 if $text =~ /Cc: (.*?(,\n .*?)*)\n/s; $addr =~ s/\n//g;
+print $addr;'`
The quoting is broken
Il 31/08/2012 19:08, Junio C Hamano ha scritto:
Marco Stornelli marco.storne...@gmail.com writes:
The current script has got the following problems:
1) It doesn't work if the language used by Thunderbird is not english;
2) The field To: filled by format-patch is not evaluated;
3) The field Cc
in the email
header.
Added comments for point 1), added parsing of To: for point 2) and
added parsing of Cc: in the email header for point 3), removing the
Cc: parsing from commit message.
Signed-off-by: Marco Stornelli marco.storne...@gmail.com
---
v5: fixed comments by Junio C Hamano
v4: create
Il 01/09/2012 15:59, Johannes Sixt ha scritto:
Am 01.09.2012 09:43, schrieb Marco Stornelli:
Il 31/08/2012 23:35, Johannes Sixt ha scritto:
Am 31.08.2012 16:09, schrieb Marco Stornelli:
+CCS=`perl -e 'local $/=undef; open FILE, $ENV{'PATCHTMP'};
$text=FILE;
+close FILE; $addr = $1 if $text
Il 30/08/2012 19:00, Junio C Hamano ha scritto:
Marco Stornelli marco.storne...@gmail.com writes:
The current script has got the following problems:
1) It doesn't work if the language used by Thunderbird is not English;
2) The field To: filled by format-patch is not evaluated;
3) The field Cc
in the email
header.
Added comments for point 1), added parsing of To: for point 2) and
added parsing of Cc: in the email header for point 3), removing the
Cc: parsing from commit message.
Signed-off-by: Marco Stornelli marco.storne...@gmail.com
---
v4: create a tmp file to allow correct perl
in the email
header. In addition, in the commit message we can find several tags
(acked-by, tested-by, reported-by...), so it'd better to use only the
information provided by format-patch.
Added comments for point 1). Fixed point 2) and 3).
Signed-off-by: Marco Stornelli marco.storne...@gmail.com
by format-patch in the email
header.
Added comments for point 1). Fixed point 2) and 3).
Signed-off-by: Marco Stornelli marco.storne...@gmail.com
---
v2: changed the commit message to reflect better the script implementation
contrib/thunderbird-patch-inline/appp.sh | 18 ++
1 files
13 matches
Mail list logo