)]}'
{"aclmanager.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"ac498a6ec7ce97784ae7bad8b7da2df7f3eceb81","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    for project in config:"},{"line_number":68,"context_line":"        aclfilename \u003d project.get(\u0027acl-config\u0027)"},{"line_number":69,"context_line":"        if aclfilename:"},{"line_number":70,"context_line":"            (head, tail) \u003d os.path.split(aclfilename)"},{"line_number":71,"context_line":"            acl[project[\u0027project\u0027]] \u003d os.path.join(os.path.basename(head),"},{"line_number":72,"context_line":"                                                   tail)"},{"line_number":73,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_24569929","line":70,"updated":"2016-08-15 19:27:49.000000000","message":"Will the path always only have 2 parts?","commit_id":"9269e97af55c51fabca1a79587c920dd922bbf99"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"131c84f4c7250c11cdb3ddb08d8de61edef9a231","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    for project in config:"},{"line_number":68,"context_line":"        aclfilename \u003d project.get(\u0027acl-config\u0027)"},{"line_number":69,"context_line":"        if aclfilename:"},{"line_number":70,"context_line":"            (head, tail) \u003d os.path.split(aclfilename)"},{"line_number":71,"context_line":"            acl[project[\u0027project\u0027]] \u003d os.path.join(os.path.basename(head),"},{"line_number":72,"context_line":"                                                   tail)"},{"line_number":73,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_980873db","line":70,"in_reply_to":"3ac371cc_24569929","updated":"2016-08-16 09:43:42.000000000","message":"The path is the full path on the Gerrit host as specified in the projects.yaml file. Currently contains values like \"/home/gerrit2/acls/openstack/akanda.config\" -- the goal of the code here is to extract \"openstack/akanda.config\".\n\nSince our repository naming is using the GitHub convention (org/repo) I think we are safe assuming these have 2 components.","commit_id":"9269e97af55c51fabca1a79587c920dd922bbf99"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"cffe9b5e01dd7304e6d12a4829995500c536c0e5","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    for project in config:"},{"line_number":68,"context_line":"        aclfilename \u003d project.get(\u0027acl-config\u0027)"},{"line_number":69,"context_line":"        if aclfilename:"},{"line_number":70,"context_line":"            (head, tail) \u003d os.path.split(aclfilename)"},{"line_number":71,"context_line":"            acl[project[\u0027project\u0027]] \u003d os.path.join(os.path.basename(head),"},{"line_number":72,"context_line":"                                                   tail)"},{"line_number":73,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_02c1982e","line":70,"in_reply_to":"3ac371cc_980873db","updated":"2016-08-16 16:50:58.000000000","message":"I always forget that os.path.split() returns 2 items. I keep thinking it works like string.split().","commit_id":"9269e97af55c51fabca1a79587c920dd922bbf99"}]}