diff options
author | Jacob Helwig <jacob@puppetlabs.com> | 2011-03-28 11:08:29 -0700 |
---|---|---|
committer | Jacob Helwig <jacob@puppetlabs.com> | 2011-03-28 11:08:29 -0700 |
commit | 1063753df8938d4ea350199a4dcedd6c08c2d092 (patch) | |
tree | f88f54b22c603ed6fd602f802e72fd81dd429285 /tasks/rake/mail_patches.rake | |
parent | 623c2a67d45a47e0cf20b589a03adab5f867f182 (diff) | |
download | facter-1063753df8938d4ea350199a4dcedd6c08c2d092.tar.gz facter-1063753df8938d4ea350199a4dcedd6c08c2d092.tar.xz facter-1063753df8938d4ea350199a4dcedd6c08c2d092.zip |
(#6862) Add a default subject for the mail_patches rake task
This will prevent committers to forget to write cover letter subject
when rake mail_patches needs to send multiple e-mails.
It defaults to the "type" and "branch name" of the patch-set.
This is taken from Brice's patch against Puppet
(37f9ca09135330ed180fb68d9295a4967a5cc857).
Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
Diffstat (limited to 'tasks/rake/mail_patches.rake')
-rw-r--r-- | tasks/rake/mail_patches.rake | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tasks/rake/mail_patches.rake b/tasks/rake/mail_patches.rake index be8dda1..1346faa 100644 --- a/tasks/rake/mail_patches.rake +++ b/tasks/rake/mail_patches.rake @@ -34,12 +34,14 @@ task :mail_patches do # If we've got more than one patch, add --compose if files.length > 1 compose = "--compose" + subject = %Q{--subject "#{type} #{name} against #{parent}"} else compose = "" + subject = "" end # Now send the mail. - sh "git send-email #{compose} --no-signed-off-by-cc --suppress-from --to puppet-dev@googlegroups.com 00*.patch" + sh "git send-email #{compose} #{subject} --no-signed-off-by-cc --suppress-from --to puppet-dev@googlegroups.com 00*.patch" # Finally, clean up the patches sh "rm 00*.patch" |