Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-05-03 Thread Andriy Beregovenko
Hi Jonas, Alessio, I complete fixes. Please review changes and upload package. On Tue, Apr 24, 2012 at 09:23:56AM -0400, Jonas Smedegaard wrote: > On 12-04-24 at 10:17am, Andriy Beregovenko wrote: > > Hi Jonas, > > > > On Tue, Apr 24, 2012 at 02:40:22AM -0400, Jonas Smedegaard wrote: > > > On 12

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-30 Thread Andriy Beregovenko
Hi, Also thank you for your job and pointing me to the right way. :) On Wed, Apr 25, 2012 at 12:43:20AM +0200, Alessio Treglia wrote: > On Tue, Apr 24, 2012 at 3:23 PM, Jonas Smedegaard wrote: > > Either the files are *only* provided as examples or they are handled by > > the packaging scripts.

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-30 Thread Andriy Beregovenko
Hi Alessio, On Wed, Apr 25, 2012 at 12:43:20AM +0200, Alessio Treglia wrote: > On Tue, Apr 24, 2012 at 3:23 PM, Jonas Smedegaard wrote: > > Either the files are *only* provided as examples or they are handled by > > the packaging scripts. > > > > If only example files, then they are not owned by

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-24 Thread Alessio Treglia
On Tue, Apr 24, 2012 at 3:23 PM, Jonas Smedegaard wrote: > Either the files are *only* provided as examples or they are handled by > the packaging scripts. > > If only example files, then they are not owned by the package and should > not be handled by the package. > > If handled by the package -

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-24 Thread Jonas Smedegaard
On 12-04-24 at 10:17am, Andriy Beregovenko wrote: > Hi Jonas, > > On Tue, Apr 24, 2012 at 02:40:22AM -0400, Jonas Smedegaard wrote: > > On 12-04-24 at 08:53am, Andriy Beregovenko wrote: > > > Hi Alessio, > > > > > > On Tue, Apr 24, 2012 at 02:06:43AM +0200, Alessio Treglia wrote: > > > > This doe

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-24 Thread Andriy Beregovenko
Hi Jonas, On Tue, Apr 24, 2012 at 02:40:22AM -0400, Jonas Smedegaard wrote: > On 12-04-24 at 08:53am, Andriy Beregovenko wrote: > > Hi Alessio, > > > > On Tue, Apr 24, 2012 at 02:06:43AM +0200, Alessio Treglia wrote: > > > This doesn't look sane to me: > > > > > > On Sat, Apr 14, 2012 at 9:45 PM

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-23 Thread Jonas Smedegaard
On 12-04-24 at 08:53am, Andriy Beregovenko wrote: > Hi Alessio, > > On Tue, Apr 24, 2012 at 02:06:43AM +0200, Alessio Treglia wrote: > > This doesn't look sane to me: > > > > On Sat, Apr 14, 2012 at 9:45 PM, wrote: > > > +do_remove() { > > > +       for app in $APPLICATIONS; do > > > +        

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-23 Thread Andriy Beregovenko
Hi Alessio, On Tue, Apr 24, 2012 at 02:06:43AM +0200, Alessio Treglia wrote: > This doesn't look sane to me: > > On Sat, Apr 14, 2012 at 9:45 PM, wrote: > > +do_remove() { > > +       for app in $APPLICATIONS; do > > +               if [ -f "/etc/crtmpserver/applications/$app.lua" ]; then > > +

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-23 Thread Alessio Treglia
On Sat, Apr 14, 2012 at 9:45 PM, wrote: > +compare() { > +       [ $# != 2 ] && exit 1 > +       A=$(md5sum $1 | cut -d' ' -f1) > +       B=$(md5sum $2 | cut -d' ' -f1) > +       [ "$A" = "$B" ] && return 0 || return 1 > +} > + > +do_remove() { > +       for app in $APPLICATIONS; do > +          

Re: [SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

2012-04-23 Thread Alessio Treglia
This doesn't look sane to me: On Sat, Apr 14, 2012 at 9:45 PM, wrote: > +do_remove() { > +       for app in $APPLICATIONS; do > +               if [ -f "/etc/crtmpserver/applications/$app.lua" ]; then > +                       compare "/etc/crtmpserver/applications/$app.lua" \ > +