649678 - Date Missing from About Nightly Window

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[WildcatRay]

Attachment: 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.

Comment 1

[Jonathan Kew [:jfkthame]]

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.

Comment 2

[Jonathan Kew [:jfkthame]]

Attachment: 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.

Comment 3

[juan becerra [:juanb]]

This happens on Mac as well. I'll mark it affects all.

Comment 5

[Jonathan Kew [:jfkthame]]

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?

Comment 6

[Dão Gottwald [:dao]]

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

[LegNeato]

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.

Comment 8

[:Gavin Sharp [email: [e-mail]]]

I'm on the ground in Toronto! Dao or I can review it.

Comment 9

[Dão Gottwald [:dao]]

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?

Comment 10

[Jonathan Kew [:jfkthame]]

Attachment: 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#".

Comment 11

[Jonathan Kew [:jfkthame]]

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.

Comment 12

[:Gavin Sharp [email: [e-mail]]]

I concur - we want it in Aurora and Nightly but not Beta or Release.

Comment 13

[LegNeato]

Yep, correct in your reasoning.

Comment 14

[Jonathan Kew [:jfkthame]]

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?

Comment 16

[Chris B.]

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

Comment 17

[WildcatRay]

(In reply to comment #16)
> cc: [e-mail] 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

[Chris B.]

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

Comment 19

[u279076]

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:6.0a1) Gecko/20110414 Firefox/6.0a1

Verified FIXED

Comment 20

[WildcatRay]

Thanks!

Comment 21

[Marc Rufert]

When will the patch be pushed to Aurora?

Comment 22

[JP Rosevear [:jpr]]

This is not something to track for aurora, it would not prevent us from cutting over to beta for ff5.

Comment 23

[LegNeato]

Comment on attachment 525757 [details] [diff] [review]
patch, include build date for Nightly/Aurora in About - revised

Approved for mozilla-aurora

Comment 24

[Jonathan Kew [:jfkthame]]

http://hg.mozilla.org/mozilla-aurora/rev/768ec7eec906

Comment 25

[WildcatRay]

Confirmed that date now shows in the About Aurora window. Thanks and good work!