[PATCH] kernel/linux: Cleanup all unneeded files from 'headers_check' and 'headers_install'

[Bryan Hundven]

On Tue, Dec 21, 2010 at 9:20 AM, Yann E. MORIN
<yann.morin.1998@anciens.enib.fr> wrote:
> Bryan, All,
>
> On Tuesday 21 December 2010 01:55:25 Bryan Hundven wrote:
>> diff -r 13e4f7126cf7 -r dceb4bf2e188 scripts/build/kernel/linux.sh
>> --- a/scripts/build/kernel/linux.sh   Tue Dec 14 11:49:18 2010 -0800
>> +++ b/scripts/build/kernel/linux.sh   Mon Dec 20 16:54:30 2010 -0800
>> @@ -112,7 +112,7 @@
>>               INSTALL_HDR_PATH="${CT_SYSROOT_DIR}/usr"       \
>>               ${V_OPT}                                       \
>>               headers_check
>> -        find "${CT_SYSROOT_DIR}" -type f -name '.check*' -exec rm {} \;
>> +        find "${CT_SYSROOT_DIR}" -type f -name '.*install*' -o -name '.*check*' | xargs rm
>
> Why not something like:
>  find "${CT_SYSROOT_DIR}" -type f                  \
>       \( -name '.*install*' -o -name '.*check*' \) \
>       -exec rm {} \;
>
> Then, I can see why the '.check*' did not match. There are only four types of
> files being installed:
>  .check
>  ..check.cmd
>  .install
>  ..install.cmd

Technically, only 3:

..check.cmd
.install
..install.cmd

> Can't we use the full, known names instead of the wildcards?

I don't see why not. I will push a new patch today.

> Regards,
> Yann E.MORIN.
>
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
>

-Bryan

--
For unsubscribe information see http://sourceware.org/lists.html#faq