)]}'
{"setup.py":[{"author":{"_account_id":917,"name":"Chuck Thier","email":"cthier@gmail.com","username":"cthier"},"change_message_id":"fae030ede160df0706b073ec7fdc98a566d1a89d","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"with open(\u0027tools/pip-requires\u0027, \u0027r\u0027) as f:"},{"line_number":26,"context_line":"    requires \u003d f.read().split(\u0027\\n\u0027)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"setup("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3sM%3D","line":26,"updated":"2012-12-11 16:50:36.000000000","message":"Really minor nit:  This leaves an extra blank entry in the list (\"\") which setuptools seems to ignore, but we could do something like:\n\nrequires\u003d[l.strip(\u0027\\n\u0027) for l in f.readlines()]\n\nWhich would not have the empty string in the requires list","commit_id":"c535fb56c4f4cd136adaa3834c2632203a9e0197"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"f3d6d41f9c4ee902ec23b3c98c10582388359210","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"with open(\u0027tools/pip-requires\u0027, \u0027r\u0027) as f:"},{"line_number":26,"context_line":"    requires \u003d f.read().split(\u0027\\n\u0027)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"setup("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F25k%3D","line":26,"in_reply_to":"AAAAMX%2F%2F3mY%3D","updated":"2012-12-12 10:18:52.000000000","message":"Done","commit_id":"c535fb56c4f4cd136adaa3834c2632203a9e0197"},{"author":{"_account_id":1216,"name":"gholt","email":"z-launchpad@brim.net","username":"gholt"},"change_message_id":"976debadfcad15b7ba7a3e7697fa376c09f8b93a","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"with open(\u0027tools/pip-requires\u0027, \u0027r\u0027) as f:"},{"line_number":26,"context_line":"    requires \u003d f.read().split(\u0027\\n\u0027)"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"setup("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMX%2F%2F3mY%3D","line":26,"in_reply_to":"AAAAMX%2F%2F3sM%3D","updated":"2012-12-11 17:54:37.000000000","message":"or\nrequires \u003d [x.strip() for x in f if x.strip()]\nBut I tease...","commit_id":"c535fb56c4f4cd136adaa3834c2632203a9e0197"}]}