[JIRA] Assigned: (CC-1022) [PATCH] Include label in subject of build failure email

"Dan Rollo (JIRA)" <[email protected]> Sat, 9 Apr 2011 15:08:43 -0500 (CDT)
Newsgroups gmane.comp.java.cruise-control.devel
Message-ID <1896694799.1302379723649.JavaMail.root@chidmzhosting02.thoughtworks.com>
     [ http://jira.public.thoughtworks.org/browse/CC-1022?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dan Rollo reassigned CC-1022:
-----------------------------

    Assignee: Dan Rollo

> [PATCH] Include label in subject of build failure email
> -------------------------------------------------------
>
>                 Key: CC-1022
>                 URL: http://jira.public.thoughtworks.org/browse/CC-1022
>             Project: CruiseControl
>          Issue Type: Improvement
>          Components: Core Application
>    Affects Versions: 2.8.4
>            Reporter: Quinn Styl
>         Assigned To: Dan Rollo
>             Fix For: 2.8.5
>
>   Original Estimate: 1 hour
>  Remaining Estimate: 1 hour
>
> If we have the build label, why not include it in the email subject of a failed build?  In our case, we're using the p4changelistlabelincrementer, so we have a label to report.
> --- EmailPublisher.java.284	2010-09-15 19:42:04.000000000 -0700
> +++ EmailPublisher.java	2010-12-10 16:05:49.000000000 -0800
> @@ -174,12 +174,14 @@
>              subjectLine.append(subjectPrefix).append(" ");
>          }
>          subjectLine.append(logHelper.getProjectName());
> -        if (logHelper.isBuildSuccessful()) {
> +    
> +        // we want the label whether the build passed or failed
>              final String label = logHelper.getLabel();
>              if (label.trim().length() > 0) {
>                  subjectLine.append(" ").append(logHelper.getLabel());
>              }
>  
> +        if (logHelper.isBuildSuccessful()) {
>              // Anytime the build is "fixed" the subjest line
>              // should read "fixed". It might confuse recipients...but
>              // it shouldn't

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.public.thoughtworks.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Xperia(TM) PLAY
It's a major breakthrough. An authentic gaming
smartphone on the nation's most reliable network.
And it wants your games.
http://p.sf.net/sfu/verizon-sfdev