Bug 178612 - konq-plugins: jpegorient is not available
Summary: konq-plugins: jpegorient is not available
Status: REOPENED
Alias: None
Product: konqueror
Classification: Applications
Component: imagerotation (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Carsten Pfeiffer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-23 23:11 UTC by Jean-Christophe Dubois
Modified: 2013-12-16 21:34 UTC (History)
2 users (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 Jean-Christophe Dubois 2008-12-23 23:11:27 UTC
Version:           1.1.80 (using 4.1.85 (KDE 4.1.85 (KDE 4.2 Beta2)), Kubuntu packages)
Compiler:          cc
OS:                Linux (x86_64) release 2.6.27-9-generic

When trying to apply a "transformation" like a rotation on a jpeg image through dolphin, we are getting a message stating that the "jpergorient" application cannot be found. 

It is a fact that while all dolphin/konqueror dependencies are installed (including konqueror-plugin-imagerotation, jpegtran or libjpeg-progs) the jpegorient binary is missing even if its man page is installed.

The closest thing I have on my system is "jpegexiforient" but this is not the application used by dolphin for jpeg manipulation.

It is not necessarily specific to dolphin as Konqueror is using the same setup which is incoherent on my actual system
Comment 1 Peter Penz 2008-12-24 09:53:19 UTC
Thanks for the report, but the provided services menus (see [1] for more details) are out of scope of Dolphin and it's a task of the distribution to assure that no service menus are installed when the corresponding applications are missing. I'd kindly ask to report this issue to Kubuntu.

[1] http://techbase.kde.org/index.php?title=Development/Tutorials/Creating_Konqueror_Service_Menus
Comment 2 Rex Dieter 2009-01-05 21:06:27 UTC
It's a valid bug against konq-plugins from extragear, reopening.  It includes a jpegorient script, but doesn't install it since r693875 (I'm assuming it was an oversight).

See downstream bug, 
https://bugzilla.redhat.com/show_bug.cgi?id=478736

I'm taking a look now how best to address this.

In the meantime, I've added TryExec calls so that missing jpegorient/convert will simply mean those items won't be shown in the menus.
Comment 3 Rex Dieter 2009-01-05 21:09:48 UTC
but... definitely not a dolpin bug, reassigning to unknown/me, for lack of somethingn better.
Comment 4 Rex Dieter 2009-01-05 22:12:47 UTC
Oi, jpegorient looks like it could use some serious work to be usable (last touched ~05), I'd opt for simply removing it until fixed/improved.
Comment 5 Rex Dieter 2009-01-05 22:27:45 UTC
SVN commit 906290 by rdieter:

omit broken jpegorient, at least until it can get some proper kde4 love

BUG: 178612


 M  +4 -5      CMakeLists.txt  


WebSVN link: http://websvn.kde.org/?view=rev&revision=906290
Comment 6 Kevin Kofler 2009-01-05 23:24:30 UTC
Uh, removing the file doesn't fix the issue (which is that the file is not being installed and a plugin needs it, you aren't disabling the plugin).

The script itself only needs fixing for the dcop calls in notify(). (There's no other KDE stuff used in the script.) Is this notification of file changes even still needed at all? The Dolphin part should be figuring that out by itself using inotify.

I think we can simply remove the notify() function and the notify "$file" call from the jpegorient script and reenable its installation in CMakeLists.txt.
Comment 7 Rex Dieter 2009-01-06 00:26:39 UTC
Shrug... my point was... it *never* worked, (in kde4's konq-plugins), so removing it isn't much of a regression.

Regarding notify() etc... I tested both konq/dolphin, and they both behaved badly (using cache'd thumbnails, etc...).  so I figured it best to omit rather than enabling something known bad.

Comment 8 Kevin Kofler 2009-01-06 00:34:13 UTC
Suckage... :-(
Comment 9 Rex Dieter 2009-08-03 18:56:43 UTC
marking as wishlist item looking for love.
Comment 10 Christoph Feck 2013-12-16 21:34:36 UTC
Konqueror plugins are now part of konqueror, reassigning.