By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2325 - Delete k3b-iso.desktop
Summary: Delete k3b-iso.desktop
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks: R14.1.0
  Show dependency tree / graph
 
Reported: 2015-01-25 17:52 CST by Darrell
Modified: 2015-01-28 17:00 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Patch to remove redundant k3b-iso.desktop (807 bytes, patch)
2015-01-25 17:52 CST, Darrell 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2015-01-25 17:52:58 CST 
Created attachment 2433 [details]
Patch to remove redundant k3b-iso.desktop

In applications/k3b/src/ there are two desktop files:

k3b.desktop
k3b-iso.desktop

Installing both desktop files creates two 'K3b' menu entries in the konqueror 'Open With' menu.

As k3b.desktop contains the same Mimetypes as k3b-iso.desktop, the latter is redundant. To reduce redundancy and menu confusion, delete k3b-iso.desktop.
Comment 1 Michele Calgaro 2015-01-25 18:16:05 CST 
They launch two different commands:
- k3b %U
- k3b --image %U
but they seems to do exactly the same thing when tested on an .iso image: they both open the burning dialog.
It seems safe to remove the 'k3b-iso.desktop' file without loss of functionality and reduce menu cluttering.
Comment 2 Michele Calgaro 2015-01-27 16:38:25 CST 
The patch looks good. Darrell, you can either push it yourself or provide a "Signed-off-by" clause here and then I can push it.
Comment 3 Darrell 2015-01-27 20:19:33 CST 
Patch pushed in commit 17800ee3. Patch needs to be pushed to the R14.x.x branches before closing the report as resolved.
Comment 4 Michele Calgaro 2015-01-27 21:03:20 CST 
> Patch needs to be pushed to the R14.x.x branches before closing the report as 
> resolved.
New functionalities and meaningful changes to GUI (i.e. changes to what the user can/can't do) should not be integrated in R14.0.x, only in R14.1.x. So we do not need to push the change to R14.0.x and we can close the bug.