Closed
Bug 1007238
Opened 12 years ago
Closed 12 years ago
Print mercurial revision id after downloading builds
Categories
(Testing :: mozregression, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Unassigned)
Details
(Whiteboard: [good first bug][mentor=wlach][lang=python])
Attachments
(1 external file)
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•12 years ago
|
||
Attachment #8419129 -
Flags: review?(wlachance)
| Reporter | ||
Updated•12 years ago
|
Attachment #8419129 -
Flags: review?(wlachance) → review+
| Reporter | ||
Comment 2•12 years ago
|
||
Merged, thanks!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•