On 9/11/06, Dan Nicholson <[EMAIL PROTECTED]> wrote:
On 9/11/06, Peter Ennis <[EMAIL PROTECTED]> wrote:
>
> LFS 6.2
>
> 6.27 Bash
>
> I noticed this in the log:
>
> lfs:/mnt/lfs/lfs-logs$ grep "as root" 6.27*
> < execscript: the test suite should not be run as root
> < exec7.sub: the test suite should not be run as root
> lfs:/mnt/lfs/lfs-logs$
>
> Ch6 runs as root so there will not be full
> coverage of these tests.
> Is this a concern/bug/feature ?
Thanks for the notice. We should be able to drop to an unprivelaged
user like with perl and coreutils. Here's what DIY is doing:
http://www.diy-linux.org/x86-reference-build/chroot.html#c-bash
I took a look more at this. The issue is not full coverage of tests.
The tests are run regardless (DIY says the same). They are just
warnings that they shouldn't be run as root. I assume it's just being
safe that the tests don't do something bad as root.
However, I did look into switching users to get rid of the warnings.
The problem is that we don't have an su program at that point. Not
until shadow is installed at the end of the chapter. During the
coreutils build we get away with dropping privileges by using the su
built during coreutils (that we don't install). DIY gets around this
by installing the su from the temp system in /tools which normally is
only installed if you're the root user since it needs to be suid. We
could do something similar. I'm not sure it's worth it.
I have a diff for the bash page, but it's worthless because there's no
su to use.
--
Dan
--
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page
