[Buildbot-commits] [Buildbot] #180: buildbot try does the wrong thing with mercurial queues

Buildbot nobody at buildbot.net
Tue Nov 8 12:15:23 UTC 2011


#180: buildbot try does the wrong thing with mercurial queues
------------------------+------------------------
Reporter:  dsallings    |       Owner:  dsallings
    Type:  enhancement  |      Status:  new
Priority:  major        |   Milestone:  0.8.+
 Version:  0.7.6        |  Resolution:
Keywords:  hg           |
------------------------+------------------------

Comment (by marcusl):

 The commit was made to only disable the testTryMq() test, not the entire
 Mercurial test case, when mq is not enabled. (Since I and many others
 didn't use MQ, Mercurial tests weren't run at all.)

 That was the intention at least. Looking at it now, it seems as if
 MercurialHelper._qnew_supported might be set (or not), and the commit
 removed some code which would update that value again.

 The right thing would be to make sure that _qnew_supported has a correct
 value always, and not to rely on the unit tests to check and set it.
 (Could be a property that's set lazily inside MercurialHelper, on first
 access....)

-- 
Ticket URL: <http://trac.buildbot.net/ticket/180#comment:31>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the Commits mailing list