Closed Bug 631448 Opened 13 years ago Closed 13 years ago

Try TBPL should link to build directory

Categories

(Tree Management Graveyard :: TBPL, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jruderman, Assigned: mounir)

References

()

Details

Attachments

(1 file)

It would be nice if I get from TBPL to downloadable builds easily.

For Try, the URLs can be constructed from the pusher and rev:
https://ftp.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/jdaggett@mozilla.com-920fe6f57d17/

For non-Try, it might be harder, because the URLs are like:
https://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-central-macosx-debug/1296793077/
Attached patch Patch v1Splinter Review
This is adding a link to the build directory just before short/long log links.
The build is only shown if you click on a build. Not that we can't show it every time but I assume you might not be interested by it if you click on a test job.

This patch handles thunderbird try server too ;)
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #512158 - Flags: review?(arpad.borsos)
Summary: Try TBPL should link to downloadable builds → Try TBPL should link to build directory
Attachment #512158 - Flags: review?(mstange)
Comment on attachment 512158 [details] [diff] [review]
Patch v1

I hope you have checked if there is enough space for the link? (I didn’t)

>+            var dir = "firefox";
>+            if (repo == 'try-comm-central') {
>+              dir = "thunderbird";
>+            }

I like the ? : syntax more, but its fine either way.

>+            return '<a href="https://ftp.mozilla.org/pub/mozilla.org/' +
>+                   dir + '/tryserver-builds/' + result.push.pusher + '-' +
>+                   result.revs[repo] + '/">go to build directory</a>';

Never knew this was available using https, guess I learn new things all the time :-)
Attachment #512158 - Flags: review?(mstange)
Attachment #512158 - Flags: review?(arpad.borsos)
Attachment #512158 - Flags: review+
Pushed:
https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/574b51478ded
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Depends on: 639921
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: