718234 - tbpl should use 'log_url' property if it exists

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Chris AtLee [:catlee]]

As part of bug 713846 I'm going to be adding a 'log_url' property to builds to indicate where they can be downloaded from.

TBPL should use this property if it exists.

Comment 1

[Phil Ringnalda (:philor)]

Which is just a matter of returning it if self._props["log_url"] is set, in http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/file/dde5be0a9d65/dataimport/import-buildbot-data.py#l106, but I'm afraid you're going to find that neither of the two semi-active tbpl developers actually speak Python, and you'll wind up having to patch it yourself.

Comment 2

[Arpad Borsos [:Swatinem]]

Attachment: patch

Well I’m learning Python right now, the problem is, as you said, I’m just semi-active.

Comment 3

[Chris AtLee [:catlee]]

Comment on attachment 591459 [details] [diff] [review]
patch

lgtm

Comment 4

[Arpad Borsos [:Swatinem]]

Pushed: http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/61b49334bc31

Comment 5

[Phil Ringnalda (:philor)]

We'll see how it goes when there actually is such a beast, but so far, so fixed.