1390850 - (Enable-32bit-to-64bit-migration) Enable the 32 bit to 64 bit migration for Thunderbird.

Status: RESOLVED FIXED

(Thunderbird :: Installer, enhancement)

Description

[Richard Marti (:Paenglab)]

When everything is ready to migrate 32 bit TB to 64 bit (build infrastructure, tests, no remaining blockers etc.), we can enable by uncommenting the call to the 'Set32to64DidMigrateReg' macro in installer.nsi and shared.nsh.

Comment 1

[Maverick]

Personal experience: I just migrated from 32bit to 64bit on Windows10-1803 without any problems.

Beside the 32bit i already had, i installed the 64bit side by side with it (default options). 
I opened the 64bit after install to check if everything was there. Once i saw it was i removed the 32bit. 

Worked without any problems.

Comment 2

[Worcester12345]

(In reply to Maverick from comment #1)
> Personal experience: I just migrated from 32bit to 64bit on Windows10-1803
> without any problems.
> 
> Beside the 32bit i already had, i installed the 64bit side by side with it
> (default options). 
> I opened the 64bit after install to check if everything was there. Once i
> saw it was i removed the 32bit. 
> 
> Worked without any problems.

I saw in another bug a reference to doing this in the Fall.  Let's get this done.

Comment 3

[Wayne Mery (:wsmwk)]

not happening for beta 61, which in fact is long past several months ago.  You can check via bug 634233 that there is more work to be done, and no one is predicting when that work will happen.

Comment 4

[Worcester12345]

It looks like the only dependency is bug 393302. 

Since beta 61 is "long past", please select the cycle after that, then.

Comment 5

[Wayne Mery (:wsmwk)]

I have just updated bug 634233 suggesting that maybe this bug doesn't block bug 634233, so ...

(In reply to Richard Marti (:Paenglab) from comment #0)

... uncommenting 'Set32to64DidMigrateReg' macro in installer.nsi and shared.nsh.

Richard, what exactly does this change do?

Comment 6

[Richard Marti (:Paenglab)]

It enables the removal of 32bit registry keys, see https://searchfox.org/mozilla-central/source/browser/installer/windows/nsis/shared.nsh#559-562

We should enable this when we support 64bit officially and the user don't need to go back to 32bit for an official build.

Comment 7

[Wayne Mery (:wsmwk)]

Any difference in execution/results between user who updates a 32bit system vs a user who downloads a 64bit install?

Comment 8

[Richard Marti (:Paenglab)]

From the comment in shared.nsh where should be no difference.

But better ask an expert. Robert, Set32to64DidMigrateReg is executed on install and update. Is that correct?

Comment 9

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

(In reply to Richard Marti (:Paenglab) from comment #8)

From the comment in shared.nsh where should be no difference.

But better ask an expert. Robert, Set32to64DidMigrateReg is executed on install and update. Is that correct?

It is
https://searchfox.org/mozilla-central/source/browser/installer/windows/nsis/installer.nsi#404
https://searchfox.org/mozilla-central/source/browser/installer/windows/nsis/shared.nsh#100

BTW: I no longer work on installer code.

I'm planning on removing the one-off support for 32 to 64 bit migration in app update when I get a chance. I can hold off a few releases though it has been over a year and a half since that code was added. When is this going to happen?

Comment 10

[Richard Marti (:Paenglab)]

(In reply to Robert Strong (Robert they/them) [:rstrong] (use needinfo to contact me) from comment #9)

I'm planning on removing the one-off support for 32 to 64 bit migration in app update when I get a chance. I can hold off a few releases though it has been over a year and a half since that code was added. When is this going to happen?

If all goes well, we enable it for 68. We have the code copied in our shared.nsh on c-c. Removing it in /browser shouldn't affect us and we can remove it later in our code.

Comment 11

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

OK... I'll hold off removal in /toolkit/mozapps/update/ until 69 or 70.

Comment 12

[Jorg K (CEST = GMT+2)]

As discussed with Magnus in my weekly meeting, can we do what's required here (and ignore bug 1509918 for the time being).

Comment 13

[Richard Marti (:Paenglab)]

Attachment: 1390850-enable 32to64bit-migration.patch

This patch enables the 32bit to 64bit registration migration.

What is needed, is on server side that instead of the 32bit- the 64bit installer/update is sent to the user when his OS is 64bit capable.

Comment 14

[Jorg K (CEST = GMT+2)]

Comment on attachment 9069427 [details] [diff] [review]
1390850-enable 32to64bit-migration.patch

Thanks, not too hard then.

Comment 15

[Jorg K (CEST = GMT+2)]

(In reply to Richard Marti (:Paenglab) from comment #13)

What is needed, is on server side that instead of the 32bit- the 64bit installer/update is sent to the user when his OS is 64bit capable.

Rob, can you take care of this.

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/ae833bff1165
Enable the 32 bit to 64 bit migration for Thunderbird. r=jorgk

Comment 17

[Jorg K (CEST = GMT+2)]

TB 68 beta:
https://hg.mozilla.org/releases/comm-beta/rev/c285ff6b14fd87bda1965717d7fde4c8910f8970

Comment 18

[Rob Lemley [:rjl]]

The process Firefox used server-side is documented in bug 1393447, I'll open a new bug for us and attach it to the metabug so it can be tracked.

Comment 19

[Worcester12345]

(In reply to Rob Lemley [:rjl] from comment #18)

The process Firefox used server-side is documented in bug 1393447, I'll open a new bug for us and attach it to the metabug so it can be tracked.

Please link here also. Thank you.

Comment 20

[Worcester12345]

Wait. The title of this bug is "Enable the 32 bit to 64 bit migration for Thunderbird.", and its status is fixed.

Does this mean it will start working automatically? On which versions?

Comment 21

[Rob Lemley [:rjl]]

This just enables the registry setting to allow a future migration.

Comment 22

[Worcester12345]

Maybe the title should be changed to better reflect this.

Comment 23

[Jorg K (CEST = GMT+2)]

TB 60.8 ESR:
https://hg.mozilla.org/releases/comm-esr60/rev/d45cc02f03e83efcdd0e4eb97d50286b2a16146b

Comment 24

[Worcester12345]

It looks like things are finally moving forward with 64 bit Thunderbird.