[LTP] [PATCH V2 3/5] syscalls: semctl: Add new test to verify the _time_high fields

Viresh Kumar viresh.kumar@linaro.org
Tue Jun 16 12:09:38 CEST 2020


On 16-06-20, 11:52, Cyril Hrubis wrote:
> Ah, right, the rule that rebuilds the libraries does not work correctly
> for more than one library because I do pass "$^" there which is the list
> of directories and not a single directory.
> 
> As a quick hack this should fix it:
> 
> diff --git a/include/mk/testcases.mk b/include/mk/testcases.mk
> index 684655fbf..7e0ec13f9 100644
> --- a/include/mk/testcases.mk
> +++ b/include/mk/testcases.mk
> @@ -49,7 +49,9 @@ LTPLIBS_FILES = $(addsuffix .a, $(addprefix $(abs_top_builddir)/libs/, $(foreach
>  MAKE_DEPS += $(LTPLIBS_FILES)
>  
>  $(LTPLIBS_FILES): $(LTPLIBS_DIRS)
> -       $(MAKE) -C "$^" -f "$(subst $(abs_top_builddir),$(abs_top_srcdir),$^)/Makefile" all
> +       for dir in $^; do \
> +               $(MAKE) -C "$$dir" -f "$(subst $(abs_top_builddir),$(abs_top_srcdir),$$dir)/Makefile" all; \
> +       done
>  
>  LDFLAGS += $(addprefix -L$(top_builddir)/libs/lib, $(LTPLIBS))
> 
> 
> This does not parallelize the build but I guess that it would be fine since
> when we rebuild LTP from top directory the libraries are build first anyways
> and this is no-op.

Tested-by: Viresh Kumar <viresh.kumar@linaro.org>

Please merge it, I am sending my patches again based on this.

-- 
viresh


More information about the ltp mailing list