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

Issue 545130: ARM build: replacement postinst for netbase (Closed)

Created:
10 years, 11 months ago by adlr
Modified:
9 years ago
Reviewers:
tedbo
CC:
chromium-os-reviews_googlegroups.com
Visibility:
Public.

Description

ARM build: replacement postinst for netbase

Patch Set 1 #

Total comments: 1

Patch Set 2 : rewrite to do less work #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -1 line) Patch
A src/package_scripts/netbase.postinst View 1 1 chunk +23 lines, -0 lines 0 comments Download
M src/package_scripts/package.whitelist View 1 1 chunk +1 line, -0 lines 0 comments Download
M src/scripts/install_packages.sh View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
adlr
10 years, 11 months ago (2010-01-20 05:56:04 UTC) #1
adlr
made the fixes. please have another look
10 years, 11 months ago (2010-01-20 06:31:45 UTC) #2
tedbo
10 years, 11 months ago (2010-01-20 06:34:45 UTC) #3
LGTM

http://codereview.chromium.org/545130/diff/1/2
File src/package_scripts/netbase.postinst (right):

http://codereview.chromium.org/545130/diff/1/2#newcode15
src/package_scripts/netbase.postinst:15: if [ -f" ${ROOT}/etc/hosts" ] && \
It looks like the spacing is messed up in '[ -f" ${ROOT}...'

Powered by Google App Engine
This is Rietveld 408576698