From 498a0e8a79ab76eeb6adc40f12b04d59820716f9 Mon Sep 17 00:00:00 2001 From: "Shawn O. Pearce" Date: Mon, 18 May 2009 12:28:54 -0700 Subject: [PATCH] Make 'repo branches -a' the default behavior Extensive discussion with users lead to the fact that needing to supply -a to view what they really wanted to see was just wrong. Signed-off-by: Shawn O. Pearce --- subcmds/branches.py | 19 +------------------ 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/subcmds/branches.py b/subcmds/branches.py index 57d8c88..87c4f9b 100644 --- a/subcmds/branches.py +++ b/subcmds/branches.py @@ -63,11 +63,6 @@ class Branches(Command): Summarizes the currently available topic branches. """ - def _Options(self, p): - p.add_option('-a', '--all', - dest='all', action='store_true', - help='show all branches, not just the majority') - def Execute(self, opt, args): projects = self.GetProjects(args) out = BranchColoring(self.manifest.manifestProject.config) @@ -84,18 +79,6 @@ Summarizes the currently available topic branches. names = all.keys() names.sort() - if not opt.all and not args: - # No -a and no specific projects listed; try to filter the - # results down to only the majority of projects. - # - n = [] - for name in names: - i = all[name] - if i.IsCurrent \ - or 80 <= (100 * len(i.projects)) / project_cnt: - n.append(name) - names = n - if not names: print >>sys.stderr, ' (no branches)' return @@ -126,7 +109,7 @@ Summarizes the currently available topic branches. hdr('%c%c %-*s' % (current, published, width, name)) out.write(' |') - if in_cnt < project_cnt and (in_cnt == 1 or opt.all): + if in_cnt < project_cnt and (in_cnt == 1): fmt = out.write paths = [] if in_cnt < project_cnt - in_cnt: