Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Add -p to `repo forall` to improve output formatting

When trying to read log output from many projects at once it can
be difficult to make sense of which messages came from where.

For many professional developers it is common to want to view the
last week's worth of your work, so you can write a weekly summary
of your activity for your status report.

This is easier with the new -p option:

  repo forall -pc git log --reverse --since=1.week.ago --author=sop

produces a report of all commits written by me in the last week,
formatted in a paged output display, with headers inserted in
front of each project's output.

Where this can be even more useful is with git log's pickaxe,
e.g. now we can use:

  repo forall -pc git log -Sbar v1.0..v1.1

to locate all additions or removals of the symbol 'bar' since v1.0,
up to and including v1.1.  Before displaying the matching commits in
a project, a project header is shown, giving the user some context
information for the matching results.

Signed-off-by: Shawn O. Pearce <sop@google.com>
  • Loading branch information...
commit db45da12089bf131579d100ff7990cbc18d07325 1 parent 50fa1ac
@spearce spearce authored
Showing with 151 additions and 5 deletions.
  1. +3 −0  color.py
  2. +7 −0 command.py
  3. +3 −2 main.py
  4. +138 −3 subcmds/forall.py
View
3  color.py
@@ -110,6 +110,9 @@ def is_on(self):
def write(self, fmt, *args):
self._out.write(fmt % args)
+ def flush(self):
+ self._out.flush()
+
def nl(self):
self._out.write('\n')
View
7 command.py
@@ -27,6 +27,9 @@ class Command(object):
manifest = None
_optparse = None
+ def WantPager(self, opt):
+ return False
+
@property
def OptionParser(self):
if self._optparse is None:
@@ -109,11 +112,15 @@ class InteractiveCommand(Command):
"""Command which requires user interaction on the tty and
must not run within a pager, even if the user asks to.
"""
+ def WantPager(self, opt):
+ return False
class PagedCommand(Command):
"""Command which defaults to output in a pager, as its
display tends to be larger than one screen full.
"""
+ def WantPager(self, opt):
+ return True
class MirrorSafeCommand(object):
"""Command permits itself to run within a mirror,
View
5 main.py
@@ -105,6 +105,8 @@ def _Run(self, argv):
% name
sys.exit(1)
+ copts, cargs = cmd.OptionParser.parse_args(argv)
+
if not gopts.no_pager and not isinstance(cmd, InteractiveCommand):
config = cmd.manifest.globalConfig
if gopts.pager:
@@ -112,11 +114,10 @@ def _Run(self, argv):
else:
use_pager = config.GetBoolean('pager.%s' % name)
if use_pager is None:
- use_pager = isinstance(cmd, PagedCommand)
+ use_pager = cmd.WantPager(copts)
if use_pager:
RunPager(config)
- copts, cargs = cmd.OptionParser.parse_args(argv)
try:
cmd.Execute(copts, cargs)
except ManifestInvalidRevisionError, e:
View
141 subcmds/forall.py
@@ -13,12 +13,29 @@
# See the License for the specific language governing permissions and
# limitations under the License.
+import fcntl
import re
import os
+import select
import sys
import subprocess
+
+from color import Coloring
from command import Command, MirrorSafeCommand
+_CAN_COLOR = [
+ 'branch',
+ 'diff',
+ 'grep',
+ 'log',
+]
+
+class ForallColoring(Coloring):
+ def __init__(self, config):
+ Coloring.__init__(self, config, 'forall')
+ self.project = self.printer('project', attr='bold')
+
+
class Forall(Command, MirrorSafeCommand):
common = False
helpSummary = "Run a shell command in each project"
@@ -28,6 +45,24 @@ class Forall(Command, MirrorSafeCommand):
helpDescription = """
Executes the same shell command in each project.
+Output Formatting
+-----------------
+
+The -p option causes '%prog' to bind pipes to the command's stdin,
+stdout and stderr streams, and pipe all output into a continuous
+stream that is displayed in a single pager session. Project headings
+are inserted before the output of each command is displayed. If the
+command produces no output in a project, no heading is displayed.
+
+The formatting convention used by -p is very suitable for some
+types of searching, e.g. `repo forall -p -c git log -SFoo` will
+print all commits that add or remove references to Foo.
+
+The -v option causes '%prog' to display stderr messages if a
+command produces output only on stderr. Normally the -p option
+causes command output to be suppressed until the command produces
+at least one byte of output on stdout.
+
Environment
-----------
@@ -50,8 +85,8 @@ class Forall(Command, MirrorSafeCommand):
shell positional arguments ($1, $2, .., $#) are set to any arguments
following <command>.
-stdin, stdout, stderr are inherited from the terminal and are
-not redirected.
+Unless -p is used, stdin, stdout, stderr are inherited from the
+terminal and are not redirected.
"""
def _Options(self, p):
@@ -65,6 +100,17 @@ def cmd(option, opt_str, value, parser):
action='callback',
callback=cmd)
+ g = p.add_option_group('Output')
+ g.add_option('-p',
+ dest='project_header', action='store_true',
+ help='Show project headers before output')
+ g.add_option('-v', '--verbose',
+ dest='verbose', action='store_true',
+ help='Show command error messages')
+
+ def WantPager(self, opt):
+ return opt.project_header
+
def Execute(self, opt, args):
if not opt.command:
self.Usage()
@@ -79,8 +125,31 @@ def Execute(self, opt, args):
cmd.append(cmd[0])
cmd.extend(opt.command[1:])
+ if opt.project_header \
+ and not shell \
+ and cmd[0] == 'git':
+ # If this is a direct git command that can enable colorized
+ # output and the user prefers coloring, add --color into the
+ # command line because we are going to wrap the command into
+ # a pipe and git won't know coloring should activate.
+ #
+ for cn in cmd[1:]:
+ if not cn.startswith('-'):
+ break
+ if cn in _CAN_COLOR:
+ class ColorCmd(Coloring):
+ def __init__(self, config, cmd):
+ Coloring.__init__(self, config, cmd)
+ if ColorCmd(self.manifest.manifestProject.config, cn).is_on:
+ cmd.insert(cmd.index(cn) + 1, '--color')
+
mirror = self.manifest.IsMirror
+ out = ForallColoring(self.manifest.manifestProject.config)
+ out.redirect(sys.stderr)
+
rc = 0
+ first = True
+
for project in self.GetProjects(args):
env = dict(os.environ.iteritems())
def setenv(name, val):
@@ -102,10 +171,76 @@ def setenv(name, val):
else:
cwd = project.worktree
+ if opt.project_header:
+ stdin = subprocess.PIPE
+ stdout = subprocess.PIPE
+ stderr = subprocess.PIPE
+ else:
+ stdin = None
+ stdout = None
+ stderr = None
+
p = subprocess.Popen(cmd,
cwd = cwd,
shell = shell,
- env = env)
+ env = env,
+ stdin = stdin,
+ stdout = stdout,
+ stderr = stderr)
+
+ if opt.project_header:
+ class sfd(object):
+ def __init__(self, fd, dest):
+ self.fd = fd
+ self.dest = dest
+ def fileno(self):
+ return self.fd.fileno()
+
+ empty = True
+ didout = False
+ errbuf = ''
+
+ p.stdin.close()
+ s_in = [sfd(p.stdout, sys.stdout),
+ sfd(p.stderr, sys.stderr)]
+
+ for s in s_in:
+ flags = fcntl.fcntl(s.fd, fcntl.F_GETFL)
+ fcntl.fcntl(s.fd, fcntl.F_SETFL, flags | os.O_NONBLOCK)
+
+ while s_in:
+ in_ready, out_ready, err_ready = select.select(s_in, [], [])
+ for s in in_ready:
+ buf = s.fd.read(4096)
+ if not buf:
+ s.fd.close()
+ s_in.remove(s)
+ continue
+
+ if not opt.verbose:
+ if s.fd == p.stdout:
+ didout = True
+ else:
+ errbuf += buf
+ continue
+
+ if empty:
+ if first:
+ first = False
+ else:
+ out.nl()
+ out.project('project %s/', project.relpath)
+ out.nl()
+ out.flush()
+ if errbuf:
+ sys.stderr.write(errbuf)
+ sys.stderr.flush()
+ errbuf = ''
+ empty = False
+
+ s.dest.write(buf)
+ s.dest.flush()
+
r = p.wait()
if r != 0 and r != rc:
rc = r
Please sign in to comment.
Something went wrong with that request. Please try again.