[Buildroot] [PATCH v2 3/3] rpi-userland: Fix vcfiled startup

Benoît Thébaudeau benoit.thebaudeau at advansee.com
Wed Aug 6 19:56:27 UTC 2014


Dear Yann E. MORIN,

On Wed, Aug 6, 2014 at 7:40 PM, Benoît Thébaudeau
<benoit.thebaudeau at advansee.com> wrote:
> Dear Yann E. MORIN,
>
> On Wednesday, August 6, 2014 7:33:51 PM, Yann E. MORIN wrote:
>> Benoit, All,
>>
>> On 2014-08-06 15:39 +0200, Benoît Thébaudeau spake thusly:

[...]

>> Otherwise, see below...
>>
>> > Signed-off-by: Benoît Thébaudeau <benoit.thebaudeau at advansee.com>
>> [--SNIP--]
>> > diff --git a/package/rpi-userland/S94vcfiled
>> > b/package/rpi-userland/S94vcfiled
>> > new file mode 100755
>> > index 0000000..25a0fcd
>> > --- /dev/null
>> > +++ b/package/rpi-userland/S94vcfiled
>> > @@ -0,0 +1,47 @@
>> > +#!/bin/sh
>> > +
>> > +NAME=vcfiled
>> > +DESC="VideoCore file server daemon $NAME"
>> > +DAEMON="/usr/sbin/$NAME"
>> > +DAEMON_ARGS=""
>> > +CFGFILE="/etc/default/$NAME"
>> > +PIDFILE="/var/run/$NAME/$NAME"
>>
>> Are you sure about the path to the PID file?
>> What about:
>>     PIDFILE="/var/run/$NAME.pid"
>
> Yes, sure, this is the path used by vcfiled, which creates its PID file itself
> rather than relying on start-stop-daemon -m for that.

I have checked the code, and this path can be overridden by defining
VCFILED_LOCKFILE, so I can set it to '/var/run/vcfiled.pid' if you
really prefer. It's up to you. You tell me.

[...]

Regards,
Benoît



More information about the buildroot mailing list