diff options
author | Shawn O. Pearce <sop@google.com> | 2009-03-03 17:47:06 -0800 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2009-03-03 17:47:06 -0800 |
commit | c95583bf4f17b8467f815b6391ffc6c7add08804 (patch) | |
tree | c69bf8a061279b79326fb00f2197efcd8726864c /subcmds/help.py | |
parent | 6a5644d392069b67f17c8ce6cb10f07cce71cc1c (diff) | |
download | git-repo-c95583bf4f17b8467f815b6391ffc6c7add08804.tar.gz |
Don't permit users to run repo status in a mirror client
If a client was created with "repo init --mirror" then there are
no working directories present, and no files checked out. Using
a command like "repo status" in this context makes no sense, and
actually throws back a Pytyon traceback at the console when the
underlying commands fail out.
We now tag commands with the MirrorSafeCommand type if they are
able to be executed within a mirror directory safely. Using a
command in a mirror which lacks this base class results in a
useful error letting you know the command isn't supported.
Bug: REPO-14
Signed-off-by: Shawn O. Pearce <sop@google.com>
Diffstat (limited to 'subcmds/help.py')
-rw-r--r-- | subcmds/help.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/help.py b/subcmds/help.py index 6e0238a0..a2814e11 100644 --- a/subcmds/help.py +++ b/subcmds/help.py | |||
@@ -17,9 +17,9 @@ import sys | |||
17 | from formatter import AbstractFormatter, DumbWriter | 17 | from formatter import AbstractFormatter, DumbWriter |
18 | 18 | ||
19 | from color import Coloring | 19 | from color import Coloring |
20 | from command import PagedCommand | 20 | from command import PagedCommand, MirrorSafeCommand |
21 | 21 | ||
22 | class Help(PagedCommand): | 22 | class Help(PagedCommand, MirrorSafeCommand): |
23 | common = False | 23 | common = False |
24 | helpSummary = "Display detailed help on a command" | 24 | helpSummary = "Display detailed help on a command" |
25 | helpUsage = """ | 25 | helpUsage = """ |