Irit Goihman has posted comments on this change. Change subject: misc.py py3 compliance via six ......................................................................
Patch Set 2: Code-Review-1 (4 comments) https://gerrit.ovirt.org/#/c/63192/2//COMMIT_MSG Commit Message: PS2, Line 7: misc.py py3 compliance via six please use this commit message format: topic: short summary longer description (if needed) https://gerrit.ovirt.org/#/c/63192/2/lib/vdsm/storage/misc.py File lib/vdsm/storage/misc.py: PS2, Line 70: six.moves.imap i think it's six.moves.map, please check again PS2, Line 668: six.moves.queue it will be more readable to import six.moves.queue and six.moves.map than call the full name of the module. PS2, Line 698: xrange xrange is removed in python 3, please replace it also -- To view, visit https://gerrit.ovirt.org/63192 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I151c627efd0836bc6ffdf1d4126f504cf378a68c Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Leon Goldberg <leon.ot...@gmail.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org