Date Missing from About Nightly Window

VERIFIED FIXED

Status

()

defect
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: olrrm2020, Assigned: jfkthame)

Tracking

({regression})

Trunk
x86
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox5- fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

In the About Minefield window, the date of the (nightly) build appeared following the version (4.2a1pre most recently). Starting with the new Nightly, in the About Nightly (Help > About Nightly), the date no longer appears like it did in Minefield.

On a download and clean install of Nightly (C-set: 6bcaec19d09e), I tried with a brand new profile and the date is still missing.
Looks like the cause is here (http://mxr.mozilla.org/mozilla-central/source/browser/base/content/aboutDialog.js#68):

68   // Include the build ID if this is a "pre" (i.e. non-release) build
69   let version = Services.appinfo.version;
70   if (version.indexOf("pre") != -1) {
71     let buildID = Services.appinfo.appBuildID;
72     let buildDate = buildID.slice(0,4) + "-" + buildID.slice(4,6) + "-" + buildID.slice(6,8);
73     document.getElementById("version").value += " (" + buildDate + ")";
74   }

Note that this depends on the presence of a literal "pre" in Services.appinfo.version.
Blocks: 648368
Keywords: regression
This is designed to include the date in the About window for builds with version numbers that end in "a#" (for any number #), which should include both Nightly and Aurora builds, IIRC.
This happens on Mac as well. I'll mark it affects all.
OS: Windows 7 → All
Duplicate of this bug: 649444
Depends on: 600569
Comment on attachment 525720 [details] [diff] [review]
patch - include build date in About for nightly and aurora builds

Seems to work as intended in my local build; are we happy to do this and bring back the date?
Attachment #525720 - Attachment description: patch (untested) - include build date in About for nightly and aurora builds → patch - include build date in About for nightly and aurora builds
Attachment #525720 - Flags: review?(clegnitto)
Comment on attachment 525720 [details] [diff] [review]
patch - include build date in About for nightly and aurora builds

Please use regex.test(string) instead of string.match(regex) != null.
Yep, and I am not the correct person to review this. Dao, are you? If not I'll ping Gavin but he's in the air.
I'm on the ground in Toronto! Dao or I can review it.
I can review this, but I'm not sure what the outcome really should be. Is it correct to have the date in aurora but not in beta?
Updated to use regex.test.

I'm assuming we don't want the build date in final release builds (at least, we didn't have it previously). Therefore, we can't have it in betas either, as in principle the (last) beta can _become_ the release, with the exact same bits. At least, that's how I understood things. So that's why I went for versions that end with "a#".
Assignee: nobody → jfkthame
Attachment #525720 - Attachment is obsolete: true
Attachment #525720 - Flags: review?(clegnitto)
Attachment #525757 - Flags: review?
Attachment #525757 - Flags: review? → review?(dao)
And as for Aurora - it seemed like a useful piece of info to show, just as it is for Nightly. I see no downside to including it there, as the version info is in any case not identical to final.
Attachment #525757 - Flags: review?(dao) → review+
I concur - we want it in Aurora and Nightly but not Beta or Release.
Yep, correct in your reasoning.
Pushed to mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/222fc193d280

Setting tracking-firefox5? as I think we want this on Aurora too, right?
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Attachment #525757 - Flags: approval-mozilla-aurora?
Duplicate of this bug: 649950
I just downloaded the 4/14/11 Nightly and still don't see the date in the About Dialog.  Maybe I don't have the latest build.  The time stamp from the download is 6:12 AM (Pacific).  My Build ID = http://hg.mozilla.org/mozilla-central/rev/2a343ee6acd3, which doesn't match comment 14 above.  Will there be another Nightly update later?  Sometimes Nightly updates itself twice in a day (once in the morning and again later in the afternoon).
(In reply to comment #16)
> cc: cab26715@live.comI just downloaded the 4/14/11 Nightly and still don't see the date in the About
> Dialog.  Maybe I don't have the latest build.  The time stamp from the download
> is 6:12 AM (Pacific).  My Build ID =
> http://hg.mozilla.org/mozilla-central/rev/2a343ee6acd3, which doesn't match
> comment 14 above.  Will there be another Nightly update later?  Sometimes
> Nightly updates itself twice in a day (once in the morning and again later in
> the afternoon).

That build--an hourly build--is from before this patch landed. The full nightly Nightly build for today (2011-04-14) should be out soon.
(In reply to comment #16)
> I just downloaded the 4/14/11 Nightly and still don't see the date in the About
> Dialog.  Maybe I don't have the latest build.  The time stamp from the download
> is 6:12 AM (Pacific).  My Build ID =
> http://hg.mozilla.org/mozilla-central/rev/2a343ee6acd3, which doesn't match
> comment 14 above.  Will there be another Nightly update later?  Sometimes
> Nightly updates itself twice in a day (once in the morning and again later in
> the afternoon).

Disregard my previous post.  Nightly just updated itself again less than 15 minutes after my original update.  I now have (http://hg.mozilla.org/mozilla-central/rev/aa200a803e07) and the date stamp is now in the About Dialog.
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:6.0a1) Gecko/20110414 Firefox/6.0a1

Verified FIXED
Status: RESOLVED → VERIFIED
Thanks!
When will the patch be pushed to Aurora?
This is not something to track for aurora, it would not prevent us from cutting over to beta for ff5.
Comment on attachment 525757 [details] [diff] [review]
patch, include build date for Nightly/Aurora in About - revised

Approved for mozilla-aurora
Attachment #525757 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Confirmed that date now shows in the About Aurora window. Thanks and good work!
You need to log in before you can comment on or make changes to this bug.