On 4/23/20 9:25 PM, Randy Dunlap wrote:
Hi,
On 4/14/20 2:22 PM, Shuah Khan wrote:
-CFLAGS += -O2 -g -std=gnu99 -Wall -I../../../../usr/include/ $(MOUNT_CFLAGS)
-LDLIBS += $(MOUNT_LDLIBS)
+CFLAGS += -O2 -g -std=gnu99 -Wall -I../../../../usr/include/ $(VAR_CFLAGS)
+LDLIBS += $(VAR_LDLIBS)
(1) Can that series of ../../../.. be replaced by $(objtree)?
If so, that would be much cleaner IMO.
Agreed. It can be done in a separate patch. We have other tests that
do similar hard coded header paths.
(2) I can't find anything that checks that ../../../../usr/include exists
(or has been installed via 'make headers_install'). Or anything that
requires that CONFIG_HEADERS_INSTALL be set/enabled. Well, other than
a Makefile error, but that's not a nice way to find out.
At the moment no. When this Makefile is fixed, that is another check
to add. It is addressed by headers install during selftest make.
Headers are installed as part of selftests make since selftests are
often dependent on recent headers and headers install is a necessary
step.
You are right. There are several tests that need cleanup for such
hard coded values and this dependency check script in this patch
currently does libs check. My plan is to extend this to check for
headers installed or not and flag headers as missing dependency.
I can do that in my next version of this dependency check script.
thanks,
-- Shuah
