1507618 - Mozilla Thunderbird version 60.3.1 (Default) registry not incremented, remains at 60.3.0

Status: RESOLVED FIXED

(Thunderbird :: Installer, defect)

Description

[richard.dong]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36

Steps to reproduce:

1. Install Mozilla Thunderbird 60.3.1
2. Examine the Data value in registry key:
HKEY_LOCAL_MACHINE\SOFTWARE\Mozilla\Mozilla Thunderbird\(Default)
OR:
HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Mozilla\Mozilla Thunderbird\(Default)
3. The (Default) value displays 60.3.0



Actual results:

Installing Mozilla Thunderbird 60.3.1 does not display the correct value for the (Default) key. It is incorrectly displaying as 60.3.0.


Expected results:

Installing Mozilla Thunderbird 60.3.1 should display the correct value for the (Default) key. It should correctly display as 60.3.1.

Comment 1

[Wayne Mery (:wsmwk)]

installer, or build

Comment 2

[Jorg K (CEST = GMT+2)]

My registry says 52.9.1 although I have 60.3.1 installed.

Comment 3

[Florent]

Same problem with the value of "Milestone" on the "platform.ini" file

Comment 4

[Jorg K (CEST = GMT+2)]

Richard, you're our installer expert, can you help? Or get help from the M-C people.

Comment 5

[Richard Marti (:Paenglab)]

Expert is way too much, I ported some patches. I think this is a build issue. But let's ask Robert. I hope he can point us to the problem.

Comment 6

[Robert Strong (they/them - no direct email)]

The code that sets that value was ported over from the original Firefox installer code for Firefox 2.0 and the default value was the GRE version which in this case appears to be 63.3.0.

https://dxr.mozilla.org/comm-central/source/comm/mail/installer/windows/nsis/shared.nsh#596

The define is picked up here for the installer

https://dxr.mozilla.org/comm-central/source/comm/mail/installer/windows/nsis/defines.nsi.in#11

Comment 7

[Jorg K (CEST = GMT+2)]

Well, yes, TB 60.3.1 is based on mozilla-esr60 at 60.3.0. The .1 was our own invention to ship some fixes to mail bugs.

Comment 8

[richard.dong]

This was resolved for versions 60.4.0 through 60.5.1, however I'm seeing the issue arise again on the latest 60.5.2.

Installing 60.5.2 will have 60.5.1 appear as the Data value for the (Default) key under:

HKEY_LOCAL_MACHINE\SOFTWARE\Mozilla\Mozilla Thunderbird(Default)
OR:
HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Mozilla\Mozilla Thunderbird(Default)

Comment 9

[Jorg K (CEST = GMT+2)]

Yes, sorry, the release manager here. We shipped TB 60.5.2 base on Mozilla 60.5.1 since I didn't know that they were doing a 60.5.2 as well :-(

Will be OK again for TB 60.6.

Rob, can we tweak the build system so it uses the TB version and not the Mozilla version?

Comment 10

[Rob Lemley [:rjl]]

