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):
Last updated: May 02 2025 at 03:31 UTC