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

Issue 6693001: Ignore git config errors that may occur in enter_chroot. (Closed)

Created:
9 years, 9 months ago by sosa
Modified:
9 years, 6 months ago
Reviewers:
petkov
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Ignore git config errors that may occur in enter_chroot. git config doesn't behave well if there are multiple callers calling it at the same time. Since this is best effort anyway, I'm adding an || true so that multithreads calls to this don't fail. I removed my previous work around to this since it wasn't generic enough. Change-Id: I12f2d3faaa745c1ff675a297bb09c567a88aa185 BUG=chromium-os:13070 TEST=Ran it a lot Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=9d30ce8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -10 lines) Patch
M bin/au_test_harness/cros_au_test_harness.py View 1 chunk +0 lines, -1 line 0 comments Download
M enter_chroot.sh View 2 chunks +7 lines, -9 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
sosa
9 years, 9 months ago (2011-03-14 21:31:19 UTC) #1
petkov
9 years, 9 months ago (2011-03-14 21:57:10 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698