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@xxxxxxxxxxxxx> >> --- >> 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 >
