Two small patches for CTDB

[Mathieu PARENT]

On Feb 12, 2008 10:26 PM, ronnie sahlberg <ronniesahlberg at gmail.com> wrote:
> Hi Mathieu,
>
> I have applied your patches with one exception and pushed to my git
> tree. It will eventually show up in tridges official tree.
>
Thanks

> I did not apply the part where you add #!/bin/sh to
> /etc/ctdb/functions since this file is not a standalone shellscript,
> instead it is just
> a file that is sourced from other scripts, and thus shouldnt have #!
>
So, maybe it should install as 644 (it is called as ". /etc/ctdb/functions").
I have made this patch because lintian (a debian package checker)
complains about
files which are executable and neither ELF neither #! prefixed.


>
> Thanks for using and contributing to CTDB.
You're welcome
> Please tell us later how it works for you on your platform.
I will, we are currently testing it.

>
>
> regards
> ronnie s
>
>
> On Wed, Feb 13, 2008 at 7:30 AM, Mathieu PARENT <math.parent at gmail.com> wrote:
> > Hi Tridge, hi everybody,
> >
> >  I am currently packaging CTDB for debian, it works good.
> >
> >  I have some patches that I think can go on upstream git :
> >  - a patch to make ''nice service'' works on systems without
> >  /sbin/service (this is bug 5258)
> >  - a patch to sanitize /etc/ctdb : config/events.d/README should not be
> >  executable, and config/functions is a shell script (these are bugs
> >  5216 and 5217)
> >
> >  Thanks
> >
> >  Mathieu Parent
> >
> >  PS : please reply to or cc me, i'm not on samba-technical
> >
> >  PPS : others debian specific patches can be seen here :
> >  http://svn.debian.org/wsvn/pkg-samba/trunk/ctdb/debian/patches/?rev=0&sc=0
> >
>