Print mercurial revision id after downloading builds

RESOLVED FIXED

Status

Testing
mozregression
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: wlach, Unassigned)

Tracking

Trunk
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=wlach][lang=python])

Attachments

(1 attachment)

According to :jrmuizel, it would be useful if we printed the revision id after downloading a build. E.g. instead of printing this:

(mozregression)wlach@eideticker:~/src/mozregression$ mozregression -b 2014-05-06 -g 2014-05-01
Running nightly for 2014-05-03`
Downloading build from: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/05/2014-05-03-03-02-04-mozilla-central/firefox-32.0a1.en-US.linux-x86_64.tar.bz2
===== Downloaded 100% =====
Installing nightly
Starting nightly
Was this nightly build good, bad, or broken? (type 'good', 'bad', 'skip', 'retry', or 'exit' and press Enter):

Print this instead:

(mozregression)wlach@eideticker:~/src/mozregression$ mozregression -b 2014-05-06 -g 2014-05-01
Running nightly for 2014-05-03
Downloading build from: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/05/2014-05-03-03-02-04-mozilla-central/firefox-32.0a1.en-US.linux-x86_64.tar.bz2
===== Downloaded 100% =====
Installing nightly
Starting nightly (revision: fef1a56f0237)
Was this nightly build good, bad, or broken? (type 'good', 'bad', 'skip', 'retry', or 'exit' and press Enter):

--

Adding this should be quite straightforward. Just modify the print statement here to print out the revision information:

https://github.com/mozilla/mozregression/blob/master/mozregression/runnightly.py#L276

using the information gathered using the "getAppInfo" method which is part of the same class. Search the source for existing usages of this method to get revision information.

Comment 1

4 years ago
Created attachment 8419129 [details] [review]
show revision when starting mozregression build
Attachment #8419129 - Flags: review?(wlachance)
Attachment #8419129 - Flags: review?(wlachance) → review+
Merged, thanks!
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.