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

extend appimage debugging notes #459

Merged
merged 1 commit into from
Jan 11, 2023
Merged

extend appimage debugging notes #459

merged 1 commit into from
Jan 11, 2023

Conversation

GitMensch
Copy link
Contributor

No description provided.

@GitMensch GitMensch force-pushed the patch-1 branch 3 times, most recently from 6758159 to f6419fe Compare January 11, 2023 11:50

```bash
$ ./hotspot-v1.4.0-43-g02fd82c-x86_64.AppImage --appimage-extract
$ PATH=./squashfs-root/usr/bin:$PATH LD_LIBRARY_PATH=./squashfs-root/usr/lib64:/usr/local/lib64:/usr/lib64:./squashfs-root/usr/lib:$LD_LIBRARY_PATH gdb -q --exec=hotspot --symbols hotspot-debuginfo-v1.4.0-43-g02fd82c/usr/bin/hotspot -ex run
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we should add some kind of --appimage-debug gdb or similar to our AppRun which would prepend the debugger before starting hotspot?

or maybe better: extract the code that sets up the env from the AppRun, such that we can do:

source appimage.env
gdb -q --exec=hotspot --symbols hotspot-debuginfo-v1.4.0-43-g02fd82c/usr/bin/hotspot -ex run

directly?

Copy link
Contributor Author

@GitMensch GitMensch Jan 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

appimage.env would be marvelous! Even better would be if appimage can be executed, too: /path/to/appimage.env hotspot (or, using gdb that way).

--appimage-debug Name-Of-Debugger-Here [additional debugger arguments] [-- additional arguments passed to hotspot] would be nice, too; allowing things like /hotspot-v1.4.0-43-g02fd82c-x86_64.AppImage --appimage-debug /opt/path/to/newer/gdb/bin/gdb -x additional_printers.gdb --args -- some.perf.data

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we create one (or two) issue(s) for those good ideas of yours?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes please do that

Copy link
Member

@milianw milianw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm already, do you want to make it even better or should I merge as-is already?

@GitMensch
Copy link
Contributor Author

Thanks for the review - no further changes planned.

@milianw milianw merged commit 0e1db7a into KDAB:master Jan 11, 2023
@GitMensch GitMensch deleted the patch-1 branch January 11, 2023 19:49
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