summaryrefslogtreecommitdiff
path: root/scripts/send-pull-request
AgeCommit message (Collapse)AuthorFiles
2016-12-07scripts/send-pull-request: Avoid multiple chain headersPatrick Ohly1
When creating a patch set with cover letter using the send-pull-request script, both the "In-Reply-To" and "References" headers are appended twice in patch 2 and subsequent. That's because git-format-patch already inserted them and then git-send-email repeats that. Suppressing mail threading in git-send-email with --no-thread avoids the problem and is the right solution because it works regardless whether git-send-email is called once or twicee. Repeating these headers is a violation of RFC 2822 and can confuse mail programs. For example, Patchwork does not detect a patch series problem when there are these extra headers. [YOCTO #10718] Signed-off-by: Patrick Ohly <patrick.ohly@intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com>
2013-02-06send-pull-request: add extra CC argumentMartin Jansa1
* useful e.g. when sending pull-request to release branch with extra CC for release maintainer Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
2012-08-02send-pull-request: Make -a really imply -cDarren Hart1
The switch statement does not fall through as the comment suggests. This results in -a not implying -c as was intended. Add the two lines from -c to -a to achieve the intended behavior. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2011-12-12scripts/send-pull-request: fix typoOtavio Salvador1
s/No cover letter will be send./No cover letter will be sent./ Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
2011-12-09scripts/send-pull-request: allow sending of patches without a cover letterOtavio Salvador1
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
2011-07-19send-pull-request: default to --supress-cc=allDarren Hart1
Unless the user specifies -c, don't pull in any email addresses from the patches in the series. This avoids having to remove the email addresses from patches being pulled in from upstream. If you want the email addresses on the patch to be added, continue to use the -c option as before. Signed-off-by: Darren Hart <dvhart@linux.intel.com>
2011-06-02send-pull-request: fix a small typo that fails the scriptDexuan Cui1
Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>
2011-05-23send-pull-request: drop sendemail checksOtavio Salvador1
git send-email has the correct check on it. Basically the From is taken from the git 'user' and 'email' config values and in case 'sendemail.smtpserver' is not provided it defaults to use local sendmail command. Signed-off-by: Otavio Salvador <otavio@ossystems.com.br> Acked-by: Darren Hart <dvhart@linux.intel.com>
2011-05-19*pull-request: add copyright, license, and descriptionsDarren Hart1
Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: streamline git-send-email usageDarren Hart1
The script was sending one patch at a time, which defeats the internal confirmation mechanism of git-send-email (which would otherwise allow the user to send all patches or abort immediately). Rework the sending logic to use no more than two commands. Use two commands when the cover letter is to be sent to all recipients with the -a argument. Otherwise, send all patches via the same command. The script duplicates git's send confirmation, eliminate that. Reported-by: Khem Raj <raj.khem@gmail.com> Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Joshua Lock <josh@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Khem Raj <raj.khem@gmail.com> Cc: Joshua Lock <josh@linux.intel.com> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: verify git sendemail configDarren Hart1
Perform a quick sanity check to be able to direct users to configure git.sendemail if they haven't yet. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: don't send all patches to everyone even with -aDarren Hart1
Rather than sending every patch to every recipient of the entire series when -a is used, only send the cover letter to everyone and use git's --signed-off-by-cc feature to generate an auto cc list for the individual patches. Add a -c option to use --signed-off-by-cc to auto cc recipeients at the individual patch level. This is implied by -a. Using git to harvest the Cc list means only collecting Signed-off-by and Cc lines, rather than the more generic *-by lines previously. This is a fair trade-off for significantly reduced complexity. If users want to add Acked-by and Tested-by lines and want to use the -a feature, they should include those recipients as Cc lines as well. Now that we rely on git for auto-cc for the individual patches, make sure the user is prompted before sending each patch by forcing --confirm=always. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Khem Raj <raj.khem@gmail.com> Cc: Koen Kooi <koen@dominion.thruhere.net> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: fix greedy auto-cc regexDarren Hart1
A greedy regular expression caused emails to be harvested from patches that were quoted in the commit message. Ensure only tags that start at the beginning of the line are considered for harvesting. NOTE: users are still responsible for verifying the recipients list and to ensure they do not spam people! Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Bruce Ashfield <bruce.ashfield@windriver.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Richard Purdie <richard.purdie@linuxfoundation.org> Cc: Saul Wold <sgw@linux.intel.com> Cc: Bruce Ashfield <bruce.ashfield@windriver.com> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: remove local mta supportDarren Hart1
There is no real value in supporting sendmail directly when git can be configured to use it. The script used to generate the pull request mails relies heavily on git, so doing so here does not impose any additional dependencies and it greatly reduces the complexity of this script. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-05-19send-pull-request: whitespace cleanupDarren Hart1
Indent with tabs, not spaces, for consistency with other bash scripts. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Otavio Salvador <otavio@ossystems.com.br>
2011-03-16send-pull-request: Fetch TO email address from git config if specifiedKhem Raj1
Usually people using git send-email has git config sendmail.to configured to the usual mailing list or person so we harness that here. Signed-off-by: Khem Raj <raj.khem@gmail.com> Acked-by: Darren Hart <dvhart@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2011-01-26send-pull-request: ensure a proper FROM header is includedDarren Hart1
Commit 94629f2521711055b412f954af19e48b9bda6e50 removes the FROM header when sending via sendmail to avoid sending mail as the original change committer (as opposed to the local user). This resulted in mail going out without any FROM header, which some mailing lists correct by adding the *bounce address as the FROM. Correct this by reading FROM from the environment, from a new -f argument, or from the git user.name and user.email config settings, in that order of preference. Also display the FROM that will be used prior to the send confirmation. This has no effect if the -g (send via git) argument is specified, other than printing the git sendemail.from config setting. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Reported-by: Saul Wold <saul.wold@intel.com>
2011-01-24send-pull-request: send all patches as the local userDarren Hart1
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>
2011-01-24send-pull-request: unset $TO and $CC from environmentScott Garman1
Darren Hart and I discovered that when $CC is set (which our meta-toolchain environment script sets up), the value leaks into the use of this script. Unsetting $TO as well just to be thorough. Signed-off-by: Scott Garman <scott.a.garman@intel.com>
2010-12-23send-pull-request: allow users to select git-send-email or sendmailDarren Hart1
Some users find it easier to use their git sendmail setup over a local MTA to deliver mail with the send-pull-request script. If you would like to do this, please read the git-send-email man page and set the relevant entries in your git config. In particular, be sure to set sendemail.from to avoid being asked each time. Reported-by: Khem Raj <raj.khem@gmail.com> CC: Bruce Ashfield <bruce.ashfield@windriver.com> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
2010-12-21send-pull-request: Use current date in mail headersDarren Hart1
Some users experience problems viewing the pull requests as a sequential mail series due to the script using the git commit date for the patches and today's date for the cover letter. Address this by renaming the email Date: header to Old-Date: and adding a new Date: header with a current timestamp. Signed-off-by: Darren Hart <dvhart@linux.intel.com> Reported-by: Gary Thomas <gary@mlbassoc.com> Cc: Josh Lock <josh@linux.intel.com>
2010-11-10git-pull: add send-pull-request scriptDarren Hart1
send-pull-request facilitates sending pull requests generated by create-pull-request. The primary role of this script is to harvest email addresses from the patches and send them out. A working installation of sendmail (exim, postfix, msmtp, etc.) is required to use this script. You can explicitly specify To addresses with the -t option. As this can be tedious, the -a option will scan all the patches for To, CC, and *-by lines and the collected addresses to the To and CC headers for each patch. This script uses an identical recipients list for every patch, including the cover letter. This is by design. Existing tools will auto-generate the CC header for individual patches, but since they don't apply it to the other patches, the recipients can lack the necessary context to provide a meaningful review. This is especially true of the cover letter. The pull directory generated by the create-pull-request script is specified using the -p option. Signed-off-by: Darren Hart <dvhart@linux.intel.com> CC: Nitin A Kamble <nitin.a.kamble@intel.com> CC: Richard Purdie <rpurdie@linux.intel.com> CC: Saul Wold <saul.wold@intel.com> CC: Bruce Ashfield <bruce.ashfield@windriver.com>