On 15.02.2024 17:08, Roger Pau Monné wrote:
> On Thu, Feb 15, 2024 at 02:02:41PM +0100, Jan Beulich wrote:
>> --- a/xen/Rules.mk
>> +++ b/xen/Rules.mk
>> @@ -15,7 +15,11 @@ srcdir := $(srctree)/$(src)
>>  PHONY := __build
>>  __build:
>>  
>> --include $(objtree)/include/config/auto.conf
>> +ifneq ($(obj),tools)
>> +ifneq ($(obj),tools/kconfig)
>> +include $(objtree)/include/config/auto.conf
>> +endif
>> +endif
> 
> Trying to understand this, I assume it's to avoid an infinite
> dependency loop that generating include/config/auto.conf requires some
> tools that are build using xen/Rules.mk?

The file has dependencies only in xen/Makefile. This is about the
file simply not being there when initially building. Perhaps the
patch description helps that I've written in the meantime:

"Because of using "-include", failure to (re)build auto.conf (with
 auto.conf.cmd produced as a secondary target) won't stop make from
 continuing the build. Arrange for it being possible to drop the - from
 Rules.mk, requiring that the include be skipped for tools-only targets.
 Note that relying on the inclusion in those cases wouldn't be correct
 anyway, as it might be a stale file (yet to be rebuilt) which would be
 included, while during initial build, the file would be absent
 altogether."

Jan

Reply via email to