Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(406)

Issue 124813002: Make the update installer fail when attempting to install a 64-bit update on a 32-bit-only system (Closed)

Created:
6 years, 11 months ago by Mark Mentovai
Modified:
6 years, 11 months ago
Reviewers:
Nico
CC:
chromium-reviews, grt+watch_chromium.org
Visibility:
Public.

Description

Make the update installer fail when attempting to install a 64-bit-only update on a 32-bit-only system. The server infrastructure should not deliver 64-bit-only updates to 32-bit-only systems, but this depends on the new -32bit tag suffix support already being on the system prior to the attempted delivery of a 64-bit-only update. The update installer will now sanity-check this, and if the system will not be capable of running the updated version, it will exit before attempting to install it. Additionally, it will set the -32bit tag suffix so that during a subsequent update check, the server should deliver the proper 32-bit-only update (likely the final 32-bit version), or no update at all. This also serves as a failsafe against breaking users on 32-bit-only systems if the server-side -32bit tag suffix detection isn't working properly. BUG=18323, 303280 R=thakis@chromium.org, thakis Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243088

Patch Set 1 : #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+118 lines, -5 lines) Patch
M chrome/installer/mac/keystone_install.sh View 6 chunks +118 lines, -5 lines 1 comment Download

Messages

Total messages: 5 (0 generated)
Mark Mentovai
This addresses your use case of “I didn’t open that old laptop in 3 months.”
6 years, 11 months ago (2014-01-06 01:44:54 UTC) #1
Nico
lgtm Once this is in, you will want to work with vclarke to get this ...
6 years, 11 months ago (2014-01-06 05:32:13 UTC) #2
Mark Mentovai
Nothing in Chrome checks these codes. They only show up in diagnostic logs. The 12 ...
6 years, 11 months ago (2014-01-06 14:34:58 UTC) #3
Mark Mentovai
I’ve already talked with Victoria. She doesn’t have access to any 32-bit-only Macs. I do, ...
6 years, 11 months ago (2014-01-06 14:35:35 UTC) #4
Mark Mentovai
6 years, 11 months ago (2014-01-06 14:37:02 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 manually as r243088 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698