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-04-24 at

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 d...@jones.dk 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

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 d...@jones.dk wrote: Either the files are *only* provided as examples or they are handled by the packaging

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

2012-04-24 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, jet-gu...@users.alioth.debian.org wrote: +do_remove() { +       for app in $APPLICATIONS;

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-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 doesn't look sane to

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 d...@jones.dk 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

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, jet-gu...@users.alioth.debian.org wrote: +do_remove() { +       for app in $APPLICATIONS; do +               if [ -f /etc/crtmpserver/applications/$app.lua ]; then +                       compare

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, jet-gu...@users.alioth.debian.org 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;

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, jet-gu...@users.alioth.debian.org wrote: +do_remove() { +       for app in $APPLICATIONS; do +               if [ -f