Date Missing from About Nightly Window

VERIFIED FIXED

Status

()

Firefox
General
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: WildcatRay, Assigned: jfkthame)

Tracking

({regression})

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

Firefox Tracking Flags

(firefox5- fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Created attachment 525709 [details]
About Nightly Window showing missing date

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.
(Assignee)

Comment 1

6 years ago
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.

Updated

6 years ago
Blocks: 648368
Keywords: regression
(Assignee)

Comment 2

6 years ago
Created attachment 525720 [details] [diff] [review]
patch - include build date in About for nightly and aurora builds

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

Updated

6 years ago
Duplicate of this bug: 649444

Updated

6 years ago
Depends on: 600569
(Assignee)

Comment 5

6 years ago
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.

Comment 7

6 years ago
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?
(Assignee)

Comment 10

6 years ago
Created attachment 525757 [details] [diff] [review]
patch, include build date for Nightly/Aurora in About - revised

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?
(Assignee)

Updated

6 years ago
Attachment #525757 - Flags: review? → review?(dao)
(Assignee)

Comment 11

6 years ago
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.

Updated

6 years ago
Attachment #525757 - Flags: review?(dao) → review+
I concur - we want it in Aurora and Nightly but not Beta or Release.

Comment 13

6 years ago
Yep, correct in your reasoning.
(Assignee)

Comment 14

6 years ago
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?
tracking-firefox5: --- → ?
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Attachment #525757 - Flags: approval-mozilla-aurora?

Updated

6 years ago
Duplicate of this bug: 649950

Comment 16

6 years ago
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).
(Reporter)

Comment 17

6 years ago
(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.

Comment 18

6 years ago
(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
(Reporter)

Comment 20

6 years ago
Thanks!

Comment 21

6 years ago
When will the patch be pushed to Aurora?

Comment 22

6 years ago
This is not something to track for aurora, it would not prevent us from cutting over to beta for ff5.
tracking-firefox5: ? → -

Comment 23

6 years ago
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+
(Assignee)

Comment 24

6 years ago
http://hg.mozilla.org/mozilla-aurora/rev/768ec7eec906
(Reporter)

Comment 25

6 years ago
Confirmed that date now shows in the About Aurora window. Thanks and good work!
(Assignee)

Updated

6 years ago
status-firefox5: --- → fixed
You need to log in before you can comment on or make changes to this bug.