Bug 781221 - mhsh exception when running "sleep"
Summary: mhsh exception when running "sleep"
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Matahari
Classification: Retired
Component: matahari
Version: 0.6
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Zane Bitter
QA Contact: Dave Johnson
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-13 01:48 UTC by Russell Bryant
Modified: 2016-04-26 15:19 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-07-17 12:30:16 UTC


Attachments (Terms of Use)

Description Russell Bryant 2012-01-13 01:48:37 UTC
When running "sleep" without an argument, an exception is raised.  I realize that this is invalid input, but I wouldn't expect any Python exceptions to be shown in the shell.

mhsh> sleep 
% Missing arguments: 'FLOAT'

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/matahari/shell/interpreter/interpreter.py", line 81, in onecmd
    return cmd.Cmd.onecmd(self, line)
  File "/usr/lib64/python2.7/cmd.py", line 219, in onecmd
    return func(arg)
  File "/usr/lib/python2.7/site-packages/matahari/shell/interpreter/command.py", line 185, in __call__
    message="Missing arguments: '%s'" % miss)
InvalidCommandException: Missing arguments: 'FLOAT'

Comment 1 Russell Bryant 2012-01-13 01:50:00 UTC
The same goes for "list", "show", "select", "clear"

Comment 2 Zane Bitter 2012-01-13 09:26:49 UTC
This is expected behaviour as per the release announcement:
"Currently debugging is turned on, which means that all exceptions are printed out in full. This will eventually be disabled, but for now if you see one that you think is incorrect, please report it and include the stack trace."
https://fedorahosted.org/pipermail/matahari/2011-December/002270.html

However I posted a patch to the ML that makes this a selectable option from the command line instead:
https://fedorahosted.org/pipermail/matahari/2012-January/002284.html

Comment 3 Russell Bryant 2012-01-13 21:45:27 UTC
Ok, sounds good.  Feel free to close this out.  Thanks!


Note You need to log in before you can comment on or make changes to this bug.