[Buildbot] #2844: make c['www']['url'] mandatory

Buildbot trac trac at buildbot.net
Sat Nov 8 01:22:09 UTC 2014


#2844: make c['www']['url'] mandatory
------------------------+-----------------------
Reporter:  dustin       |       Owner:  dustin
    Type:  enhancement  |      Status:  assigned
Priority:  major        |   Milestone:  0.9.0
 Version:  0.8.9        |  Resolution:
Keywords:  simple       |
------------------------+-----------------------

Comment (by dustin):

 From that pull req:
 ---
 The Status class will be dead before 0.9.0 is released, so it doesn't
 matter.

 Yes, this is about compatibility, to help users who are migrating their
 configuration.

 There's an argument to be made that, since the c['www'] section must be
 added in 0.9.0, users should add c['www']['url'] there and we don't need a
 fallback. We could just make it mandatory -- that would save others from
 the error I made here, which is what made me file the bug that begat this
 pull request.

 So let's do that: make `c['www']['url']` mandatory, with no default at
 all.

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


More information about the bugs mailing list