summaryrefslogtreecommitdiff
path: root/scripts/send-pull-request
Commit message (Collapse)AuthorAgeFilesLines
* *pull-request: add copyright, license, and descriptionsDarren Hart2011-05-191-0/+25
| | | | | | Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: Otavio Salvador <otavio@ossystems.com.br> Cc: Otavio Salvador <otavio@ossystems.com.br>
* send-pull-request: streamline git-send-email usageDarren Hart2011-05-191-42/+26
| | | | | | | | | | | | | | | | | | | | 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>
* send-pull-request: verify git sendemail configDarren Hart2011-05-191-0/+15
| | | | | | | | | 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>
* send-pull-request: don't send all patches to everyone even with -aDarren Hart2011-05-191-28/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* send-pull-request: fix greedy auto-cc regexDarren Hart2011-05-191-1/+1
| | | | | | | | | | | | | | | | | 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>
* send-pull-request: remove local mta supportDarren Hart2011-05-191-87/+21
| | | | | | | | | | | | 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>
* send-pull-request: whitespace cleanupDarren Hart2011-05-191-125/+125
| | | | | | | | 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>
* send-pull-request: Fetch TO email address from git config if specifiedKhem Raj2011-03-161-6/+15
| | | | | | | | | | 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>
* send-pull-request: ensure a proper FROM header is includedDarren Hart2011-01-261-3/+26
| | | | | | | | | | | | | | | | | | | 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>
* send-pull-request: send all patches as the local userDarren Hart2011-01-241-1/+6
| | | | | | | | | | 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>
* send-pull-request: unset $TO and $CC from environmentScott Garman2011-01-241-0/+4
| | | | | | | | | 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>
* send-pull-request: allow users to select git-send-email or sendmailDarren Hart2010-12-231-19/+50
| | | | | | | | | | | | 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>
* send-pull-request: Use current date in mail headersDarren Hart2010-12-211-1/+5
| | | | | | | | | | | | | 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>
* git-pull: add send-pull-request scriptDarren Hart2010-11-101-0/+133
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>