Review Request: Packagers: remove duplicate code, consistent naming, simplify some code
Wolfgang Rohdewald
wolfgang at rohdewald.de
Tue Feb 15 21:23:46 CET 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6486/
-----------------------------------------------------------
Review request for kde-windows.
Summary
-------
__installedDBPrefix existed identically in PackageBase and InternalPackageBase. Remove it from InternalPackageBase and rename to _installedDBPrefix
rename self.packager to self.packagerExe for consistency
simplify code in PackagerFactory (this code is actually unused but it looks like a plan for the future)
a few other small simplifications around packager code
Diffs
-----
/trunk/kdesupport/emerge/bin/Package/InternalPackageBase.py 1220833
/trunk/kdesupport/emerge/bin/Package/PackageBase.py 1220833
/trunk/kdesupport/emerge/bin/Packager/InnoSetupPackager.py 1220902
/trunk/kdesupport/emerge/bin/Packager/KDEWinPackager.py 1220905
/trunk/kdesupport/emerge/bin/Packager/PackagerFactory.py 1220833
/trunk/kdesupport/emerge/bin/Packager/SevenZipPackager.py 1220833
Diff: http://svn.reviewboard.kde.org/r/6486/diff
Testing
-------
Thanks,
Wolfgang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-windows/attachments/20110215/31b5b3f3/attachment-0001.htm
More information about the Kde-windows
mailing list