Bug 201751 - Export graph feature tries to use kghostview, which is not in KDE 4
Summary: Export graph feature tries to use kghostview, which is not in KDE 4
Status: CONFIRMED
Alias: None
Product: kcachegrind
Classification: Developer tools
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: Josef Weidendorfer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-07-28 11:39 UTC by Sami Liedes
Modified: 2021-03-09 23:01 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sami Liedes 2009-07-28 11:39:46 UTC
Version:           0.5.0kde (using KDE 4.2.4)
OS:                Linux
Installed from:    Debian testing/unstable Packages

Hi,

When I choose the Export graph menu option, kcachegrind tries to start kghostview, which AFAIK no longer exists in KDE 4.

From quick grepping, I found two places in kcachegrind sources where kghostview is mentioned.

Perhaps okular should be used instead.
Comment 1 Josef Weidendorfer 2009-07-28 14:42:25 UTC
Better no such dependence at all.
Running the PS viewer is only a convenience here. Similar to the
PNG export, it probably simply should provide a "Save as..." dialog
for the .dot file. The user can convert this himself.
Comment 2 Pino Toscano 2013-03-09 00:57:00 UTC
Git commit c2b9d1a38fda41b3c514aa573f12debc8be392f5 by Pino Toscano.
Committed on 09/03/2013 at 01:54.
Pushed by pino into branch 'KDE/4.10'.

use xdg-open instead of kghostview

instead of hardcoding kghostview as PS viewer, use xdg-open (which will open the default handler);
still not ideal (it should properly use temporary files and different QProcess'es for the tasks), but better than nothing for now

M  +1    -1    kcachegrind/toplevel.cpp
M  +1    -1    qcachegrind/qcgtoplevel.cpp

http://commits.kde.org/kcachegrind/c2b9d1a38fda41b3c514aa573f12debc8be392f5
Comment 3 Josef Weidendorfer 2013-03-11 14:52:51 UTC
The call graph view already has a context menu option to export the graph as dot/png
(we could add PS or better PDF there). So, what about simply removing that menu item
completely?

I remember that I added that item to make it easier for users to see
that such a feature is there at all. On the other hand, quite some stuff
is hidden in context menus in KCachegrind anyway.
Comment 4 Justin Zobel 2021-03-09 23:01:48 UTC
Thank you for the bug report.

As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.