-
Notifications
You must be signed in to change notification settings - Fork 243
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
please add optional plugin kgraphviewerpart
to the AppImage
#416
Comments
yes this is planned to some degree. konsolepart would dramatically increase the size of the AppImage - I'm unsure if that's worth it. kgraphviewerpart triggers a crash in libkgraphviewer, which I first have to fix upstream. |
And FTR: all other optional dependencies should already be included, including QCustomPlot and the rust and D demangling support |
Sounds good. As I could not found an issue for that: what is konsolepart used for and how would one install it manually? |
it's used in the record page of hotspot to show CLI output of client applications in a proper way, i.e. including ansi escape sequence highlighting and so forth. |
OK, then other than libkgraphviewer that's completely "optional" and only changes the "beauty" of the output, not lead to a "missing feature" in general. Thanks for that information. |
kgraphviewerpart
to the AppImage
Is there an upstream issue to check its status, which you can link to? |
ping @milianw - this would allow me to properly check upstream side note as mentioned in #318:
|
This comment was marked as outdated.
This comment was marked as outdated.
still based on KDDW 1.6. |
:-) Friendly ping @milianw after a bit more than a year about the upstream issue and the documentation for this feature. |
I don't think there's an upstream issue, and upstream is pretty "dead", meaning we'd have to fix it ourselves. I'm tempted to leave this disabled for now as it's just not working properly and the visualization isn't one I'm missing myself. help welcome! |
If it would really need a fix ourselves - can you please fork upstream in KDAB and create an issue there with the details (how to reproduce whatever issue happens, the known details of the problem, ...)? This would then allow me to at least inspect it further, possibly fix it and send a PR with a fix. |
It is totally reasonable to have the minimal AppImage dependencies on the system, in this case:
apt install libfuse2 fuse
- or manual extraction of the image.But after doing that and executing either the AppImage and starting hotspot I'm shown:
It is definitely good to have those dependencies be optional (hotspot starts and works in general) but I'll obviously won't have the full feature set without additional system libraries (and I don't knwo yet how to even get those).
Can you please include these (and possibly more optional) plugins to the AppImage generation?
The text was updated successfully, but these errors were encountered: