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

Issue 2883012: cros_workon: quick hack at package name checking (Closed)

Created:
10 years, 5 months ago by Mandeep Singh Baines
Modified:
9 years, 7 months ago
Reviewers:
zbehan, sosa, davidjames
CC:
chromium-os-reviews_chromium.org
Base URL:
ssh://git@chromiumos-git/crosutils.git
Visibility:
Public.

Description

cros_workon: quick hack at package name checking Change-Id: I30e97760d5b96077e38414be31ef434542021361

Patch Set 1 #

Patch Set 2 : Send again. #

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

Messages

Total messages: 3 (0 generated)
Mandeep Singh Baines
10 years, 5 months ago (2010-07-01 03:08:47 UTC) #1
Mandeep Singh Baines
10 years, 5 months ago (2010-07-01 17:08:02 UTC) #2
sosa
10 years, 5 months ago (2010-07-01 17:44:07 UTC) #3
lgtm with a cpl nits

http://codereview.chromium.org/2883012/diff/3001/4001
File cros_workon (right):

http://codereview.chromium.org/2883012/diff/3001/4001#newcode50
cros_workon:50: equery-${FLAGS_board} which $1 | \
If only equery list could do this with an option it would make scripting much
cleaner :D

http://codereview.chromium.org/2883012/diff/3001/4001#newcode54
cros_workon:54: # Canonicalize a list of names
Period at end.

http://codereview.chromium.org/2883012/diff/3001/4001#newcode57
cros_workon:57: local names
Prefer if you explicitly set names  to ""

Powered by Google App Engine
This is Rietveld 408576698