Calling non-existing command causes failure of shell context about configshell-fb

[agrover]

This is a change we made to give the caller a chance to do something when an exception occurs, instead of swallowing the exception internally. You can look at the changes in targetcli-fb's script:

    while not shell._exit:
        try:
            shell.run_interactive()
        except (RTSLibError, ExecutionError) as msg:
            shell.log.error(str(msg))

basically just handling the exception (if desired) and then re-invoking the shell if an explicit exit command has not been given.

[iavael]

Ok, I agree that catching this exception is necessary. But if shell initialization is heavy it may be reasonable to let user catch exception closer to it's raise, so that there would be no need to reinitialize shell context.

[agrover]

It's just re-invoking run_interactive, which is not a big deal as far as I can tell.

[iavael]

Ok, I've got it, thanks for your help.