Build race observed with git 2.34.1 - hook-list.h wasn't generated yet

Build race observed with git 2.34.1 - hook-list.h wasn't generated yet

[Alexander Kanavin]

Specifically:

 builtin/bugreport.c:7:10: fatal error: hook-list.h: No such file or directory
|     7 | #include "hook-list.h"
|       |          ^~~~~~~~~~~~~
| compilation terminated.

Please see here for the full log:
https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/4427/steps/14/logs/stdio

Seems like COMPUTE_HEADER_DEPENDENCIES isn't working quite right? I meanwhile have set it to
'no'.

Thanks,
Alexander

Re: Build race observed with git 2.34.1 - hook-list.h wasn't generated yet

[Victoria Dye]

Alexander Kanavin wrote:
> Specifically:
> 
>  builtin/bugreport.c:7:10: fatal error: hook-list.h: No such file or directory
> |     7 | #include "hook-list.h"
> |       |          ^~~~~~~~~~~~~
> | compilation terminated.
> 
> Please see here for the full log:
> https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/4427/steps/14/logs/stdio
> 
> Seems like COMPUTE_HEADER_DEPENDENCIES isn't working quite right? I meanwhile have set it to
> 'no'.
> 
> Thanks,
> Alexander
> 

I've encountered this as well, and it appears to be a dependency graph
issue. For me, compilation was fixed by a patch [1] that isn't in next or
master yet ("Needs review" - ab/make-dependency [2]). If this an important
fix to fast-track (I'll defer to more experienced contributors to determine
that), it might speed up review to separate that patch from the rest of the
series?

[1] https://lore.kernel.org/git/patch-v4-19.23-2710f8af6cd-20211117T101807Z-avarab@gmail.com/

[2] https://lore.kernel.org/git/xmqqbl1xqheu.fsf@gitster.g/

Re: Build race observed with git 2.34.1 - hook-list.h wasn't generated yet

[Ævar Arnfjörð Bjarmason]

On Tue, Dec 07 2021, Victoria Dye wrote:

> Alexander Kanavin wrote:
>> Specifically:
>> 
>>  builtin/bugreport.c:7:10: fatal error: hook-list.h: No such file or directory
>> |     7 | #include "hook-list.h"
>> |       |          ^~~~~~~~~~~~~
>> | compilation terminated.
>> 
>> Please see here for the full log:
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/59/builds/4427/steps/14/logs/stdio
>> 
>> Seems like COMPUTE_HEADER_DEPENDENCIES isn't working quite right? I meanwhile have set it to
>> 'no'.
>> 
>> Thanks,
>> Alexander
>> 
>
> I've encountered this as well, and it appears to be a dependency graph
> issue. For me, compilation was fixed by a patch [1] that isn't in next or
> master yet ("Needs review" - ab/make-dependency [2]). If this an important
> fix to fast-track (I'll defer to more experienced contributors to determine
> that), it might speed up review to separate that patch from the rest of the
> series?
>
> [1] https://lore.kernel.org/git/patch-v4-19.23-2710f8af6cd-20211117T101807Z-avarab@gmail.com/
>
> [2] https://lore.kernel.org/git/xmqqbl1xqheu.fsf@gitster.g/

As the person who broke it sorry again about the bother.

If Junio would like to fast-track it that patch is cleanly
cherry-pickable on top of master.

I review by someone so that ab/make-dependency can move forward is also
most welcome.

Such a review can be done without any particular expertise of the
Makefile or the "GNU make" language. Just check out
gitster/ab/make-dependency and check if your normal build workflows work
as expected.