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

Yann E. MORIN yann.morin.1998@anciens.enib.fr
Tue Dec 21 17:20:00 GMT 2010


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

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

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.  |
'------------------------------^-------^------------------^--------------------'

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



More information about the crossgcc mailing list