* [PATCH] send-pull-request: send all patches as the local user
@ 2011-01-21 21:32 Darren Hart
2011-01-28 1:12 ` Saul Wold
0 siblings, 1 reply; 2+ messages in thread
From: Darren Hart @ 2011-01-21 21:32 UTC (permalink / raw)
To: poky
Please pull the following patch from poky-contrib/dvhart/git-pull
Pull URL: git://git.pokylinux.org/poky-contrib.git
Branch: dvhart/git-pull
Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/git-pull
Thanks,
Darren Hart <dvhart@linux.intel.com>
When using sendmail to send patches, patches would appear to be from the
original author as git adds a From: header in the generated patches. This patch
changes this behavior to match that of git-send-email, where the email From:
header is that of the current sender (according to sendmail) and a "From:
Original Author <email>" line is inserted into the body of the message.
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
---
scripts/send-pull-request | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/scripts/send-pull-request b/scripts/send-pull-request
index 9872c0d..af7ce2a 100755
--- a/scripts/send-pull-request
+++ b/scripts/send-pull-request
@@ -140,13 +140,18 @@ if [ "$cont" == "y" ] || [ "$cont" == "Y" ]; then
# appending them to the sendmail command as -- $TO $CC has
# proven to be an exercise in futility.
#
+ # Clear the From header, leaving it up to sendmail to insert an
+ # appropriate one. Insert the original sender (per git) into the
+ # body of the message.
+ #
# Use tail to remove the email envelope from git or formail as
# msmtp (sendmail) would choke on them.
#
# Modify the patch date for sequential delivery, but retain the
# original date as "Old-Date".
DATE=$(date +"%a, %d %b %Y %k:%M:%S %z")
- cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -i "Date: $DATE" | tail -n +2 | sendmail -t
+ GIT_FROM=$(cat $PATCH | formail -X "From:")
+ cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -I "From:" -i "Date: $DATE" | sed "0,/^$/s/^$/\n$GIT_FROM\n/" | tail -n +2 | sendmail -t
if [ $? -eq 1 ]; then
ERROR=1
fi
--
1.7.1
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] send-pull-request: send all patches as the local user
2011-01-21 21:32 [PATCH] send-pull-request: send all patches as the local user Darren Hart
@ 2011-01-28 1:12 ` Saul Wold
0 siblings, 0 replies; 2+ messages in thread
From: Saul Wold @ 2011-01-28 1:12 UTC (permalink / raw)
To: Darren Hart; +Cc: poky
On 01/21/2011 01:32 PM, Darren Hart wrote:
> Please pull the following patch from poky-contrib/dvhart/git-pull
>
> Pull URL: git://git.pokylinux.org/poky-contrib.git
> Branch: dvhart/git-pull
> Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/git-pull
>
> Thanks,
> Darren Hart<dvhart@linux.intel.com>
>
>
> When using sendmail to send patches, patches would appear to be from the
> original author as git adds a From: header in the generated patches. This patch
> changes this behavior to match that of git-send-email, where the email From:
> header is that of the current sender (according to sendmail) and a "From:
> Original Author<email>" line is inserted into the body of the message.
>
> Signed-off-by: Darren Hart<dvhart@linux.intel.com>
> ---
> scripts/send-pull-request | 7 ++++++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
>
> diff --git a/scripts/send-pull-request b/scripts/send-pull-request
> index 9872c0d..af7ce2a 100755
> --- a/scripts/send-pull-request
> +++ b/scripts/send-pull-request
> @@ -140,13 +140,18 @@ if [ "$cont" == "y" ] || [ "$cont" == "Y" ]; then
> # appending them to the sendmail command as -- $TO $CC has
> # proven to be an exercise in futility.
> #
> + # Clear the From header, leaving it up to sendmail to insert an
> + # appropriate one. Insert the original sender (per git) into the
> + # body of the message.
> + #
> # Use tail to remove the email envelope from git or formail as
> # msmtp (sendmail) would choke on them.
> #
> # Modify the patch date for sequential delivery, but retain the
> # original date as "Old-Date".
> DATE=$(date +"%a, %d %b %Y %k:%M:%S %z")
> - cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -i "Date: $DATE" | tail -n +2 | sendmail -t
> + GIT_FROM=$(cat $PATCH | formail -X "From:")
> + cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -I "From:" -i "Date: $DATE" | sed "0,/^$/s/^$/\n$GIT_FROM\n/" | tail -n +2 | sendmail -t
> if [ $? -eq 1 ]; then
> ERROR=1
> fi
Pulled into Master
Thanks
Sau!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-01-28 1:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-21 21:32 [PATCH] send-pull-request: send all patches as the local user Darren Hart
2011-01-28 1:12 ` Saul Wold
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.