)]}'
{"oslosphinx/__init__.py":[{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"3b9d29fe334b863e35747c5f18fbdac01018612c","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import os"},{"line_number":16,"context_line":"import subprocess"},{"line_number":17,"context_line":"import urlparse"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"CGIT_BASE \u003d \u0027http://git.openstack.org/cgit/\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_a43df9bc","line":17,"updated":"2015-09-28 08:42:12.000000000","message":"This will not work with Python 3, where urlparse has been renamed into urllib.parse","commit_id":"efec9c68fe2fcc51c147c0b20228a57be60ff2db"},{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"7ca3a54ed81ca3e573bc32ca57bd8afa9a335e9d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    if _cgit_link is None:"},{"line_number":27,"context_line":"        try:"},{"line_number":28,"context_line":"            git_remote \u003d subprocess.check_output("},{"line_number":29,"context_line":"                [\u0027git\u0027, \u0027config\u0027, \u0027--local\u0027, \u0027--get\u0027, \u0027remote.origin.url\u0027]"},{"line_number":30,"context_line":"            )"},{"line_number":31,"context_line":"        except subprocess.CheckedProcessException:"},{"line_number":32,"context_line":"            _cgit_link \u003d \u0027unknown\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_5bbb78e5","line":29,"updated":"2015-09-28 20:33:25.000000000","message":"This adds runtime dependency on git command which I\u0027m not sure how can be declared upstream but in must be added to the distro packaging\ne.g. https://review.gerrithub.io/248513","commit_id":"efec9c68fe2fcc51c147c0b20228a57be60ff2db"},{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"d29fe1604990010c092c543bd6e8fd9f305dd7fc","unresolved":false,"context_lines":[{"line_number":28,"context_line":"            git_remote \u003d subprocess.check_output("},{"line_number":29,"context_line":"                [\u0027git\u0027, \u0027config\u0027, \u0027--local\u0027, \u0027--get\u0027, \u0027remote.origin.url\u0027]"},{"line_number":30,"context_line":"            )"},{"line_number":31,"context_line":"        except subprocess.CheckedProcessException:"},{"line_number":32,"context_line":"            _cgit_link \u003d \u0027unknown\u0027"},{"line_number":33,"context_line":"        else:"},{"line_number":34,"context_line":"            parsed \u003d urlparse.urlparse(git_remote)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_2e97ac91","line":31,"updated":"2015-09-25 17:00:05.000000000","message":"from https://review.openstack.org/227869\n\"subprocess.CheckedProcessException, which does not exist. From\nthe Python documentation, it should be\nsubprocess.CalledProcessError [1]\n\n[1] - https://docs.python.org/2/library/subprocess.html#subprocess.CalledProcessError\n\"","commit_id":"efec9c68fe2fcc51c147c0b20228a57be60ff2db"}]}