diff options
| author | Jonathan "Duke" Leto <jonathan@leto.net> | 2011-02-15 14:18:37 -0800 |
|---|---|---|
| committer | Jonathan "Duke" Leto <jonathan@leto.net> | 2011-02-15 14:18:37 -0800 |
| commit | b81fd244c274eab2f19aa5c8cda7edc133104044 (patch) | |
| tree | 7e0e13801018a0be810e68ac1a70c93b312a4d7d /lib/jitterbug/Emailer.pm | |
| parent | return empty list if project is not defined (diff) | |
| download | jitterbug-b81fd244c274eab2f19aa5c8cda7edc133104044.tar.gz | |
Only put the short commit message in email subjects
The full commit message will still be in the body of the email.
Diffstat (limited to 'lib/jitterbug/Emailer.pm')
| -rw-r--r-- | lib/jitterbug/Emailer.pm | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/jitterbug/Emailer.pm b/lib/jitterbug/Emailer.pm index d93ec79..0d4d7da 100644 --- a/lib/jitterbug/Emailer.pm +++ b/lib/jitterbug/Emailer.pm @@ -58,6 +58,8 @@ sub run { $body =~ s/%%SHA1%%/$sha1/g; $body =~ s/%%SUMMARY%%/$summary/g; + my ($short_message) = split /\n/, $message; + my $stuff = Email::Stuff->from($buildconf->{"on_${status}_from_email"}) # bug in Email::Stuff brakes chaining if $email is empty @@ -65,7 +67,7 @@ sub run { ->cc($buildconf->{"on_${status}_cc_email"}) ->text_body($body) ->subject( - $buildconf->{"on_${status}_subject_prefix"} . "$project @ $shortsha1 $message" + $buildconf->{"on_${status}_subject_prefix"} . "$project @ $shortsha1 $short_message" ); # Should we attach a build log for convenience? # ->attach(io('dead_bunbun_faked.gif')->all, |