[LTP] [PATCH] configure.ac: Fix temperory directory ./confXXXXXX not cleanup

Petr Vorel pvorel@suse.cz
Mon Oct 14 11:18:23 CEST 2024


Hi all,

Reviewed-by: Petr Vorel <pvorel@suse.cz>

nit: typos in subject: temperory => temporary

> configure
>  ->config.status
>   -> create temperory directory ./confXXXXXX
>   -> cd include/lapi/syscalls
>   -> ./regen.sh
>   -> cleanup ./confXXXXXX, registered in trap

> Where config.status changes the working directory, ./confXXXXXX is not
> removed as a result.

> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

> diff --git a/configure.ac b/configure.ac
> index b4ab81e265..d327974efa 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -386,7 +386,7 @@ else
>      AC_SUBST([WITH_REALTIME_TESTSUITE],["no"])
>  fi

> -AC_CONFIG_COMMANDS([syscalls.h], [cd ${ac_top_srcdir}/include/lapi/syscalls; ./regen.sh])
> +AC_CONFIG_COMMANDS([syscalls.h], [cd ${ac_top_srcdir}/include/lapi/syscalls; ./regen.sh; cd - >/dev/null])


I was thinking whether use "cd ${ac_top_builddir}", but because cd happens
outside of regen.sh, "cd -" will always work and it's probably the best.

Also, I suppose this happen quite long time ago:

Fixes: a07008fbec ("configure.ac: Generate linux_syscall_headers.h")

I noticed the problem later on but never invested time to fix it. Li Zhijian,
thanks for taking care.

Kind regards,
Petr

>  # custom functions
>  # NOTE: don't create custom functions for simple checks, put them into this file


More information about the ltp mailing list