Skip to content

Commit 3308b28

Browse files
arndbMichal Marek
authored andcommitted
Kbuild: always prefix objtree in LINUXINCLUDE
When $(LINUXINCLUDE) is added to the cflags of a target that normall doesn't have it (e.g. HOSTCFLAGS), each entry in the list is expanded so that we search both $(objtree) and $(srctree), which is a bit silly, as we already know which of the two we want for each entry in LINUXINCLUDE. Also, a follow-up patch changes the behavior so we only look in $(srctree) for manually added include path, and that breaks finding the generated headers. This adds an explicit $(objtree) for each tree that we want to look for generated files. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Michal Marek <mmarek@suse.com>
1 parent dc33db7 commit 3308b28

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

Makefile

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -375,19 +375,19 @@ CFLAGS_KCOV := $(call cc-option,-fsanitize-coverage=trace-pc,)
375375
# Use USERINCLUDE when you must reference the UAPI directories only.
376376
USERINCLUDE := \
377377
-I$(srctree)/arch/$(hdr-arch)/include/uapi \
378-
-Iarch/$(hdr-arch)/include/generated/uapi \
378+
-I$(objtree)/arch/$(hdr-arch)/include/generated/uapi \
379379
-I$(srctree)/include/uapi \
380-
-Iinclude/generated/uapi \
380+
-I$(objtree)/include/generated/uapi \
381381
-include $(srctree)/include/linux/kconfig.h
382382

383383
# Use LINUXINCLUDE when you must reference the include/ directory.
384384
# Needed to be compatible with the O= option
385385
LINUXINCLUDE := \
386386
-I$(srctree)/arch/$(hdr-arch)/include \
387-
-Iarch/$(hdr-arch)/include/generated/uapi \
388-
-Iarch/$(hdr-arch)/include/generated \
387+
-I$(objtree)/arch/$(hdr-arch)/include/generated/uapi \
388+
-I$(objtree)/arch/$(hdr-arch)/include/generated \
389389
$(if $(KBUILD_SRC), -I$(srctree)/include) \
390-
-Iinclude
390+
-I$(objtree)/include
391391

392392
LINUXINCLUDE += $(filter-out $(LINUXINCLUDE),$(USERINCLUDE))
393393

0 commit comments

Comments
 (0)