)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22588,"name":"Naoya Harada","email":"n-harada@vx.jp.nec.com","username":"n-harada"},"change_message_id":"f32765f271ae6f40d255610c6154977b36b3257c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Revert \"Remove unused import library\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Directive \"from __future__ import print_function\" is not an actual import, but it enables print statement in py2 to behave like a real function in py3."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This allows us to use print() function in py2/py3 compatible way."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_625ec377","line":9,"updated":"2016-10-12 09:42:10.000000000","message":"This line is too long. Please see [1].\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"cf8332344fef1691f8ecbf7e8a7490ec23c6ca17"},{"author":{"_account_id":18401,"name":"Luka Krajger","email":"luka.krajger@gmail.com","username":"KLuka"},"change_message_id":"ec1f75e64da3f1c0d1643301283a691d9fc85e35","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Revert \"Remove unused import library\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Directive \"from __future__ import print_function\" is not an actual import, but it enables print statement in py2 to behave like a real function in py3."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This allows us to use print() function in py2/py3 compatible way."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_f31627f0","line":9,"in_reply_to":"fa6399be_625ec377","updated":"2016-10-12 10:39:02.000000000","message":"Done","commit_id":"cf8332344fef1691f8ecbf7e8a7490ec23c6ca17"}],"tackerclient/tacker/v1_0/vm/vnfd.py":[{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"f8826973e20b70928fe6368426f5124a11db99ef","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from __future__ import print_function"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_serialization import jsonutils"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_b20a8988","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":37},"updated":"2016-10-14 05:32:24.000000000","message":"I am not seeing any impact of this patch in current scope.","commit_id":"82e9d0507bbc58742906524273bdbf450a428dd8"},{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"3e73c5cbdcdce9d6f6e4f4d0948d6a96f8196a81","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from __future__ import print_function"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_serialization import jsonutils"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_173c07c7","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":37},"updated":"2016-10-14 04:01:05.000000000","message":"In python 2.6, we use print statement as:\nprint \u003cvalue\u003e\n\nIn python 3.0, we use it as as function:\nprint ()\n\nBut in python 2.7, we can use both above statements.\n\n\nIMO, To get the python 3.0 print function in python 2.6 we add this import for compatibility.\n\nBut since now we are not supporting python 2.6, then why we need this import?\nIt looks meaningless.\nKindly confirm if I missed something.","commit_id":"82e9d0507bbc58742906524273bdbf450a428dd8"},{"author":{"_account_id":18401,"name":"Luka Krajger","email":"luka.krajger@gmail.com","username":"KLuka"},"change_message_id":"eb750777e53bb67e71a4e76c581f781e24bf71e3","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from __future__ import print_function"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_serialization import jsonutils"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_1d06e51c","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":37},"in_reply_to":"fa6399be_b20a8988","updated":"2016-10-14 07:34:33.000000000","message":"You are correct that currently code works in both versions. But it will work this way only when 1 parameter is passed to print function.\n\n\u003e But in python 2.7, we can use both above statements.\n\n\nThis is not true. Try this without future import:\n  \n# Python 2.7\n\u003e\u003e\u003e print(\"a\")\na\n\u003e\u003e\u003e print(\"a\", \"b\")\n(\u0027a\u0027, \u0027b\u0027) # here we actually print tuple with two elements\n  \n# Python 3.4\n\u003e\u003e\u003e print(\"a\")\na\n\u003e\u003e\u003e print(\"a\", \"b\")\na b\n\nThis PS will come handy if someone changes the call to print function so that it uses more parameters, or something like that...\n\nAlso all other files where print function is used use this import. Please see [1], [2].\n\nPlease let me know what to do?\n\n[1] https://github.com/openstack/python-tackerclient/blob/master/tackerclient/shell.py\n[2] https://github.com/openstack/python-tackerclient/blob/master/tackerclient/tacker/v1_0/__init__.py","commit_id":"82e9d0507bbc58742906524273bdbf450a428dd8"}]}
