Hi Shuah, On 02/06/2017 21:31, Shuah Khan wrote: > Hi Mickaël, > > On 05/26/2017 12:43 PM, Mickaël Salaün wrote: >> Rebuild the seccomp tests when kselftest_harness.h is updated. >> >> Signed-off-by: Mickaël Salaün <m...@digikod.net> >> Acked-by: Kees Cook <keesc...@chromium.org> >> Cc: Andy Lutomirski <l...@amacapital.net> >> Cc: Shuah Khan <sh...@kernel.org> >> Cc: Will Drewry <w...@chromium.org> >> --- >> tools/testing/selftests/seccomp/Makefile | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tools/testing/selftests/seccomp/Makefile >> b/tools/testing/selftests/seccomp/Makefile >> index 5fa6fd2246b1..aeb0c805f3ca 100644 >> --- a/tools/testing/selftests/seccomp/Makefile >> +++ b/tools/testing/selftests/seccomp/Makefile >> @@ -4,3 +4,5 @@ LDFLAGS += -lpthread >> >> include ../lib.mk >> >> +$(TEST_GEN_PROGS): seccomp_bpf.c ../kselftest_harness.h >> + $(CC) $(CFLAGS) $(LDFLAGS) $< -o $@ >> > > This change breaks seccomp build: > > make -C tools/testing/selftests/seccomp/ > make: Entering directory > '/mnt/data/lkml/linux_4.12/tools/testing/selftests/seccomp' > make: *** No rule to make target '../kselftest_harness.h', needed by > '/mnt/data/lkml/linux_4.12/tools/testing/selftests/seccomp/seccomp_bpf'. > Stop. > make: Leaving directory > '/mnt/data/lkml/linux_4.12/tools/testing/selftests/seccomp' > shuah@mazurka:/mnt/data/lkml/linux_4.12$ cd tools/testing/selftests/seccomp/ > shuah@mazurka:/mnt/data/lkml/linux_4.12/tools/testing/selftests/seccomp$ make > make: *** No rule to make target '../kselftest_harness.h', needed by > '/mnt/data/lkml/linux_4.12/tools/testing/selftests/seccomp/seccomp_bpf'. > Stop. > > > Did you happen to try building with this change?
Yes I did, and it builds fine for me (with the same command and patches). There is only one warning from GCC with ../kselftest_harness.h:368 because of the trick from OPTIONAL_HANDLER(), which is intentional (and not modified by my patches). I don't get why your "make" said that there is "No rule to make target" ../kselftest_harness.h . This file exists (patch 1/7) so there is no need to create it… gen_kselftest_tar.sh works fine too. Thanks, Mickaël
signature.asc
Description: OpenPGP digital signature
