Skip to content
This repository has been archived by the owner on Jul 14, 2023. It is now read-only.

fixup an issue: IndexError in case that outside cmd is empty #84

Merged
merged 1 commit into from
Apr 18, 2017
Merged

fixup an issue: IndexError in case that outside cmd is empty #84

merged 1 commit into from
Apr 18, 2017

Conversation

yokawasa
Copy link
Contributor

Executing empty outside command causes "IndexError: list index out of range".

az>> #  <ENTER>

Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/yoichika/.virtualenvs/py3.5/lib/python3.5/site-packages/azclishell/__main__.py", line 71, in <module>
    main(sys.argv[1:])
  File "/home/yoichika/.virtualenvs/py3.5/lib/python3.5/site-packages/azclishell/__main__.py", line 67, in main
    shell_app.run()
  File "/home/yoichika/.virtualenvs/py3.5/lib/python3.5/site-packages/azclishell/app.py", line 558, in run
    b_flag, c_flag, outside, cmd = self._special_cases(text, cmd, outside)
  File "/home/yoichika/.virtualenvs/py3.5/lib/python3.5/site-packages/azclishell/app.py", line 416, in _special_cases
    if cmd.split()[0] == 'cd':
IndexError: list index out of range

@msftclas
Copy link

@yokawasa,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@oakeyc
Copy link
Contributor

oakeyc commented Apr 18, 2017

LGTM

@oakeyc oakeyc merged commit ef8bc33 into Azure:master Apr 18, 2017
@yokawasa yokawasa deleted the branch-fixup-outsidecmd-issue branch April 19, 2017 00:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants