The International Simutrans Forum

Development => Bug Reports => Topic started by: Andarix on January 10, 2025, 06:35:33 PM

Title: Action Automated Tests r11588
Post by: Andarix on January 10, 2025, 06:35:33 PM
https://github.com/simutrans/simutrans/actions/runs/12709483652/job/35428578089 (https://github.com/simutrans/simutrans/actions/runs/12709483652/job/35428578089)

Quote2025-01-10T12:29:51.4386871Z 00:00:04 SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior src/simutrans/obj/leitung2.cc:566:37
2025-01-10T12:29:52.9592564Z Process crashed (test failed)
Title: Re: Action Automated Tests r11588
Post by: ceeac on January 10, 2025, 07:09:32 PM
Looks like the new version of pak64 does not include an object named "PowerSource".

EDIT: Found the cause: The rules for the pak files in the pak64 Makefile need to have a dependency on the copy target because the copy target deletes the directory that contains the pak files. If the pak file is generated before update_externals.sh finishes, the pak file is removed.
Title: Re: Action Automated Tests r11588
Post by: prissi on January 11, 2025, 04:26:29 AM
I see. But I found no way to make sure that copy is executed before everything. Even specifying
$(DIRS64): copy
Does not do the trick. Nothing seems to work properly with make -j 4 for instance.