Add support for nesting /console/ in a sub url in console-cli.py and PEP-8 Python style compliance

[sodul]

Replaces pull requests 176 and 179

[ypujante]

Merged in 4.6.1

[ypujante]

Hi Stephane.

Debug is not working anymore :(

Traceback (most recent call last):
  File "./bin/console-cli.py", line 280, in <module>
    sys.exit(main())
  File "./bin/console-cli.py", line 178, in main
    gluconsole.rest.log.setLevel(logging.DEBUG)
AttributeError: 'module' object has no attribute 'log'

Could you take a look?

Thanks
Yan

[sodul]

Will do!

Sent from my iPhone

On Mar 20, 2013, at 2:13 PM, Yan Pujante notifications@github.com wrote:

Hi Stephane.

Debug is not working anymore :(

Traceback (most recent call last):
File "./bin/console-cli.py", line 280, in
sys.exit(main())
File "./bin/console-cli.py", line 178, in main
gluconsole.rest.log.setLevel(logging.DEBUG)
AttributeError: 'module' object has no attribute 'log'
Could you take a look?

Thanks
Yan

—
Reply to this email directly or view it on GitHub.

[ypujante]

I found the culprit and fixed it...

diff --git a/console/org.linkedin.glu.console-cli/src/cmdline/resources/bin/console-cli.py b/console/org.linkedin.glu.console-cli/src/cmdline/resources/bin/console-cli.py
index f744970..4b708f5 100755
--- a/console/org.linkedin.glu.console-cli/src/cmdline/resources/bin/console-cli.py
+++ b/console/org.linkedin.glu.console-cli/src/cmdline/resources/bin/console-cli.py
@@ -175,7 +175,7 @@ def main():

     if options.debug:
         log.setLevel(logging.DEBUG)
-        gluconsole.rest.log.setLevel(logging.DEBUG)
+        gluconsole.rest.logger.setLevel(logging.DEBUG)

     if options.filter and options.filter_filename:
         parser.error('Only --systemFilter or --systemFilterFile must be'

[ypujante]

Fixed with glu 4.7.0