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

crucible-cli: Library for sharing code between Crucible CLI frontends #1123

Merged
merged 7 commits into from
Nov 8, 2023

Conversation

langston-barrett
Copy link
Contributor

Previously, crucible-syntax had some code for setting up and executing the Crucible symbolic simulator. This functionality is very useful for testing and interactive experimentation via a CLI, but is not necessary for most library clients of crucible-syntax. Furthermore, creating additional CLI frontends entailed copy-pasting a lot of optparse-applicative-related code from crucible-syntax's' crucibler executable. Now, both these issues are addressed by factoring this code into a separate crucible-cli package.

Previously, `crucible-syntax` had some code for setting up and executing the
Crucible symbolic simulator. This functionality is very useful for testing and
interactive experimentation via a CLI, but is not necessary for most library
clients of `crucible-syntax`. Furthermore, creating additional CLI frontends
entailed copy-pasting a lot of `optparse-applicative`-related code from
`crucible-syntax`'s' `crucibler` executable. Now, both these issues are
addressed by factoring this code into a separate `crucible-cli` package.
@langston-barrett langston-barrett marked this pull request as ready for review November 8, 2023 17:06
crucible-cli/app/Main.hs Show resolved Hide resolved
crucible-cli/src/Lang/Crucible/CLI.hs Outdated Show resolved Hide resolved
@langston-barrett langston-barrett merged commit 0d26e4a into GaloisInc:master Nov 8, 2023
31 checks passed
@langston-barrett langston-barrett deleted the lb/cli branch November 8, 2023 19:58
RyanGlScott added a commit that referenced this pull request Feb 1, 2024
RyanGlScott added a commit that referenced this pull request Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants