Comment 8 for bug 676234

Revision history for this message
Martin Pitt (pitti) wrote :

If the debconf result is supposed to be a list, the original code was correct. If it's only a single value, then the "for" loop doesn't make sense, but the original code would still be correct. But the uploaded SRU change just doesn't make sense this way:

         for webserver in $webservers; do
             webserver=${webserver%,}
- if [ "$webserver" = "lighttpd" ] ; then
+ if [ "$webservers" = "lighttpd" ] ; then

Also, the source package was built in a strange way (not using the Ubuntu dpkg?), it's missing the Launchpad-Bugs-Fixed: line in the source.changes.