[Clfs-commits] [Cross-LFS]Cross-LFS Book branch, master, updated. 521544846d331c0a4fa58e1e7a0c395716929441

[Chris Staub]

On 01/03/2010 10:48 AM, git wrote:
> diff --git a/BOOK/chroot/common/changingowner.xml b/BOOK/chroot/common/changingowner.xml
> index be0fbd5..a375f0e 100644
> --- a/BOOK/chroot/common/changingowner.xml
> +++ b/BOOK/chroot/common/changingowner.xml
> @@ -10,6 +10,13 @@
>
>     <title>Changing Ownership</title>
>
> +<note>
> +<para>This step is not optional as some of the binaries in
> +<filename class="directory">/tools</filename>  are set u+s.
> +      leaving the permissions as is could cause some commands,
> +      mount in particular, to fail later.</para>
> +</note>
> +

This doesn't make sense. First, chown removes the suid bit, so anything 
in /tools that needs to be suid would have to be manually chmodded after 
the chown. Second, it makes no difference whether mount (or anything 
else) is suid root if you're supposed to be root inside chroot anyway.