bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.

Display both comm-central and mozilla-central changeset IDs on tinderbox pages



MailNews Core
Build Config
10 years ago
10 years ago


(Reporter: standard8, Assigned: standard8)


Dependency tree / graph

Firefox Tracking Flags

(Not tracked)



(1 attachment)



10 years ago
I think considering we have two hg repositories that we pull, and we're going to be picking up most changes from mozilla-central, we should have their changeset id on the tinderbox page as well.

From what I've seen, I think this is basically making client.py do the TinderboxPrint log line for the mozilla-central change (obviously on a switch so that we don't display it all the time).

I'm happy to try and do the change if this seems reasonable.
(In reply to comment #0)
> if this seems reasonable.

I would think it is :-)

Comment 2

10 years ago
I thought I'd like to have that right from the beginning, but I didn't see how to fetch the mozilla-central revision.
I'd be happy about a patch for that.

Comment 3

10 years ago
Created attachment 331001 [details] [diff] [review]
The fix

Isn't hg --template really nice? :-)

I struggled with the repository bit (because the way the script is set up), but as this is all we need at the moment, it should do.
Assignee: nobody → bugzilla
Attachment #331001 - Flags: review?(kairo)
FYI, 'hg identify -i' accomplishes the same thing.

Comment 5

10 years ago
Comment on attachment 331001 [details] [diff] [review]
The fix

>diff --git a/client.py b/client.py
>+        # I can't find a sane may to get the actual repo at the moment, so just
>+        # default to the main mozilla one.

This will probably bite us when we pull in DOMI from hg (i.e. as soon as it gets removed from mozilla-central). Please check that "repository == DEFAULT_MOZILLA_REPO" when doing this. r- because of that.

>+        check_call([hg, 'parent', '-R', fulldir,
>+                   "--template=TinderboxPrint: \"<a href=%sindex.cgi/rev/{node|short} title=\"Built from revision {node|short}\">mrev:{node|short}</a>\"\n" % (DEFAULT_MOZILLA_REPO)])

We should note that it's a revision from the Mozilla repo in the title as well. And I like "m-c" better than "mrev" so it's easier to tell it apart from the "rev" we get for comm-central.
Attachment #331001 - Flags: review?(kairo) → review-

Comment 7

10 years ago
I solved this slightly differently, this is in on the builder machines:

Will do the same for unit test machines tomorrow, but need to clear up if what I added to config.py for the moment should actually go into buildbotcustom.

Comment 8

10 years ago
Applied this to testers as well now and gozer applied to Thunderbird machines, marking fixed.
I still need to find out if the buildstep should go into buildbotcustom, but that will be done in a different bug, if so.
Last Resolved: 10 years ago
Resolution: --- → FIXED


10 years ago
Blocks: 447772


10 years ago
Component: Build Config → Build Config
Product: SeaMonkey → MailNews Core
QA Contact: build-config → build-config
You need to log in before you can comment on or make changes to this bug.