)]}'
{"pbr/tests/test_packaging.py":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"8f0b18c08f4d863fea0399f921a27ec256193e53","unresolved":false,"context_lines":[{"line_number":56,"context_line":"import testtools"},{"line_number":57,"context_line":"from testtools import matchers"},{"line_number":58,"context_line":"import virtualenv"},{"line_number":59,"context_line":"from wheel import wheelfile"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"from pbr import git"},{"line_number":62,"context_line":"from pbr import packaging"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_d1e7630f","line":59,"updated":"2018-10-04 22:39:02.000000000","message":"This import is specifically happening in the pbr testsuite, and a minimum version constraint is added to the test-requirements.txt file accordingly.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"b032b5a7f5cf74d68ef6a4cd14abf50057fe9aec","unresolved":false,"context_lines":[{"line_number":56,"context_line":"import testtools"},{"line_number":57,"context_line":"from testtools import matchers"},{"line_number":58,"context_line":"import virtualenv"},{"line_number":59,"context_line":"from wheel import wheelfile"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"from pbr import git"},{"line_number":62,"context_line":"from pbr import packaging"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_f16dbf9a","line":59,"updated":"2018-10-04 22:35:06.000000000","message":"This is a new file in wheel, so we\u0027re going to have to have a minimum version constraint to work with it. Or we need to support both code paths.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"}],"setup.cfg":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"ce9c9daff3bc289926db27316f898c40f7de66b1","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_5e5b6285","line":56,"updated":"2018-10-04 22:01:16.000000000","message":"I\u0027m not suggesting that all projects should be updated any time in the near future, simply asking that we not reject this particular PBR change simply because it\u0027s following the configuration wheel upstream expects.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"6a48f68a04bdb80cd892fc41a527add07a385830","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_35d238d1","line":56,"updated":"2018-10-05 11:56:30.000000000","message":"There are plenty of PBR-using projects which aren\u0027t OpenStack projects and won\u0027t be using OpenStack\u0027s special release process with magic flags passed to the wheel build rather than documented in their respective repositories. So I guess it depends on who \"we\" is.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"bd7a76a204c89e7a44f577d4019001bb5879a844","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_89d47820","line":56,"updated":"2018-10-04 10:26:31.000000000","message":"does this mean we need to change all setup.cfg files?","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"bbfb409706dfaa264e003eb6f1fbd0e6e96470bc","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_fe77ce3b","line":56,"in_reply_to":"3f79a3b5_343a50e6","updated":"2018-10-04 21:51:00.000000000","message":"I\u0027ll let someone who wants that deal with the 400 * N-branches patches that will take.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d5240678a42c88df3cb93711c049eb9d639a1962","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_343a50e6","line":56,"in_reply_to":"3f79a3b5_346fb0c5","updated":"2018-10-04 15:24:38.000000000","message":"\u003e But that doesn\u0027t mean we shouldn\u0027t *also* update setup.cfg files so\n \u003e that local builds will also work as intended.\n\nAgreed. This reminds me of the \"using \u0027sphinx-build\u0027 in the gate vs. \u0027tox -e docs\u0027\" situation. That meant we end up in situations where local builds were no longer possible. I\u0027d rather we fixed this everywhere, even if it\u0027s noisy","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"b63a5ca8d8c81d800beeaa936b7b42ccc1e91d51","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_15229c03","line":56,"in_reply_to":"3f79a3b5_5e5b6285","updated":"2018-10-05 11:50:55.000000000","message":"I think we don\u0027t need to worry about that. http://lists.openstack.org/pipermail/openstack-dev/2018-October/135461.html","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"6defe03e6752eebccf1c13779886dbf4b10cf1c8","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_8cd48620","line":56,"in_reply_to":"3f79a3b5_8932380e","updated":"2018-10-04 11:14:12.000000000","message":"I proposed a couple of patches to update the packaging job so we don\u0027t have to touch every setup.cfg file. https://review.openstack.org/607902 and https://review.openstack.org/607900","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7921ea6ba64f2a49973b45936cbe3a80a6f30a09","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_8932380e","line":56,"in_reply_to":"3f79a3b5_89d47820","updated":"2018-10-04 10:56:41.000000000","message":"Eventually, yes. However, this has only just been introduced so we should have some lead time on this. In addition, the new option has been supported for \u003e 4 years (*Trusty* packages a version that supports it) so backwards compatibility shouldn\u0027t be an issue.\n\nhttps://github.com/pypa/wheel/commit/f7c9878712390414c03c64e9afa55ea4f30e965b","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c5843cd79075ded06215934ede18d1f9a521bc67","unresolved":false,"context_lines":[{"line_number":53,"context_line":"source-dir \u003d doc/source"},{"line_number":54,"context_line":"warning-is-error \u003d 1"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"[bdist_wheel]"},{"line_number":57,"context_line":"universal \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"3f79a3b5_346fb0c5","line":56,"in_reply_to":"3f79a3b5_8cd48620","updated":"2018-10-04 14:54:13.000000000","message":"But that doesn\u0027t mean we shouldn\u0027t *also* update setup.cfg files so that local builds will also work as intended.","commit_id":"f14a3b2b73c6c95edbea8de799cf1df07decdf0e"}]}