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

Request: add exit alias to vvp's terminal interface #1128

Open
parker-research opened this issue May 21, 2024 · 2 comments
Open

Request: add exit alias to vvp's terminal interface #1128

parker-research opened this issue May 21, 2024 · 2 comments
Assignees

Comments

@parker-research
Copy link

When running vvp to execute a file, pressing Ctrl+C puts you into a terminal-like interface with the following commands:

> help
Commands can be from the following table of base commands,
or can be invocations of system tasks/functions.

cd       - Synonym for push.
cont     - Resume (continue) the simulation
finish   - Finish the simulation.
help     - Get help.
list     - List items in the current scope.
load     - Load a VPI module, a la vvp -m.
ls       - Shorthand for "list".
pop      - Pop one scope from the scope stack.
push     - Descend into the named scope.
step     - Single-step the scheduler for 1 event.
time     - Print the current simulation time.
trace    - Control statement tracing (on/off) when the code is instrumented.
where    - Show current scope, and scope hierarchy stack.

...

Adding an exit alias for finish would be very helpful so that it aligns with a standard *nix approach to "getting out of tools when at a prompt".

Additionally, Ctrl+C and Ctrl+D at this prompt should both probably do the finish action (that is, kill vvp).

@caryr
Copy link
Collaborator

caryr commented May 21, 2024

Adding an alias should be relatively simple. Switching the signal handling at the prompt is possible, just needs to be handled correctly so they only have the finish functionality at the prompt.

@caryr caryr self-assigned this Jun 17, 2024
@caryr
Copy link
Collaborator

caryr commented Jun 17, 2024

I've added the alias for exit to development. Tweaking the signals needs more work.

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

No branches or pull requests

2 participants