summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* Don't reuse "parameter" local variableAndy Grover2014-11-191-3/+3
| | | | | | | | Later on in the function, we assume "parameter" is still the dict that was passed in. Reported-by: Fabian Deutsch <fdeutsch@redhat.com> Signed-off-by: Andy Grover <agrover@redhat.com>
* update versionv1.1.fb15Andy Grover2014-09-241-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Fix exception on sortingAndy Grover2014-09-221-1/+1
| | | | | | | | | | | | | | | A node named "foo4" will match the regexp in sorting_keys and will sort by ("foo", 4). A node named "foo" will not match and will sort by ("foo", None). This is incorrect because None is not orderable with integers. In the second case, having the second element in the tuple be 0 does the right thing. Fixes #14 Reported-by: Arthur Lutz Signed-off-by: Andy Grover <agrover@redhat.com>
* update versionv1.1.fb14Andy Grover2014-08-281-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #13 from cvubrugier/masterAndy Grover2014-07-011-4/+4
|\ | | | | Fix ui_command_get() when no parameter is provided
| * Fix ui_command_get() when no parameter is providedChristophe Vu-Brugier2014-07-011-4/+4
|/ | | | | | | | | | | | | | | | The "get attribute" command displays an error message after the list of attributes. For instance, in the TPG context, the following error is displayed: No parameter 't' in group 'attribute' This happens because the 'parameter' argument in ui_command_get() is overwritten in the loop that iterates over the list of parameters to display their value. This patch fixes the issue by renaming the variable in the loop so that the value of the original 'parameter' variable is preserved. Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
* update version to 1.1.fb13v1.1.fb13Andy Grover2014-03-251-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Don't use elif when previous conditional returnsAndy Grover2014-03-141-1/+2
| | | | | | | | | | To follow-on Christophe's previous fix, now that the previous conditional returns, it's a little easier to follow (IMHO) if the next conditional is an 'if' instead of an 'elif'. Also, add an empty line. Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #11 from cvubrugier/masterAndy Grover2014-03-141-1/+1
|\ | | | | Fix exception raised when 'help' is invoked
| * Fix exception raised when 'help' is invokedChristophe Vu-Brugier2014-03-141-1/+1
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
* update version to 1.1.fb12v1.1.fb12Andy Grover2014-03-141-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Another dumb thing to fix.Andy Grover2014-03-031-1/+1
| | | | | | Arrg. Can't reference self in a default parameter. Signed-off-by: Andy Grover <agrover@redhat.com>
* Remove unneeded sys importAndy Grover2014-03-031-1/+0
| | | | | | | An earlier version of the commit used it, but its use was removed in the final commit. Whups. Signed-off-by: Andy Grover <agrover@redhat.com>
* Make error-level log messages go to stderrAndy Grover2014-03-032-8/+19
| | | | | | | | | | Extend con.display() with an error boolean optional parameter. If true, output goes to stderr. Is this in log._log() to send messages >= 'error' to stderr. This is based on work by Tregaron Bayly and Jason Earl at Bluehost, thanks. Signed-off-by: Andy Grover <agrover@redhat.com>
* Let exceptions through to callers of run_interactiveAndy Grover2014-03-032-193/+182
| | | | | | | | | | | | | Change logic in run_interactive() to not recursively call itself, and to let exceptions through to callers. Raising ExecutionError instead of printing a message but not raising an exception is now allowed. This lets us reorganize a few functions for less nested logic. Change log.error()s to exceptions. Signed-off-by: Andy Grover <agrover@redhat.com>
* update version to 1.1.11v1.1.fb11Andy Grover2014-01-061-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Re-enable hack for tab-completion with nothing under the cursorAndy Grover2014-01-061-2/+8
| | | | | | | | | | | | | | | | | | A very nice behavior is having <tab><tab> bring up options for the next expected commandline token when nothing is underneath the cursor. There was a workaround to get this working with the simpleparse parser, and we need a different workaround for it to work with pyparsing. This keeps the same technique of adding a bogus input and seeing what the parser would categorize it as, and then setting current_token based on that. We also ensure current_token is defined. Without this, the call to _dispatch_completion fails with a NameError but somebody catches the error so things just silently don't work, without the debug msg in _dispatch_completion for unknown current_token ever printing. Signed-off-by: Andy Grover <agrover@redhat.com>
* update version to 1.1.10v1.1.fb10Andy Grover2013-10-301-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Update README.md for in-repo pkgscript rationaleAndy Grover2013-10-251-0/+10
| | | | | | See https://github.com/agrover/configshell-fb/pull/8 comments for more. Signed-off-by: Andy Grover <agrover@redhat.com>
* Update maintainer email in debian/controlAndy Grover2013-10-221-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #10 from JonnyJD/make_debAndy Grover2013-10-2015-110/+99
|\ | | | | update debian packaging
| * debian: also build python 3 packageJohannes Dewender2013-10-206-6/+55
| | | | | | | | | | | | This splits an exatra -docs package, too. Signed-off-by: Johannes Dewender <github@JonnyJD.net>
| * debian: update packaging to use dh_pythonJohannes Dewender2013-10-209-84/+17
| | | | | | | | Signed-off-by: Johannes Dewender <github@JonnyJD.net>
| * debian: fix dependenciesJohannes Dewender2013-10-181-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | epydoc is needed for building the docs and that again needs all usual dependencies. On the other side epydoc isn't needed on runtime anymore. This should be backwards compatible to configshell, but targetcli-fb needs this version instead of the original. Signed-off-by: Johannes Dewender <github@JonnyJD.net>
| * fix package name in debian/rulesJohannes Dewender2013-10-181-2/+3
| | | | | | | | Signed-off-by: Johannes Dewender <github@JonnyJD.net>
| * debian: fix setuptools usageJohannes Dewender2013-10-182-1/+4
| | | | | | | | Signed-off-by: Johannes Dewender <github@JonnyJD.net>
| * rename debian package to configshell-fbJohannes Dewender2013-10-1810-27/+29
| | | | | | | | | | | | This is an alternative to the original configshell. Signed-off-by: Johannes Dewender <github@JonnyJD.net>
* | Merge pull request #9 from cvubrugier/masterAndy Grover2013-10-181-1/+1
|\ \ | |/ |/| Fix an integer division on Python 3
| * Fix an integer division on Python 3Christophe Vu-Brugier2013-10-181-1/+1
|/ | | | | | | | | | | | | | | | | With Python 2, '/' returns the floor of the mathematical result of division if the arguments are ints. With Python 3, the operator returns a reasonable approximation of the mathematical result of the division. This commit uses '//', which is the floor division operator in both Python 2 and Python 3, so that the result can be safely used to index an array. 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@yahoo.fr>
* Merge pull request #8 from JonnyJD/make_debAndy Grover2013-10-161-3/+3
|\ | | | | fix version in package builds
| * fix version in package buildsJohannes Dewender2013-10-161-3/+3
|/ | | | | | | Without this fix at least dpkg complains about the version not starting with a number (starting with "v"). Signed-off-by: Johannes Dewender <github@JonnyJD.net>
* Merge pull request #6 from cvubrugier/masterAndy Grover2013-10-123-58/+12
|\ | | | | Perform some cleanup and state that configshell-fb is compatible with Python 3
| * Use 2to3 in setup and mention that configshell-fb is Python 3 compatibleChristophe Vu-Brugier2013-10-122-1/+4
| | | | | | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
| * Remove unused parameters in _complete_token_{command,path}()Christophe Vu-Brugier2013-10-121-20/+5
| | | | | | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
| * Remove the readline completion hook fallback used with Python < 2.6Christophe Vu-Brugier2013-10-121-37/+3
|/ | | | Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
* Merge pull request #5 from cvubrugier/2to3Andy Grover2013-10-112-10/+10
|\ | | | | 2to3: preparatory work for python 3
| * Comply with PEP 3310 "Catching Exceptions in Python 3000"Christophe Vu-Brugier2013-10-082-10/+10
|/ | | | | | Generated with `2to3 -f except`. Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
* update version to 1.1.fb9v1.1.fb9Andy Grover2013-09-121-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge pull request #4 from cvubrugier/pyparsing-v2Andy Grover2013-09-123-50/+50
|\ | | | | Replace simpleparse with pyparsing
| * Replace simpleparse with pyparsingChristophe Vu-Brugier2013-09-123-50/+50
|/ | | | | | | | | | | | Configshell uses simpleparse to parse the command line. Unfortunately, simpleparse does not seem to be maintained anymore: the last release was three years ago. Moreover, simpleparse is not widely used (on Debian, only configshell depends on it). On the other hand, pyparsing is actively maintained, widely used and ready for Python 3. So let's use it. Signed-off-by: Christophe Vu-Brugier <cvubrugier@yahoo.fr>
* Put package information in setup.py instead of importing configshell's initChristophe Vu-Brugier2013-08-282-26/+10
| | | | | | Also update the package name, version, URL and maintainer. Signed-off-by: Andy Grover <agrover@redhat.com>
* README.md was missed by relicensing mergev1.1.fb8Andy Grover2013-07-231-1/+1
| | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* Merge branch 'upstream' into devAndy Grover2013-07-2313-779/+286
|\ | | | | | | | | Conflicts: README
| * configshell: Change copyright to Apache License 2.01.2upstreamNicholas Bellinger2013-07-0414-794/+286
| | | | | | | | | | | | | | | | | | This patch converts configshell code copyright from AGPL to the permissive Apache License 2.0. It also update copyright owner to Datera, Inc. Signed-off-by: Nicholas Bellinger <nab@risingtidesystems.com>
* | Make sure help ends with carriage returnAndy Grover2013-06-041-0/+1
| | | | | | | | Signed-off-by: Andy Grover <agrover@redhat.com>
* | Fix build scripts for renamed READMEAndy Grover2013-05-072-2/+2
| | | | | | | | | | Reported-by: bsyd Signed-off-by: Andy Grover <agrover@redhat.com>