Display build architecture (32-bit or 64-bit) to the About dialog

RESOLVED FIXED in Thunderbird 51.0

Status

Thunderbird
General
RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: wsmwk, Assigned: Jorg K (GMT+2))

Tracking

(Blocks: 1 bug)

unspecified
Thunderbird 51.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Adapt Firefox Bug #597738 so About dialog displays build architecture (32-bit or 64-bit)
(Assignee)

Comment 1

a year ago
Created attachment 8790433 [details] [diff] [review]
1302005.patch

Straight port of bug 597738:
https://hg.mozilla.org/mozilla-central/rev/838a2c3d46ef

Note that the equivalent of "browser" in FF is "messenger" in TB.
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
Attachment #8790433 - Flags: review?(acelists)
(Assignee)

Comment 2

a year ago
Created attachment 8790436 [details] [diff] [review]
1302005.patch

s/Firefox/Thunderbird/ in the localisation note ;-)
Attachment #8790433 - Attachment is obsolete: true
Attachment #8790433 - Flags: review?(acelists)
Attachment #8790436 - Flags: review?(acelists)

Comment 3

11 months ago
Comment on attachment 8790436 [details] [diff] [review]
1302005.patch

Review of attachment 8790436 [details] [diff] [review]:
-----------------------------------------------------------------

Works for me, thanks.

::: mail/base/content/aboutDialog.js
@@ +45,5 @@
>      let buildID = Services.appinfo.appBuildID;
> +    let year = buildID.slice(0, 4);
> +    let month = buildID.slice(4, 6);
> +    let day = buildID.slice(6, 8);
> +    versionField.textContent += ` (${year}-${month}-${day})`;

What is this syntax?

@@ +63,1 @@
>  #ifdef MOZ_UPDATER

I'd love if we also ported the change to AppConstants.MOZ_UPDATER here. In a new bug :)

::: mail/locales/en-US/chrome/messenger/messenger.properties
@@ +826,5 @@
> +# current Thunderbird build: 32-bit or 64-bit. These strings are used in parentheses
> +# after the Thunderbird version in the About dialog,
> +# e.g.: "48.0.2 (32-bit)" or "51.0a1 (2016-09-05) (64-bit)".
> +aboutDialog.architecture.sixtyFourBit = 64-bit
> +aboutDialog.architecture.thirtyTwoBit = 32-bit

They could have added this to the toolkit so we could maybe inherit it.
Attachment #8790436 - Flags: review?(acelists) → review+
(Assignee)

Comment 4

11 months ago
(In reply to :aceman from comment #3)
> Works for me, thanks.
Works for me too, on Windows. But the local build is only 32-bit which is soooo uncool.

> What is this syntax?
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Template_literals
(Assignee)

Comment 5

11 months ago
https://hg.mozilla.org/comm-central/rev/201a214e959c

Thanks, Wayne, for reporting this.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 51.0

Comment 6

11 months ago
(In reply to Jorg K (GMT+2, PTO during summer) from comment #4)
> (In reply to :aceman from comment #3)
> > Works for me, thanks.
> Works for me too, on Windows. But the local build is only 32-bit which is
> soooo uncool.

I've got the 64-bit string so it seems to work for both cases.

> > What is this syntax?
> https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/
> Template_literals

Do they need to drag everything into JS? ;)

Updated

11 months ago
Blocks: 1302830
You need to log in before you can comment on or make changes to this bug.