DevPak is unable to choose between library versions

[techtonik]

DevPak installer plugin selects packs from the list using only library name, without taking into account version info. If there are several versions of the same library then only the first one can be installed.

The original bug report at BerliOS doesn't allow to attach patches, so I post it here.
Patch fixes DevPak selection logic by comparing version and revision fields also.

P.S. There are some more fixes (mostly cosmetic) pending for this plugin.

[attachment deleted by admin]

[techtonik]

Fix is is posted in revision 3 in this branch - https://code.launchpad.net/~techtonik/codeblocks/devpak-plugin.

The problem appears as "file size mismatch" when installing another version of library. Also reported here.

[MortenMacFly]

https://code.launchpad.net/~techtonik/codeblocks/devpak-plugin.

We are not using bazaar. You make it really hard to obtain the "patch" by using another version control system. Mind posting a "true" patch at BerliOS for inspection?

[techtonik]

The "true" patch is the same as attached in my first post. I forgot to mentioned that I finally found a way to submit patch in BerliOS and added a link to it in original bugreport linked, again, in the first post.

More colorful example at launchpad:
http://bazaar.launchpad.net/~techtonik/codeblocks/devpak-plugin/revision/3

Launchpad is not just about another version control system - it provides a convenient way to store a sequence of patches without waiting for them to be approved one by one by branching code of existing projects. This helps to avoid mixing bug fixing work with featurecreeped kitchensinks and not to lose something in the process.

BTW, where should I post further enhancements to DevPak plugin? BerliOS tracker, ahem, suxx.

[techtonik]

Fixed in Nightly Build from 19 July 2008 - http://forums.codeblocks.org/index.php/topic,8821.0.html

Fixed version for 8.02 C::B release is attached.

[attachment deleted by admin]