----- Original Message ----- 
From: "Bruce Dubbs" <bruce.du...@gmail.com>
To: "LFS Book Maintenance List" <lfs-book@linuxfromscratch.org>
Sent: Saturday, November 21, 2009 3:36 PM
Subject: Re: r9112 - in trunk/BOOK: chapter01 chapter06


> matt...@linuxfromscratch.org wrote:
>
> > --- trunk/BOOK/chapter06/linux-headers.xml 2009-11-21 11:10:56 UTC (rev
9111)
> > +++ trunk/BOOK/chapter06/linux-headers.xml 2009-11-21 11:11:39 UTC (rev
9112)
> > @@ -58,7 +58,8 @@
> >
> >  <screen><userinput remap="install">make headers_check
> >  make INSTALL_HDR_PATH=dest headers_install
> > -cp -rv dest/include/* /usr/include</userinput></screen>
> > +cp -rv dest/include/* /usr/include
> > +find /usr/include -name .install -o -name
..install.cmd -delete</userinput></screen>
>
> Two things:  First, I would change the order of the last two statements
> above so the files never get into /usr:
>
> find dest/include -name .install -o -name ..install.cmd -delete
> cp -rv dest/include/* /usr/include
>
> Second, there should be a couple of explanatory words about why we are
> doing this.
>
>    -- Bruce
> -- 
find usage is wrong here

find is unintuitive, the following command find files with both names
find dest/include -name .install -o -name ..install.cmd

but this other command find files only with the second name
find dest/include -name .install -o -name ..install.cmd -ls

Once you add an action, only the second name is used unless you wrap names
between \( \) like this
find dest/include \( -name .install -o -name ..install.cmd \) -delete

Gilles

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

Reply via email to