• Axel Rasmussen's avatar
    selftests: fix header dependency for pid_namespace selftests · 52035628
    Axel Rasmussen authored
    The way the test target was defined before, when building with clang we
    get a command line like this:
    
    clang -Wall -Werror -g -I../../../../usr/include/ \
    	regression_enomem.c ../pidfd/pidfd.h  -o regression_enomem
    
    This yields an error, because clang thinks we want to produce both a *.o
    file, as well as a precompiled header:
    
    clang: error: cannot specify -o when generating multiple output files
    
    gcc, for whatever reason, doesn't exhibit the same behavior which I
    suspect is why the problem wasn't noticed before.
    
    This can be fixed simply by using the LOCAL_HDRS infrastructure the
    selftests lib.mk provides. It does the right think and marks the target
    as depending on the header (so if the header changes, we rebuild), but
    it filters the header out of the compiler command line, so we don't get
    the error described above.
    Signed-off-by: default avatarAxel Rasmussen <axelrasmussen@google.com>
    Reviewed-by: default avatarChristian Brauner <brauner@kernel.org>
    Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
    52035628
Makefile 170 Bytes