2011/3/11 Gunnar Wolf <[email protected]>: > Jonas Häggqvist dijo [Thu, Mar 10, 2011 at 08:54:26AM +0100]: >> Package: cherokee >> Version: 1.0.20-1 >> Severity: important >> [cut] >> Currently the package is uninstallable unless you have cherokee-admin >> installed. [cut] > diff --git a/debian/cherokee.preinst b/debian/cherokee.preinst > index 8d84e98..06247c6 100644 > --- a/debian/cherokee.preinst > +++ b/debian/cherokee.preinst > @@ -23,7 +23,13 @@ case "$action" in > if [ -e "$CONFFILE" ] && [ "$md5sum" != "$old_md5sum" ]; then > # Finally, starting 0.99.31, there is just a generic migrator > # able to do it all. We will deprecate older migrators soon. > - /usr/share/cherokee/admin/upgrade_config.py $CONFFILE > + if [ -x /usr/share/cherokee/admin/upgrade_config.py ]; then > + /usr/share/cherokee/admin/upgrade_config.py $CONFFILE > + else > + echo "To auto-upgrade the Cherokee configuration, you need" > + echo "to install the cherokee-admin package." > + echo "Continuing with the existing installation." > + fi > fi > ;; > abort-upgrade)
That looks perfectly fine and dandy, and probably even better than outright requiring cherokee-admin. > Note that even if this issue is present in Squeeze's version > (1.0.20-1), I do not think pushing it to stable is necessary. And > given most users will have cherokee-admin installed as well, I am > waiting for the next upstream release to upload the bug. I have no opinion on this as I have no idea what the criteria are for such. Since installing recommends is the default configuration it will of course not affect most people as you say. Sounds reasonable. -- Jonas Häggqvist -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
