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

Issue 601082: enter_chroot: allow depot_tools to fail to mount (Closed)

Created:
10 years, 10 months ago by adlr
Modified:
9 years ago
Reviewers:
Alexey Marinichev
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

enter_chroot: allow depot_tools to fail to mount

Patch Set 1 #

Total comments: 1

Patch Set 2 : fixes for review #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M src/scripts/enter_chroot.sh View 1 1 chunk +3 lines, -1 line 1 comment Download

Messages

Total messages: 5 (0 generated)
adlr
10 years, 10 months ago (2010-02-16 21:33:39 UTC) #1
Alexey Marinichev
http://codereview.chromium.org/601082/diff/1/2 File src/scripts/enter_chroot.sh (right): http://codereview.chromium.org/601082/diff/1/2#newcode125 src/scripts/enter_chroot.sh:125: sudo mount --bind "$DEPOT_TOOLS" "$MOUNTED_PATH" || true In this ...
10 years, 10 months ago (2010-02-16 21:37:25 UTC) #2
Alexey Marinichev
I misunderstood. I thought chromium build will fail, but I'm told it will simply download ...
10 years, 10 months ago (2010-02-16 21:46:13 UTC) #3
adlr
mind having another look based on our conversation?
10 years, 10 months ago (2010-02-16 22:00:58 UTC) #4
Alexey Marinichev
10 years, 10 months ago (2010-02-16 22:13:17 UTC) #5
http://codereview.chromium.org/601082/diff/1003/3002
File src/scripts/enter_chroot.sh (right):

http://codereview.chromium.org/601082/diff/1003/3002#newcode126
src/scripts/enter_chroot.sh:126: echo "depot_tools failed to mount. This may
impact chromium build"
Maybe we could add something like "check your NFS permissions".  LGTM otherwise.

Powered by Google App Engine
This is Rietveld 408576698