Skip to content

Commit

Permalink
Backout debugger change from #427 that introduced stdin usage regress…
Browse files Browse the repository at this point in the history
…ion (see #449)
  • Loading branch information
stennie committed Sep 27, 2016
1 parent 3a9e2ba commit f5a91e2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
1 change: 0 additions & 1 deletion CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ Changes to mtools
* mlaunch: Allow --setParameter options (#445)
* mloginfo: fixed showing the host in `rsstate` (#410)
* mloginfo: fixed check for WT engine (#426)
* Enabled mtools utilities to be run under debugger

#### version 1.2.1

Expand Down
10 changes: 1 addition & 9 deletions mtools/util/cmdlinetool.py
Original file line number Diff line number Diff line change
Expand Up @@ -75,15 +75,7 @@ def __init__(self):
self.argparser = argparse.ArgumentParser()
self.argparser.add_argument('--version', action='version', version="mtools version %s" % __version__)
self.argparser.add_argument('--no-progressbar', action='store_true', default=False, help='disables progress bar')
""" PyCharm and other debuggers apparently run script in a non-TTY process.
For now, just ignore the fact that we are debugging
"""
if __debug__:
self.is_stdin = False
else:
self.is_stdin = not sys.stdin.isatty()


self.is_stdin = not sys.stdin.isatty()


def run(self, arguments=None, get_unknowns=False):
Expand Down

0 comments on commit f5a91e2

Please sign in to comment.