)]}'
{"doc/source/testing/serial-console.rst":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"5bfaa9b9c1cfcb7f07ab0b59535febe01d5921e9","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"  # sudo easy_install ws4py || sudo pip install ws4py"},{"line_number":48,"context_line":"  # cat \u003e\u003e client.py \u003c\u003cEOF"},{"line_number":49,"context_line":"  from __future__ import print_function"},{"line_number":50,"context_line":"  import sys"},{"line_number":51,"context_line":"  from ws4py.client.threadedclient import WebSocketClient"},{"line_number":52,"context_line":"  class LazyClient(WebSocketClient):"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_575636d3","line":49,"updated":"2015-07-20 22:14:38.000000000","message":"I think this can be left out of documentation.  Besides, it\u0027s not needed for the below print usages anyway.","commit_id":"d5fb869db02f683bfd603483d171de130d32eeff"}],"tools/xenserver/cleanup_sm_locks.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"5bfaa9b9c1cfcb7f07ab0b59535febe01d5921e9","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"This script should be run on the dom0 of the affected machine."},{"line_number":26,"context_line":"\"\"\""},{"line_number":27,"context_line":"from __future__ import print_function"},{"line_number":28,"context_line":"import errno"},{"line_number":29,"context_line":"import optparse"},{"line_number":30,"context_line":"import os"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_37304a9e","line":27,"updated":"2015-07-20 22:14:38.000000000","message":"I believe all these tools are meant to run on the Xen host.  I am not sure what version of Python is currently installed on Xen, but I\u0027m pretty confident it\u0027s not Python 3, and not likely to be Python 3 in the near term.  Last I looked, Python 2.4 was the version installed; that *might* have been bumped to 2.6 in the most recent XenServer release.  All this to say, we should be extra careful making any modifications to scripts in a \"xenserver\" directory, particularly given that there is no test suite for these scripts.","commit_id":"d5fb869db02f683bfd603483d171de130d32eeff"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"5bfaa9b9c1cfcb7f07ab0b59535febe01d5921e9","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    if not os.path.exists(BASE):"},{"line_number":69,"context_line":"        print(\"error: \u0027%s\u0027 doesn\u0027t exist. Make sure you\u0027re\"\\"},{"line_number":70,"context_line":"                     \" running this on the dom0.\" % BASE, file \u003d sys.stderr)"},{"line_number":71,"context_line":"        sys.exit(1)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    lockpaths_removed \u003d 0"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_77ed72ee","line":70,"updated":"2015-07-20 22:14:38.000000000","message":"This isn\u0027t the correct style anyway.  First, the addition of parentheses eliminates the need for a backslash continuation, so that should be removed.  Second, keyword parameters to functions are stylistically expressed without extra whitespace, e.g. \"file\u003dsys.stderr\".  And finally, I rather suspect that Python 2.4 does not understand print as a function.","commit_id":"d5fb869db02f683bfd603483d171de130d32eeff"}]}
