Yaniv Bronhaim has posted comments on this change. Change subject: spec: Remove the lib64 vdsm python directory ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/50115/2/vdsm.spec.in File vdsm.spec.in: Line 338: Line 339: %post python Line 340: # Remove vdsm python lib only on upgrade Line 341: if [ "$1" -ge 2 ]; then Line 342: %define old_python_path /usr/lib64/python2.7/site-packages/vdsm > Did you remove also the __init__.pyc file? We must remove both to make the mmm right :) Line 343: if [ -d "%{old_python_path}" ]; then Line 344: logger 'Removing "%{old_python_path}"' Line 345: rm -rf "%{old_python_path}" Line 346: fi -- To view, visit https://gerrit.ovirt.org/50115 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9dc611cfad83739af3d0706570420cb6dc9630c4 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabi...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
