Add " (CQ)" to the author name in the console if the commit-queue was used.

buildbot/status/web/base.py

 # This file is part of Buildbot.  Buildbot is free software: you can
 # redistribute it and/or modify it under the terms of the GNU General Public
 # License as published by the Free Software Foundation, version 2.
 #
 # This program is distributed in the hope that it will be useful, but WITHOUT
 # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
 # FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
 # details.
 #
 # You should have received a copy of the GNU General Public License along with
@@ skipping 480 matching lines @@
                              extensions=['jinja2.ext.i18n'],
                              trim_blocks=True,
                              undefined=AlmostStrictUndefined)
     
     env.install_null_translations() # needed until we have a proper i18n backend
     
     env.filters.update(dict(
         urlencode = urllib.quote,
         email = emailfilter,
         user = userfilter,
+        cq = cqfilter,

[M-A Ruel, 2012/01/26 18:53:47]

An option is to monkey patch it in scripts/master/chromium_status_bb8.py, e.g.

def MonkeyPatch():
  from buildbot.status.web import base
  x = base.createJinjaEnv
  def new(*args, **kwargs):
    out = x(*args, **kwargs)
    out.filters.update({'cq': cqfilter})
    return out
  base.createJinjaEnv = new

(with better names)

[sadrul, 2012/01/26 20:24:41]

Interesting idea! I have made that change.

         shortrev = shortrevfilter(revlink, env),
         revlink = revlinkfilter(revlink, env),
         changecomment = changelinkfilter(changecommentlink), 
         repolink = dictlinkfilter(repositories),
         projectlink = dictlinkfilter(projects)
         ))
     
     return env    
 
 def emailfilter(value):
@@ skipping 15 matching lines @@
         it on mouse-over or similar etc 
     '''
     r = re.compile('(.*) +<(.*)>')
     m = r.search(value)
     if m:
         user = jinja2.escape(m.group(1))
         email = emailfilter(m.group(2))        
         return jinja2.Markup('<div class="user">%s<div class="email">%s</div></div>' % (user, email))
     else:
         return emailfilter(value) # filter for emails here for safety
-        
+ 
+def cqfilter(value):
+    ''' Adds a helpful '(CQ)' to make it visibly obvious which commits were made
+        through the commit queue.
+'''
+    if value:

[sadrul, 2012/01/26 17:57:49]

I am assuming |value| here will continue to be a boolean, and not a string. Is
this assumption correct?

[M-A Ruel, 2012/01/26 18:53:47]

:/ No idea.

+        return " (CQ)"
+    return ""
+
 def _revlinkcfg(replace, templates):
     '''Helper function that returns suitable macros and functions
        for building revision links depending on replacement mechanism
 '''
    
     assert not replace or callable(replace) or isinstance(replace, dict) or \
            isinstance(replace, str) or isinstance(replace, unicode)
     
     if not replace:
         return lambda rev, repo: None
@@ skipping 214 matching lines @@
     return filter
 
 class AlmostStrictUndefined(jinja2.StrictUndefined):
     ''' An undefined that allows boolean testing but 
         fails properly on every other use.
         
         Much better than the default Undefined, but not
         fully as strict as StrictUndefined '''
     def __nonzero__(self):
         return False