Subject: Re: pkg/32835: rc.d/estd doesn't have the 'x' bit set
To: Steven M. Bellovin <smb@cs.columbia.edu>
From: Johnny Lam <jlam@pkgsrc.org>
List: pkgsrc-bugs
Date: 02/14/2006 16:39:18
Steven M. Bellovin wrote:
>
> It has that near the bottom, but a 'make install' shows
>
> ===> Installing for estd-0.4nb1
> install -d -o root -g wheel -m 0755 /usr/pkg/sbin
> install -s -o root -g wheel -m 0755 estd /usr/pkg/sbin/estd
> install -d -o root -g wheel -m 0755 /usr/pkg/man/man1
> install -o root -g wheel -m 0644 estd.1 /usr/pkg/man/man1/estd.1
> install -o root -g wheel -m 0644 estd.sh /usr/pkg/share/examples/rc.d/estd
> ===> [Automatic manual page handling]
> ===> Registering installation for estd-0.4nb1
>
> and
>
> # ls -l /usr/pkg/share/examples/rc.d/estd
> -rw-r--r-- 1 root wheel 322 Feb 14 16:21 /usr/pkg/share/examples/rc.d/estd
The sysutils/estd/patches/patch-aa file is incorrect. Firstly, it
should be using the BSD_INSTALL_* macros, otherwise, this package will
not install correctly when UNPRIVILEGED=yes. Secondly, it should be
installing the rc.d script using BSD_INSTALL_SCRIPT instead of
hardcoding to mode 0644.
Cheers,
-- Johnny Lam <jlam@pkgsrc.org>