[Buildbot-devel] Dynamically updating text displayed in build step status

Nathaniel Smith njs at pobox.com
Wed Apr 19 06:03:04 UTC 2006


On Tue, Apr 18, 2006 at 09:26:55PM -0700, Brian Warner wrote:
> The problem here is that self.describe() is returning a list of strings, but
> it isn't expecting anyone to mutate it. When you do 'text += ["warnings"]',
> you're actually changing the class-wide value, which is why you see multiple
> copies of the same text showing up later. (python expands 'l1 += l2' into
> 'l1.extend(l2)', which modifies l1 in place).

Maybe self.describe() should be returning a tuple, so this mistake
would be impossible to make?

-- Nathaniel

-- 
When the flush of a new-born sun fell first on Eden's green and gold,
Our father Adam sat under the Tree and scratched with a stick in the mould;
And the first rude sketch that the world had seen was joy to his mighty heart,
Till the Devil whispered behind the leaves, "It's pretty, but is it Art?"
  -- The Conundrum of the Workshops, Rudyard Kipling




More information about the devel mailing list