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

Command line switch rflx --no-verification --unsafe generate doesn't seem to work #1295

Closed
dalybrown opened this issue Jan 25, 2024 · 3 comments
Assignees
Labels

Comments

@dalybrown
Copy link

I might not understand the purpose of the --no-verification --unsafe so you can delete this issue if that is the case.

There are times I want to speed up the generation of the SPARK code from our model when I know the model has already been checked and verified and the code proved (e.g., when the it is used as a dependency in another project).

I thought this switch would skip the Verifying ... part of the generate command. However, I still see those steps output every time I run the command with that switch which leads me to believe the switch isn't doing anything or the output is incorrect.

Please advise - thanks!

@dalybrown dalybrown added the bug label Jan 25, 2024
@treiher
Copy link
Collaborator

treiher commented Jan 26, 2024

Your expectation is correct, the output is just misleading. Despite the Verifying ... output, no verification is done when --no-verification is used. We will fix this. Thanks for reporting this!

Note that it is not necessary to use --no-verification if the verification has already been done on the same machine. The successful verification result will be cached (in $HOME/.cache/RecordFlux) and the verification will be skipped on the next run. Caching is only done for messages and refinements, as the verification time for other declarations is negligible.

@treiher treiher self-assigned this Jan 26, 2024
@dalybrown
Copy link
Author

Thanks!

Actually it's for CI pipelines that we use it when we know it has previously been verified but we might not have a cache available to us.

adacore-bot pushed a commit that referenced this issue Jan 29, 2024
Ref. #1295, eng/recordflux/RecordFlux#1522
@treiher
Copy link
Collaborator

treiher commented Jan 29, 2024

This is now fixed on main and will be part of version 0.18.0.

@treiher treiher closed this as completed Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants