Messages in this thread | | | Date | Tue, 8 Feb 2022 01:50:09 +0500 | Subject | Re: [PATCH] selftests: futex: set DEFAULT_INSTALL_HDR_PATH | From | Muhammad Usama Anjum <> |
| |
On 2/7/22 11:40 PM, Shuah Khan wrote: > On 1/19/22 3:33 AM, Muhammad Usama Anjum wrote: >> If only futex selftest is compiled, uapi header files are copied to the >> selftests/futex/functional directory. This copy isn't needed. Set the >> DEFAULT_INSTALL_HDR_PATH variable to 1 to use the default header install >> path only. This removes extra copy of header file. >> >> Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com> >> --- >> tools/testing/selftests/futex/functional/Makefile | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/tools/testing/selftests/futex/functional/Makefile >> b/tools/testing/selftests/futex/functional/Makefile >> index 5cc38de9d8ea..9a8c3700d773 100644 >> --- a/tools/testing/selftests/futex/functional/Makefile >> +++ b/tools/testing/selftests/futex/functional/Makefile >> @@ -24,6 +24,7 @@ TEST_PROGS := run.sh >> top_srcdir = ../../../../.. >> KSFT_KHDR_INSTALL := 1 >> +DEFAULT_INSTALL_HDR_PATH := 1 >> include ../../lib.mk >> $(TEST_GEN_FILES): $(HEADERS) >> > > This will be a good cleanup. However, please make there is no dependencies > in the futex test sub-dirs on being able to find the headers under futex > I've tried the following variations. The build is fine.
cd tools/testing/selftests/futex/functional make
cd tools/testing/selftests/futex/ make
make -C tools/testing/selftests/
> I vaguely remember leaving this in place for reason. > This must have been fixed in previous changes made to the build system.
> thanks, > -- Shuah >
| |