tags 617640 + patch pending
end
Jonas Häggqvist dijo [Thu, Mar 10, 2011 at 08:54:26AM +0100]:
> Package: cherokee
> Version: 1.0.20-1
> Severity: important
>
> As mentioned in the subject, preinst fails at line 26 because of a
> missing file /usr/share/cherokee/admin/upgrade_config.py, which is
> part of cherokee-admin. Hence cherokee-admin needs to be a dependency,
> rather than Recommends (or the prinst changed).
>
> Currently the package is uninstallable unless you have cherokee-admin
> installed.
You are right - I am batching this for the next upstream release in
the unstable branch. As cherokee-admin is already recommended, it will
most often be installed. Would you agree this will fix the situation?
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)
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.
Greetings,
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
