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