(In reply to Jorg K (GMT+1) from comment #9)

Yes, sorry, the release manager here. We shipped TB 60.5.2 base on Mozilla 60.5.1 since I didn't know that they were doing a 60.5.2 as well :-(

Will be OK again for TB 60.6.

Rob, can we tweak the build system so it uses the TB version and not the Mozilla version?

I'll take a look and see what I can do.

Speaking to Florent in comment 3, information platform.ini should all refer to Firefox/Gecko versions and Mercurial changesets. It is entirely possible that the Milestone field in platform.ini does not match the Thunderbird version. Currently, bug 1507754 is open and documents that the SourceRepository field in platform.ini incorrectly refers to a comm- repository rather than a mozilla- one.

In contrast, most of the information in application.ini refers to Thunderbird. The exception is the [Gecko] section, which will refer to the Gecko platform version on which a particular build is based.

Comment 11

[Rob Lemley [:rjl]]

Attachment: regedit.png

Comment 12

[Rob Lemley [:rjl]]

With respect to the version numbers found in the Windows Registry, it is working as designed.

This is taken from a registry dump after a clean install of 60.5.2. The same information is in the screenshot in attachment 9047254 [details].

[Software\\Wow6432Node\\Mozilla\\Mozilla Thunderbird]
@="60.5.1"
"CurrentVersion"="60.5.2 (en-US)"`

This is the NSIS installer code that writes the above.

StrCpy $0 "Software\Mozilla\${BrandFullNameInternal}"
  ${WriteRegStr2} $TmpVal "$0" "" "${GREVersion}" 0
  ${WriteRegStr2} $TmpVal "$0" "CurrentVersion" "${AppVersion} (${AB_CD})" 0

The @ key (Default) is set to the Gecko Runtime version, ${GREVersion}. In this case it's 60.5.1.
Then the CurrentVersion key is set to the application version, ${AppVersion}, or 60.5.2 (en-US) in this case.

So, it's correct in that the NSIS code very clearly is using ${GREVersion} as the value for the key in question. It's not a case of the Gecko version accidentally getting used instead of the Thunderbird version, which was my original suspicion.

The keys could probably be named more appropriately, and if you're trying to ascertain the currently installed Thunderbird version by interrogating the registry, there isn't a nice simple version number that you can get. The closest is the CurrentVersion key, but it has a l10n tag appended to it which means parsing out the version number somehow.

Without knowing the use case, I would probably change the Default key to match the Thunderbird version, then add a GeckoVer key for the Gecko version. I'll defer to Wayne to make that decision.

Comment 13

[Wayne Mery (:wsmwk)]

Sorry I don't really have experience here, so I'll defer to Jorg and others.

Comment 14

[Jorg K (CEST = GMT+2)]

Yes, Rob, we can certainly change/fix the installer. Richard Marti has been working on installers. He or I would be happy to try out a try build if you don't have access to a Windows machine.

Comment 15

[Rob Lemley [:rjl]]

Attachment: installer_regkeys.patch

This seems to do the right thing when run under Wine (also tested on Windows 10 x64).

I did not test a signed installer (Bug 1458385 may be causing issues here), and I did not test running the updater.

I suspect the updater will not be a problem since this is shared code.

Comment 16

[Richard Marti (:Paenglab)]

Comment on attachment 9058037 [details] [diff] [review]
installer_regkeys.patch

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

Looks logic to me. thanks.

::: mail/installer/windows/nsis/shared.nsh
@@ +601,3 @@
>    ${WriteRegStr2} $TmpVal "$0" "CurrentVersion" "${AppVersion} (${AB_CD})" 0
> +  ${WriteRegStr2} $TmpVal "$0" "GeckoVersion" "${GREVersion}" 0
> +

The newline isn't needed. All !macroend, except one, follow directly without a empty line. This is only cosmetics, if you want let it in, then okay.

Comment 17

[Rob Lemley [:rjl]]

Attachment: installer_regkeys_v2.patch

Removed extra newline

Comment 18

[Richard Marti (:Paenglab)]

Comment on attachment 9058358 [details] [diff] [review]
installer_regkeys_v2.patch

Thanks.

Comment 19

[Jorg K (CEST = GMT+2)]

I'll land that soon.

Comment 20

[Jorg K (CEST = GMT+2)]

Comment on attachment 9058358 [details] [diff] [review]
installer_regkeys_v2.patch

I guess you want this backported :-)

Comment 21

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/205c80a11ab9
Use TB version as default version in Win registry. r=Paenglab

Comment 22

[Jorg K (CEST = GMT+2)]

TB 67 beta 2:
https://hg.mozilla.org/releases/comm-beta/rev/daee3654255f6c952510ab43e043d30f9e15eb7c

Comment 23

[Jorg K (CEST = GMT+2)]

TB 60.7 ESR (or earlier, like TB 60.6.2):
https://hg.mozilla.org/releases/comm-esr60/rev/7787632227e0da0b59c1ee1a946ce1d048b9199f