From 588142dfcb464a0db9ebc7b323ab2338d8f730de Mon Sep 17 00:00:00 2001 From: Etan Cohen Date: Wed, 9 Jul 2014 21:33:31 -0700 Subject: [PATCH] Provide detail print-out when not all projects of a branch are current. When current is "split" (i.e. some projects are current while others are not): - Disable 'not in' printout (i.e. will print out all projects) - Disable printing of multiple projects on one line - Print current projects in green, non-current in white Since using color to differentiate current from non-current in "split" cases: - In non-split cases also print out project names in color (green for current white for non-current) Change-Id: Ia6b826612c708447cecfe5954dc767f7b2ea2ea7 --- subcmds/branches.py | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/subcmds/branches.py b/subcmds/branches.py index f714c1e8..2902684a 100644 --- a/subcmds/branches.py +++ b/subcmds/branches.py @@ -46,6 +46,10 @@ class BranchInfo(object): def IsCurrent(self): return self.current > 0 + @property + def IsSplitCurrent(self): + return self.current != 0 and self.current != len(self.projects) + @property def IsPublished(self): return self.published > 0 @@ -139,10 +143,14 @@ is shown, then the branch appears in all projects. if in_cnt < project_cnt: fmt = out.write paths = [] - if in_cnt < project_cnt - in_cnt: + non_cur_paths = [] + if i.IsSplitCurrent or (in_cnt < project_cnt - in_cnt): in_type = 'in' for b in i.projects: - paths.append(b.project.relpath) + if not i.IsSplitCurrent or b.current: + paths.append(b.project.relpath) + else: + non_cur_paths.append(b.project.relpath) else: fmt = out.notinproject in_type = 'not in' @@ -154,13 +162,19 @@ is shown, then the branch appears in all projects. paths.append(p.relpath) s = ' %s %s' % (in_type, ', '.join(paths)) - if width + 7 + len(s) < 80: + if not i.IsSplitCurrent and (width + 7 + len(s) < 80): + fmt = out.current if i.IsCurrent else fmt fmt(s) else: fmt(' %s:' % in_type) + fmt = out.current if i.IsCurrent else out.write for p in paths: out.nl() fmt(width*' ' + ' %s' % p) + fmt = out.write + for p in non_cur_paths: + out.nl() + fmt(width*' ' + ' %s' % p) else: out.write(' in all projects') out.nl()