summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* version 1.1.fb24v1.1.fb24Andy Grover2018-01-261-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #40 from gonzoleeman/masterAndy Grover2018-01-261-1/+1
|\ | | | | Remove explicit dependency on /usr/bin/python
| * Remove explicit dependency on /usr/bin/pythonLee Duncan2017-12-081-1/+1
|/ | | | | | | | | The example script specifies /usr/bin/python, but the code can all handle python2 or python2, so update the script to work with either. Note: this replaces a SPEC-file dependency on /usr/bin/python with one on /usr/bin/env.
* Merge pull request #38 from ryran/run-cmdline-new-verbose-loglevelAndy Grover2017-09-292-3/+12
|\ | | | | make run_cmdline() log "Running command line" msg to new loglevel
| * make run_cmdline() log "Running command line" msg to new loglevelRyan Sawhill Aroha2017-09-262-3/+12
| |
* | Merge pull request #37 from ryran/run-stdin-strip-whitespaceAndy Grover2017-09-291-1/+1
|\ \ | | | | | | tweak ConfigShell.run_stdin() to strip whitespace/newline from cmds
| * | tweak ConfigShell.run_stdin() to strip whitespace/newline from cmdsRyan Sawhill Aroha2017-09-261-1/+1
| |/
* | Merge pull request #36 from lxbsz/fix_for_tcmuAndy Grover2017-09-291-1/+1
|\ \ | |/ |/| Fix failing to pasre par=val parameters
| * Fix failing to pasre par=val parametersXiubo Li2017-09-061-1/+1
|/ | | | | | | | Through targetcli-fb to create LIO ceph target device like: /backstores/user:rbd> create block0 10G pool/rbd1/osd_op_timeout=30 the =30 will be ignored. Signed-off-by: Xiubo Li <lixiubo@cmss.chinamobile.com>
* Merge pull request #35 from cvubrugier/fix-install-requiresAndy Grover2017-08-201-0/+4
|\ | | | | Add missing dependency on pyparsing and six
| * Add missing dependency on pyparsing and sixChristophe Vu-Brugier2017-08-201-0/+4
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Fix path regex for [ and ]Andy Grover2017-05-301-1/+1
| | | | | | | | When dealing with ipv6, we now have paths consisting of square brackets. We need to add these characters to the path regex for them to match properly. Signed-off-by: Andy Grover <agrover@redhat.com>
* Handle if TERM is not setAndy Grover2017-05-231-2/+3
| | | | | | Previous patch causes problems otherwise in some cases. Signed-off-by: Andy Grover <agrover@redhat.com>
* version 1.1.fb23v1.1.fb23Andy Grover2017-03-011-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #30 from czhang512/masterAndy Grover2017-01-051-0/+10
|\ | | | | Fix the readline not defined error
| * Fix the readline not defined bugChongshi Zhang2016-12-131-0/+10
| | | | | | | | | | | | | | | | | | | | TargetCLI will throw "readline not defined" error when running it as process in interactive shell mode. The cause of the problem is the readline module would not be imported in this case. Set the TERM to empty is to fix readline emits unwanted characters problem when import it. Signed-off-by: Chongshi Zhang <zhangcho@us.ibm.com>
* | version 1.1.fb22v1.1.fb22Andy Grover2017-01-041-1/+1
| | | | | | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* | Merge pull request #29 from mykaul/masterAndy Grover2016-12-051-1/+1
|\ \ | |/ |/| Fix import warning
| * Fix import warningYaniv Kaul2016-12-051-1/+1
|/ | | | | | | | [ykaul@ykaul examples]$ ./myshell ./myshell:20: UserWarning: 'configshell' package name for configshell-fb is deprecated, please instead import 'configshell_fb' import configshell Instead, importing configshell_fb as configshell
* Merge pull request #28 from cvubrugier/masterAndy Grover2016-10-281-1/+2
|\ | | | | Add Gentoo to the list of distributions that ship configshell-fb
| * Add Gentoo to the list of distributions that ship configshell-fbChristophe Vu-Brugier2016-10-281-1/+2
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Merge pull request #27 from cvubrugier/remove-packagingAndy Grover2016-10-2712-247/+7
|\ | | | | Remove build scripts for RPM and Debian packages
| * Remove build scripts for RPM and Debian packagesChristophe Vu-Brugier2016-10-2412-247/+7
|/ | | | | | | | | | | | | | | Remove the "debian" directory because Debian packagers have reported in the past that having a "debian" directory makes their work harder when merging the upstream branch into their packaging branch. Moreover, our in-repo Debian packaging is less complete than the Debian packaging. Also remove our in-repo RPM packaging which is probably lagging behind Fedora's packaging. Finally, this patch adds a few links to RPM and Debian build scripts in our README.md. Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* version 1.1.fb21v1.1.fb21Andy Grover2016-10-031-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #26 from pkalever/masterAndy Grover2016-10-011-1/+1
|\ | | | | fix typo in GENERALITIES section of help message
| * fix typo in GENERALITIES section of help messagePrasanna Kumar Kalever2016-10-011-1/+1
|/ | | | Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
* Merge pull request #24 from cvubrugier/masterAndy Grover2016-08-222-3/+3
|\ | | | | Change the URL of the GitHub repo to the open-iscsi organization
| * Change the URL of the GitHub repo to the open-iscsi organizationChristophe Vu-Brugier2016-08-222-3/+3
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Merge pull request #23 from jaredeh/for_upstreamAndy Grover2016-06-221-2/+2
|\ | | | | fixes to debian package build scripts
| * fixes to debian package build scriptsJared Hulbert2016-06-211-2/+2
|/
* Fix 'make rpm'Andy Grover2016-04-132-3/+4
| | | | | | Patch provided by https://github.com/rlenkala Signed-off-by: Andy Grover <agrover@redhat.com>
* version 1.1.fb20v1.1.fb20Andy Grover2016-04-071-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* configshell-fb: depend on python-six/python3-six in debian/controlChristoph Hellwig2016-03-231-2/+2
| | | | | Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Andy Grover <agrover@redhat.com>
* version 1.1.fb19v1.1.fb19Andy Grover2015-11-301-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #21 from cvubrugier/fix-ipv6-portal-completionAndy Grover2015-11-151-2/+2
|\ | | | | Fix completion of IPv6 portals
| * Fix completion of IPv6 portalsChristophe Vu-Brugier2015-11-151-2/+2
|/ | | | | | | | | | Allow square brackets in parameters and remove them from readline's set_completer_delims() so that readline does not remove the square brackets from the "text" argument in _complete_token_pparam() This patch fixes issue #20. Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Handle getting urwid focus position betterAndy Grover2015-11-131-8/+3
| | | | | | | | Experiencing weirdness when MainLoop is not broken out of with an ExitMainLoop exception. Change handler to do this, and then get the current focus position using the listbox.focus_position property. Signed-off-by: Andy Grover <agrover@redhat.com>
* update version to 1.1.fb18v1.1.fb18Andy Grover2015-06-231-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #19 from cvubrugier/simplify-completionAndy Grover2015-06-232-86/+32
|\ | | | | Simplify completion
| * Simplify the _display_completions() methodChristophe Vu-Brugier2015-06-112-84/+31
| | | | | | | | | | | | | | | | | | | | * remove the "completions in columns" configuration option * merge _display_completions() and _display_completions_python() * remove the hovering hint * simplify the logic that colorizes options * use a slice to format the possible completions in columns Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
| * Remove the complete_key attribute from class ConfigShellChristophe Vu-Brugier2015-06-111-2/+1
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Merge pull request #18 from cvubrugier/fix-completionAndy Grover2015-06-081-5/+2
|\ | | | | Fix the command line completion with Python 3 (issue #17)
| * Fix the command line completion with Python 3Christophe Vu-Brugier2015-06-081-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The command line completion is broken with Python 3 because the _display_completions() method raises a TypeError exception. The root cause is a comparison between a variable named "len" and two integer values. But because "len" is not defined at all, what is compared is the builtin len() function and two integers. Python 3 rightly raises a TypeError exception whereas Python 2.7 does not complain. The fix is just to remove the "if" block containing the faulty comparison because it always evaluates to False with Python 2.7 (len() is always greater than "num_per_line"). This patch fixes issue #17. Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
| * Fix integer divisions that are broken with Python 3Christophe Vu-Brugier2015-06-081-3/+2
|/ | | | | | | | | | | Replace '/' with '//' which the floor division operation on both Python 2 and Python 3. For more details, see PEP 238 "changing the division operator" http://www.python.org/dev/peps/pep-0238 Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* Merge pull request #16 from cvubrugier/python3-sixAndy Grover2015-06-037-36/+45
|\ | | | | Support Python 3 with "six" instead of running `2to3`
| * Support Python 3 with "six" instead of running `2to3`Christophe Vu-Brugier2015-06-037-36/+45
|/ | | | | | | | | | | | | | | * Replace dict.iteritems() with six.iteritems(dict) * Use six.string_types instead of basestring * Remove calls to unicode() in the handle_input() function called by Urwid: it is not necessary. However, during my tests, I observed that the Urwid listbox displayed for `cd` is broken: I see garbage on the screen when moving the cursor. * Use six.moves to import pickle on Python 3 or cPickle on Python 2 * Use six.moves to call input() on Python 3 or raw_input() on Python 2 * Add classifiers to setup.py * Sort imports Signed-off-by: Christophe Vu-Brugier <cvubrugier@fastmail.fm>
* update version to 1.1.fb17v1.1.fb17Andy Grover2015-01-131-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Add symlink from configshell_fb to configshell/Andy Grover2014-12-022-1/+1
| | | | | | | | | | We want configshell_fb/ to exist so we can more easily do prerelease development using PYTHONPATH. Remove package_dir from setup.py since with this change we don't need it any more. Signed-off-by: Andy Grover <agrover@redhat.com>
* Package as both 'configshell' and 'configshell_fb'Andy Grover2014-12-023-6/+11
| | | | | | | | | | | We have changed the API from Datera configshell, so we should be using a different package name to eliminate confusion. Still package under configshell pkg name, but issue a warning to switch. Change code in shell.py to use relative imports instead of package name. Signed-off-by: Andy Grover <agrover@redhat.com>
* version 1.1.fb16v1.1.fb16Andy Grover2014-12-021-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>