Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm-tool: remove 'vdsm/tool/*.pyc' files on uninstallation ......................................................................
vdsm-tool: remove 'vdsm/tool/*.pyc' files on uninstallation It's found that 'vdsm/tool/*.pyc' files are not removed on uninstallation. It could cause a break after installing a different version vdsm. That's because vdsm-tool tries to load all python modules under vdsm.tool. If the requirements of these old modules can't be sytisfied by the new installed packages, vdsm-tool will abort. This problem is caused by a wrong target type for python files in lib/vdsm/tool. They're not compiled during 'make install' and therefore complied python files are not tracked by rpm. This patch also fixes a wrong target name in CLEANFILES. Change-Id: I5fd67f3fbcbf4c8f72180ba0129b0bc09b20def0 Signed-off-by: Mark Wu <[email protected]> Reviewed-on: http://gerrit.ovirt.org/16761 Reviewed-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/tool/Makefile.am 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Mark Wu: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16761 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5fd67f3fbcbf4c8f72180ba0129b0bc09b20def0 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Mark Wu <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
