On 11/11/06, Jeremy Huntwork <[EMAIL PROTECTED]> wrote:
Jeremy Huntwork wrote:
> If you don't like the above approach, then I suggest we put in an explanation 
as
> to why we install to a temporary directory first and add the '-v' flag to the 
cp
> command.

Actually, after looking through the Linux Makefile a bit, I think our
commands for chapter 5 linux-headers can be simplified to the following:

patch -Np1 -i ../linux-2.6.18.1-unifdef-1.patch
make mrproper
make headers_check
cp -Rv usr/include/* /tools/include

This is because the target 'headers_check' includes 'headers_install' as
a dependency, and therefore runs that first:

PHONY += headers_check
headers_check: headers_install
         $(Q)$(MAKE) -rR -f $(srctree)/scripts/Makefile.headersinst
obj=include HDRCHECK=1

And since we don't specify the variable INSTALL_HDR_PATH, it uses the
default of "INSTALL_HDR_PATH=$(objtree)/usr", which means that the 'rm
-rf' won't affect us at all. Also, diff shows no difference between the
files in /tools/tmp/include built via the current method and those in
usr/include.

I like this better. Cleaner, shorter, and it uses the defaults from
the package. And doing the check doesn't hurt ever. Jeremy, what tools
are needed in the check? I just don't want to add to the host system
requirements for the Ch. 5 headers.

--
Dan
--
http://linuxfromscratch.org/mailman/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page

Reply via email to