On Sun, 2010-09-19 at 22:59 -0500, Charlie Smotherman wrote:
> On Sun, Sep 19, 2010 at 10:44 AM, Adam D. Barratt
> <a...@adam-barratt.org.uk> wrote:
> > On Fri, 2010-09-17 at 14:33 -0500, Charlie Smotherman wrote:
> >> On Sat, 2010-09-11 at 20:30 +0100, Adam D. Barratt wrote:
> >> I have applied the changes you have suggested. Thx :)
> >> I have tried to keep changes to a minimum but after release I plan to
> >> use a state engine for the debconf questions so if the answers to the
> >> questions is no the installation will exit gracefully.
> >
> > The postrm fix appears to be missing the final "fi"; the indentation's
> > now also confusing, but that's possibly my fault for providing a diff
> > which didn't re-indent to make it clearer where the changes were.
> >
> Done, Inserted missing "fi'.
Thanks, although it's not quite in the right place; as I said, possibly
my fault for not making it clear in the first place. The if block
surrounds the entire webserver restart section, so the fi should be
(except with proper re-intending):
diff -Nru -w ampache-3.5.4/debian/ampache.postrm
ampache-3.5.4/debian/ampache.postrm
--- ampache-3.5.4/debian/ampache.postrm 2010-08-02 05:06:51.000000000 +0100
+++ ampache-3.5.4/debian/ampache.postrm 2010-09-11 18:15:29.000000000 +0100
@@ -53,6 +54,7 @@
fi
done
fi
+fi
if [ "$1" = "purge" ]; then
#This is needed to remove user added content.
there.
Regards,
Adam
--
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive:
http://lists.debian.org/1285103960.20782.1480.ca...@kaa.jungle.aubergine.my-net-space.net
