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

Issue 2742009: Modify postinstall to avoid x86-specific dependencies. (Closed)

Created:
10 years, 6 months ago by prigge
Modified:
9 years, 7 months ago
Reviewers:
Bill Richardson
CC:
chromium-os-reviews_chromium.org, cros.approvers_codeaurora.org, chromeboy, robotboy
Base URL:
http://src.chromium.org/git/installer.git
Visibility:
Public.

Description

Modify postinstall to avoid x86-specific dependencies. ARM postinstall is broken otherwise.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Use already defined function to get root device #

Patch Set 3 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -10 lines) Patch
M chromeos-postinst View 1 2 3 chunks +14 lines, -10 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
prigge
10 years, 6 months ago (2010-06-12 00:19:12 UTC) #1
Bill Richardson
Comments. http://codereview.chromium.org/2742009/diff/1/2 File chromeos-postinst (right): http://codereview.chromium.org/2742009/diff/1/2#newcode17 chromeos-postinst:17: ROOT_DEV=echo "${INSTALL_DEV}" | sed 's/p*[0-9]*//g' Can you source ...
10 years, 6 months ago (2010-06-12 00:29:49 UTC) #2
prigge
10 years, 6 months ago (2010-06-15 21:57:17 UTC) #3
Bill Richardson
10 years, 6 months ago (2010-06-15 22:46:01 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698