)]}'
{"tox.ini":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"9d24079478d613a1869c2fc33ea7fea94eaaae3f","unresolved":false,"context_lines":[{"line_number":41,"context_line":"# NOTE(mriedem): If py35 fails with \"db type could not be determined\", delete"},{"line_number":42,"context_line":"# .testrepository and try again. Running py35 before py27 is OK, but not the"},{"line_number":43,"context_line":"# other way around. See: https://bugs.launchpad.net/testrepository/+bug/1212909"},{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_2424145e","line":44,"updated":"2016-07-01 10:17:40.000000000","message":"if you take python 3.4 as the reference, i suggest to write \"setenv \u003d {[testenv:py34]setenv}\" instead of duplicating the config.","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"166e0df7002a7ae63cd980765b603b311bd91458","unresolved":false,"context_lines":[{"line_number":41,"context_line":"# NOTE(mriedem): If py35 fails with \"db type could not be determined\", delete"},{"line_number":42,"context_line":"# .testrepository and try again. Running py35 before py27 is OK, but not the"},{"line_number":43,"context_line":"# other way around. See: https://bugs.launchpad.net/testrepository/+bug/1212909"},{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_c561c73a","line":44,"in_reply_to":"3aaa91ec_2424145e","updated":"2016-07-01 12:08:13.000000000","message":"That would be a double indirection, I think it is nicer this way.","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"9d24079478d613a1869c2fc33ea7fea94eaaae3f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# .testrepository and try again. Running py35 before py27 is OK, but not the"},{"line_number":43,"context_line":"# other way around. See: https://bugs.launchpad.net/testrepository/+bug/1212909"},{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_c4d71021","line":45,"updated":"2016-07-01 10:17:40.000000000","message":"same here:\n\ncommands \u003d {[testenv:py34]commands}\n\nBy the way, here there a way in tox.ini to \"inherit\" from a different testenv instead of the base [testenv]?","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"166e0df7002a7ae63cd980765b603b311bd91458","unresolved":false,"context_lines":[{"line_number":42,"context_line":"# .testrepository and try again. Running py35 before py27 is OK, but not the"},{"line_number":43,"context_line":"# other way around. See: https://bugs.launchpad.net/testrepository/+bug/1212909"},{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_e009512c","line":45,"in_reply_to":"3aaa91ec_c4d71021","updated":"2016-07-01 12:08:13.000000000","message":"But I agree with you here, will change as proposed.\n\nIndeed it would be even nicer if one could just say\n\n[testenv:py35]\ncopyeverythingfromenv \u003d py34\n\nbut I don\u0027t think that that is currently possible. The other option, as we discussed on IRC, would be to only define a\n\n[testenv:py3]\n\nand call that from both python34 and python35 jobs. But that would need special casing the jobs definitions for nova, so I\u0027m not sure that this would be the better solution when viewed globally.","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"0498bbd33763c915c4c61ed6a849e9500825f9ae","unresolved":false,"context_lines":[{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"[testenv:functional]"},{"line_number":50,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_2aa05a8f","line":47,"updated":"2016-07-01 11:44:30.000000000","message":"If we use \"ostestr\" right from the start we could use the blacklist feature like we do for the py34 task [1].\n\n[1] https://github.com/openstack/nova/commit/9537c8576324ebe25982723158ae83664f79b734#diff-b91f3d5bd63fcd17221b267e851608e8R42","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"166e0df7002a7ae63cd980765b603b311bd91458","unresolved":false,"context_lines":[{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"[testenv:functional]"},{"line_number":50,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_a5163ba4","line":47,"in_reply_to":"3aaa91ec_2aa05a8f","updated":"2016-07-01 12:08:13.000000000","message":"That is excatly what tools/pretty_tox3.sh does, which I introduced just recently in https://review.openstack.org/333274","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":11303,"name":"Markus Zoeller (markus_z)","email":"mzoeller@linux.vnet.ibm.com","username":"mzoeller"},"change_message_id":"ed8583d30128a8ef5b7854efe41f28b2dd0d02b7","unresolved":false,"context_lines":[{"line_number":44,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":45,"context_line":"commands \u003d"},{"line_number":46,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":47,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"[testenv:functional]"},{"line_number":50,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"3aaa91ec_c061b53a","line":47,"in_reply_to":"3aaa91ec_a5163ba4","updated":"2016-07-01 12:22:42.000000000","message":"Ah, \"pretty_tox*3*\", I did overlook that.","commit_id":"f058b1de53d50c79ef323c67c6da555775039d1a"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b59f12364700584c486253fa1598a786a887a3c3","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"[testenv:py35]"},{"line_number":41,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":42,"context_line":"commands \u003d {[testenv:py34]commands}"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"[testenv:functional]"},{"line_number":45,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":2,"id":"3aaa91ec_e6ea0274","line":42,"updated":"2016-07-01 15:05:12.000000000","message":"Hmm, too bad, this doesn\u0027t work, as tox cannot handle the \u0027{posargs}\u0027 argument recursively","commit_id":"eb31b0b3e556d053de5e8d72c5440736c3434252"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"8d419b674e949cb172e6bc0304e2e1b82b885fbb","unresolved":false,"context_lines":[{"line_number":37,"context_line":"  find . -type f -name \"*.pyc\" -delete"},{"line_number":38,"context_line":"  bash tools/pretty_tox3.sh \u0027{posargs}\u0027"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"[testenv:py35]"},{"line_number":41,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":42,"context_line":"commands \u003d"},{"line_number":43,"context_line":"  find . -type f -name \"*.pyc\" -delete"}],"source_content_type":"text/x-properties","patch_set":3,"id":"3aaa91ec_015578f9","line":40,"updated":"2016-07-01 15:09:31.000000000","message":"Might want to transcribe mriedem\u0027s NOTE here as well, if we\u0027re going to eventually do away with the py34 environment.  Unless that comment is outdated?","commit_id":"df0aa8acaad746dde19b26bc9724379c3de9c30a"}]}