Nir Soffer has posted comments on this change. Change subject: Remove __init__.py file from lib64 vdsm python directory ......................................................................
Patch Set 4: (2 comments) https://gerrit.ovirt.org/#/c/50115/4/vdsm.spec.in File vdsm.spec.in: Line 336: %description python Line 337: Shared libraries between the various VDSM packages. Line 338: Line 339: %post python Line 340: # Requires For: upgrade from 4.14 to 4.17 - this shell be removed Reauired Line 341: # HACK: Remove vdsm python lib __init__ file if installed on old location Line 342: # https://bugzilla.redhat.com/show_bug.cgi?id=1279167 Line 343: if [ "$1" -ge 2 ]; then Line 344: if [ -d "/usr/lib64/python2.7/site-packages/vdsm" ]; then Line 341: # HACK: Remove vdsm python lib __init__ file if installed on old location Line 342: # https://bugzilla.redhat.com/show_bug.cgi?id=1279167 Line 343: if [ "$1" -ge 2 ]; then Line 344: if [ -d "/usr/lib64/python2.7/site-packages/vdsm" ]; then Line 345: rm /usr/lib64/python2.7/site-packages/vdsm/__init__.py* This will fail if there is an empty directory, use "rm -f" Line 346: fi Line 347: fi Line 348: Line 349: %package tests -- 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: 4 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