Re: Freeze exception for Ampache
On Sun, 2010-09-19 at 22:59 -0500, Charlie Smotherman wrote:
> On Sun, Sep 19, 2010 at 10:44 AM, Adam D. Barratt
> <adam@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
Reply to: