extras-buildsys/www job.psp,1.3,1.4

[Daniel Williams (dcbw)]

Author: dcbw

Update of /cvs/fedora/extras-buildsys/www
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv16204/www

Modified Files:
	job.psp 
Log Message:
2005-07-29  Dan Williams <dcbw at redhat.com>

    * www/job.psp
        - Line up arch jobs
        - Only show build times for successful jobs




Index: job.psp
===================================================================
RCS file: /cvs/fedora/extras-buildsys/www/job.psp,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- job.psp	27 Jul 2005 07:25:35 -0000	1.3
+++ job.psp	29 Jul 2005 05:27:16 -0000	1.4
@@ -88,29 +88,23 @@
             <td colspan="2" align="left">
                 <table class="archjobs" cellspacing="0" cellpadding="0">
                     <tr>
-                        <td width="20" class="archjobs"></td>
-                        <td class="archjobs">
+                        <td width="3%" class="archjobs"></td>
+                        <td width="37%" class="archjobs">
                             <font class="archjobtag"><%=aj['arch']%>:</font>
                             <font class="archjob"> <%=aj['builder_addr']%></font>
                         </td>
-<%
-            colspan = ''
-            if aj['endtime'] == 0:
-                colspan = 'colspan="2"'
-            #endif
-%>
-                        <td class="archjobs" <%=colspan%>>
+                        <td width="30%" class="archjobs">
                             <font class="archjobtag">Status:</font>
                             <font class="archjob"> <%=aj['status']%>/<%=aj['builder_status']%></font>
                         </td>
+                        <td width="30%" class="archjobs">
 <%
-            if aj['endtime'] != 0:
+            # Only show build times for successful jobs
+            if aj['endtime'] != 0 and aj['builder_status'] == 'done':
                 aj_time = (int(aj['endtime']) - int(aj['starttime'])) / 60
 %>
-                        <td class="archjobs">
                             <font class="archjobtag">Build Time:</font>
                             <font class="archjob"> <%=aj_time%> minutes</font>
-                        </td>
 <%
             # endif
 %>