Add `new` to list of commands that should not default to using the pager.
authorW. Trevor King <wking@drexel.edu>
Sat, 25 Feb 2012 17:53:17 +0000 (12:53 -0500)
committerW. Trevor King <wking@drexel.edu>
Sat, 25 Feb 2012 17:53:17 +0000 (12:53 -0500)
If the command is run in the pager, access to stdin is tricky, so I
avoid it by default.  I'd just missed the fact that `new -` used
stdin, which caused

  https://bugzilla.redhat.com/show_bug.cgi?id=751473

  $ be new -
  Traceback (most recent call last):
    ...
    File "/usr/lib/python2.7/site-packages/libbe/command/new.py", line 89, in _run
      summary = self.stdin.readline()
    ...
    File "/usr/lib64/python2.7/codecs.py", line 473, in read
      newdata = self.stream.read(size)
  IOError: [Errno 9] Bad file descriptor

libbe/ui/command_line.py

index 3a5f5c43a45b3abbc4b132269939ef8f4d436b40..d6e8d676ec178deedcbc326164d62e17dc700168 100644 (file)
@@ -355,7 +355,7 @@ def main():
     command = Class(ui=ui)
     ui.setup_command(command)
 
-    if command.name in ['comment', 'commit', 'import-xml', 'serve']:
+    if command.name in ['new', 'comment', 'commit', 'import-xml', 'serve']:
         paginate = 'never'
     else:
         paginate = 'auto'