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

Issue 3434008: cros_workon: fix bug where cros_workon start doesn't work for first start (Closed)

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

Description

cros_workon: fix bug where cros_workon start doesn't work for first start If the checkout dir doesn't already exist cros_workon start will print the wrong dir. Fixed by using readlink -m instead. BUG=none TEST=Verified that first start is incorrect without this patch but is fixed with this patch. Change-Id: I5db29a8c1737dea1dbe4866e18576f67b9355f84

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M cros_workon View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mandeep Singh Baines
10 years, 3 months ago (2010-09-16 21:25:01 UTC) #1
zbehan
10 years, 3 months ago (2010-09-16 21:28:01 UTC) #2
On 2010/09/16 21:25:01, Mandeep Singh Baines wrote:
> 

LGTM

Powered by Google App Engine
This is Rietveld 408576698