[Buildroot] [PATCH 1/2] package/linux-tools: perf: ditch redundant -f 'rm' flag

Yann E. MORIN yann.morin.1998 at free.fr
Tue Mar 28 19:37:33 UTC 2023


Luca, All,

On 2023-03-27 16:11 +0200, Luca Ceresoli spake thusly:
> The value of the RM variable in make is 'rm -f', thus the additional -f is
> redundant.

I've added a pointer to the documentation:
    https://www.gnu.org/software/make/manual/make.html#index-RM

> Signed-off-by: Luca Ceresoli <luca.ceresoli at bootlin.com>

Both applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  package/linux-tools/linux-tool-perf.mk.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/linux-tools/linux-tool-perf.mk.in b/package/linux-tools/linux-tool-perf.mk.in
> index dda63cccecb4..5431c78152c5 100644
> --- a/package/linux-tools/linux-tool-perf.mk.in
> +++ b/package/linux-tools/linux-tool-perf.mk.in
> @@ -166,8 +166,8 @@ endef
>  # target unless BR2_PACKAGE_LINUX_TOOLS_PERF_SCRIPTS is enabled
>  ifeq ($(BR2_PACKAGE_LINUX_TOOLS_PERF_SCRIPTS),)
>  define PERF_INSTALL_REMOVE_SCRIPTS
> -	$(RM) -rf $(TARGET_DIR)/usr/libexec/perf-core/scripts/
> -	$(RM) -rf $(TARGET_DIR)/usr/libexec/perf-core/tests/
> +	$(RM) -r $(TARGET_DIR)/usr/libexec/perf-core/scripts/
> +	$(RM) -r $(TARGET_DIR)/usr/libexec/perf-core/tests/
>  endef
>  endif
>  
> -- 
> 2.34.1
> 

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



More information about the buildroot mailing list