Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework of clerk run #789

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Rework of clerk run #789

wants to merge 14 commits into from

Conversation

AltGr
Copy link
Contributor

@AltGr AltGr commented Feb 14, 2025

  • remove exec specific targets in ninja for the different backends
  • tune the ninja file to remove targets not asked for (it still contains all scanned files though)
  • add specific handling of exec targets directly in clerk

The main highlight is being able to run e.g. clerk run tests/*/good/ --backend=c -c--autotest

@AltGr AltGr force-pushed the clerkbld branch 2 times, most recently from 745fa3d to bf673dc Compare February 17, 2025 13:13
This enables `clerk run foo.catala_en`, with additional flags such as `--command=typecheck`

Flag `--backend` compiles a binary using the given backend (OCaml, C, python)
and then runs that. Useful with e.g. `-c--autotest`. There is only partial
support for OCaml yet.
AltGr added 10 commits February 19, 2025 18:33
And support for running on the OCaml and C backends, e.g.
```
clerk run tests/xxx.catala_en --backend=c -c--autotest
```
for tests, etc. This puts the interpreter on par with the other backends, unifying for cross-backend testing
Generating an executable that doesn't run any scope is now only a warning.

This is for consistency with the cross-backend testing, where "no test scopes
failed" should return with 0.
It's required since that adds more dependencies (as it makes the compiler also run the interpreter)
it makes it clearer what was measured and doesn't disrupt the debug line
Not yet under CI since they'll need some fixing first
@AltGr
Copy link
Contributor Author

AltGr commented Feb 19, 2025

Sister PR on catala-examples: CatalaLang/catala-examples#20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant