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

Feature Request: Add support for using external editors via CLI execution #3

Closed
pushfoo opened this issue Oct 23, 2022 · 4 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@pushfoo
Copy link

pushfoo commented Oct 23, 2022

What

A way to directly start executing & debugging Octo source from the CLI.

At the moment, cadmium -r source.8o loads to the editor rather than immediately compiling & running it.

How

Two options seem easiest:

  1. Execution-specific commands like c-octo's octo-cli and octo-run
  2. Args to force build & debug instead of editing source.

Why

This is probably easier and more generally useful than shoehorning vim keybindings into Cadmium.

If this feature is already present, then it may need to be documented more clearly.

@gulrak gulrak self-assigned this Oct 27, 2022
@gulrak gulrak added the enhancement New feature or request label Oct 27, 2022
@gulrak
Copy link
Owner

gulrak commented Oct 27, 2022

Thank you for your suggestion! Indeed, a direct start option would be nice, I will squeeze it in before the release v1.0.2 (versioning of Cadmium will follow the even/odd principle, where even patch numbers like v1.0.0/v1.0.2/... are releases while odd ones like the current v1.0.1 indicates this is a between-releases wip state).
Can't promise a date yet.

@pushfoo
Copy link
Author

pushfoo commented Oct 27, 2022

Thank you for the response & honest time estimate!

I apologize for not being able to contribute here. I'm not up to date with C++ build systems or project structures.

@gulrak
Copy link
Owner

gulrak commented Oct 28, 2022

New behavior is now that the rom/source is just a positional parameter and the option -r/--run starts execution.

@gulrak gulrak added this to the Release v1.0.2 milestone Oct 28, 2022
@pushfoo
Copy link
Author

pushfoo commented Nov 22, 2022

I apologize for the delay in closing this. I can confirm this works in both a641cd8 and the tip of main. Thank you for the change!

I've filed #4 as a follow-up since those changes are not in scope for this ticket.

@pushfoo pushfoo closed this as completed Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants