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.

Sign up for GitHub

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

Jump to bottom

debugger: operate on prebuilt binaries with exec command #7

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

debugger: operate on prebuilt binaries with exec command #7

wants to merge 2 commits into from

Conversation

JackMordaunt
Copy link

This command skips the compile step and uses the named binary directly.

We add a new command exec that accepts a prebuilt binary name and skips the compile step.

Closes #6

@JackMordaunt
debugger: operate on prebuilt binaries with exec command
a12a85b 
This command skips the compile step and uses the named binary directly.

Signed-off-by: Jack Mordaunt <[email protected]>
@emad-elsaid
Copy link
Owner

This was faster than I expected, I hope you didn't go into much trouble understanding the code. I'll test it later today and merge it. thank you for the effort

@emad-elsaid
Copy link
Owner

Can you describe how did you test this feature? because I tried the following:

gh pr checkout 7
go install .
debugger exec `which reflex`

where reflex is a go program I installed with go install https://github.com/cespare/reflex

but when I try to put a breakpoint for example I get this error in the terminal

^C2024/02/24 23:17:26 ERR could not find file /home/emad/go/pkg/mod/github.com/cespare/[email protected]/main.go
2024/02/24 23:17:30 ERR could not find file /home/emad/go/pkg/mod/github.com/cespare/[email protected]/main.go
2024/02/24 23:17:33 debugger needs a command: `run`, `exec` or `test`

@JackMordaunt
Copy link
Author

Hmmm. There is a bug where during a restart we try to rebuild. Need to tell it not to do that.

However, setting breakpoints works for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Run against pre-compiled executable
2 participants
@JackMordaunt @emad-elsaid