On Sept. 9, 2013, 6:57 a.m., Michal Minar wrote:
cli/completion/lmishell.bash, line 26 http://reviewboard-openlmi.rhcloud.com/r/830/diff/1/?file=4508#file4508line26
s/interactive/interact/
oh... good catch
On Sept. 9, 2013, 6:57 a.m., Michal Minar wrote:
cli/completion/lmishell.bash, line 36 http://reviewboard-openlmi.rhcloud.com/r/830/diff/1/?file=4508#file4508line36
Shouldn't *.py files complete as well?
That's a question. Not a problem to add it. What others think about it?
- Roman
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/830/#review1110 -----------------------------------------------------------
On Sept. 6, 2013, 10:31 a.m., Roman Rakus wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/830/
(Updated Sept. 6, 2013, 10:31 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
Completion: bash completion for lmi shell
It allow to complete:
- LMI Shell options (short and long)
- files with .lmi extension
- directories
Signed-off-by: Roman Rakus rrakus@redhat.com
Diffs
cli/completion/lmishell.bash PRE-CREATION
Diff: http://reviewboard-openlmi.rhcloud.com/r/830/diff/
Testing
This is my very first completion, so please test it. It should properly handle files with spaces. It allow to complete lmishell options, but only if they are before file name.
Thanks,
Roman Rakus