Zulip Chat Archive

Stream: lean4

Topic: RFC: Exclude transitively failing files from build output


Yaël Dillies (Feb 26 2025 at 13:11):

Consider the following build log: https://github.com/leanprover-community/mathlib4/actions/runs/13527525523/job/37801718042 It contains hundreds of very uniformative lines of the form error: ././././Mathlib.lean: bad import 'Mathlib.Topology.Instances.Irrational', followed by the same number of lines in the Some required builds logged failures: part

Yaël Dillies (Feb 26 2025 at 13:12):

This happens because I added an import to Mathlib.Algebra.Group.Aut, which doesn't exist. But it's really hard to figure this information from the build output!

Yaël Dillies (Feb 26 2025 at 13:13):

What seems to happen is that files with a missing import do not get an olean, and so in turn appear missing to the files that import them.

Ruben Van de Velde (Feb 26 2025 at 13:44):

Also, the first error is on the file that doesn't exist, rather than the file that has the broken import

Kevin Buzzard (Feb 26 2025 at 16:16):

At least this is better than before though, where not only was the error triggered by a bad import obscure but it didn't even mention "bad import" at all (oh wait, that was for looping imports, maybe that's different)

Yaël Dillies (Feb 28 2025 at 11:34):

Yes indeed, that was different

Kim Morrison (Mar 01 2025 at 01:05):

@Yaël Dillies, can you open an issue?

Yaël Dillies (Mar 01 2025 at 09:16):

lean4#7280


Last updated: May 02 2025 at 03:31 UTC