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.
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.