Currently this project is down ! I will rewrite this with go in near future

watchall

execute a program periodically, showing output fullscreen (watch) with scroll feature.

watchall's People

Contributors

Stargazers

Watchers

watchall's Issues

Crash when 200 or more lines

Exception in thread Thread-1:
  Traceback (most recent call last):
  File "/usr/lib64/python3.8/threading.py", line 932, in _bootstrap_inner  self.run()
                                                                                                                                                        File "/usr/lib64/python3.8/threading.py", line 870, in run
                 self._target(*self._args, **self._kwargs)
 File "/home/xk/.local/bin/watchall", line 85, in update_lines  self.refresh_screen(update_pad=True)
                                                                                                                                                                   File "/home/xk/.local/bin/watchall", line 120, in refresh_screen
       self.pad.addch(self.cur_output_lines[i][j])     _curses.error: add_wch() returned ERR

Flags in command

Hi there, it appears the package doesn't currently allow hyphens in command arguments.

For example:

python watchall.py squeue -u thejohnhoffer

prints a complaint from argparse that -u is not recognized.

I would expect this script to only use argparse on flags that precede the command, as it should pass the remaining arguments unaltered to the command. I believe I've found a solution that's the same in python 2 and 3:

Replace

parser.add_argument('Args', metavar='Arguments', type=str, nargs='*', help='Command arguments')

with

parser.add_argument('Args', metavar='Arguments', type=str, nargs=argparse.REMAINDER, help='Command arguments')

This is a stronger nargs='*' that also accepts flags untouched. I'm testing this now for my own application. Many thanks for writing a great package!