c4-project / c4t

Runs concurrent C compiler tests
MIT License
1 stars 0 forks source link

Allow invoke on already-invoked plans #59

Closed MattWindsor91 closed 3 years ago

MattWindsor91 commented 3 years ago

Currently, if you try to act-tester-invoke a plan that has already been invoked, the invoker will fail with a cryptic error message:

A fatal error has occurred:
while forwarding updates: remote error: while running compiler: duplicate compile result: compiler="clang.9"

This occurs, I think, because the existing compiler records from the last invocation clash with the ones the invoker is trying to insert.

Trying to re-invoke a plan is actually useful for debugging errors, since it lets us check compiler failures for reproducibility. As such, the tester should really be allowing re-invoking.

My current thoughts on this feel like: