[Pkg-utopia-maintainers] [Fwd: hal regression in svn head]

[Michael Biebl]

Michael Biebl schrieb:
> Michael Biebl schrieb:
>> Hi Michael,
>>
>> in hal Debian svn you recently committed some cleanups of the
>> postinst. In particular you dropped the "|| true" from the chmod'ing
>> of /var/run. This causes the package to fail to install from scratch,
>> since the directory does not exist yet.
>>
>> I fixed that in Ubuntu, you should do the same in Debian IMHO.
>>
>> Thanks for considering,
> 
> My take on this issue actually is this: imho packages should list the
> /var/run/$foo dir in the deb package, the chown should happen in
> postinst, and the mkdir/chown/chmod should be removed from all init scripts.
> 
> The current practice imho is only a hack to support tmpfs on /var/run.
> If you want to have the tmpfs feature, initscripts should backup
> /var/run and restore it for you. No need to duplicate this functionality
>  in all 20 init scripts (or so). Besides this only wastes cpu cycles on
> every boot.
> 
> And what if you don't have a init script? I had this problem once with
> an pam modules which expected an existing /var/run/bar directory.
> 

FWIW, I prefer the attached patch.
I also set the home dir to /var/run/hal, /home/haldaemon looks a bit odd.

Comments?

Cheers,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal_run_dir.patch
Type: text/x-patch
Size: 628 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-utopia-maintainers/attachments/20071212/af21b987/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-utopia-maintainers/attachments/20071212/af21b987/attachment.pgp