Bug 13949 - Fix "Exec=" for KDE systemsettings and hide ? .desktop in other DEs
Summary: Fix "Exec=" for KDE systemsettings and hide ? .desktop in other DEs
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Luc Menut
QA Contact:
URL:
Whiteboard:
Keywords: Junior_job, Triaged
Depends on:
Blocks:
 
Reported: 2014-08-20 13:00 CEST by Alex Loginov
Modified: 2014-11-30 13:22 CET (History)
4 users (show)

See Also:
Source RPM: kdebase4-workspace
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Alex Loginov 2014-08-20 13:00:19 CEST 
Description of problem: KDE systemsettings are available in all DEs, but it's KDE only.


Version-Release number of selected component (if applicable): kdebase4-workspace-4.11.11-4.mga5.i586.rpm


How reproducible: always


Solution:
Hide KDE systemsettings in all DEs except KDE.


Reproducible: 

Steps to Reproduce:
Comment 1 Alex Loginov 2014-08-20 13:01:59 CEST 
echo "OnlyShowIn=KDE;" >> /usr/share/applications/kde4/systemsettings.desktop
echo "OnlyShowIn=KDE;" >> /usr/share/applications/kde4/kdesystemsettings.desktop
Manuel Hiebel 2014-08-20 19:24:34 CEST

Keywords: (none) => Triaged
CC: (none) => balcaen.john, lmenut
Assignee: bugsquad => mageia

Rémi Verschelde 2014-09-17 14:29:44 CEST

Keywords: (none) => Junior_job
CC: (none) => remi

Comment 2 Alex Loginov 2014-11-17 21:12:36 CET 
We have
Exec=systemsettings -caption %c %i for kdesystemsettings.desktop
Exec=systemsettings -caption "%c" %i for systemsettings.desktop
and Exec=systemsettings -caption "%c" %i is correct, but Exec=systemsettings -caption %c %i is incorrect.
See also https://bugs.mageia.org/show_bug.cgi?id=14589

Summary: Add "OnlyShowIn=KDE" for KDE systemsettings => Add "OnlyShowIn=KDE" and fix "Exec=" for KDE systemsettings

Comment 3 Alex Loginov 2014-11-17 21:22:49 CET 
It seems kdesystemsettings.desktop is for all DEs and there is NotShowIn=KDE.
It seems systemsettings.desktop for KDE only and there is OnlyShowIn=KDE.
So, this bug is for Exec= only.

Summary: Add "OnlyShowIn=KDE" and fix "Exec=" for KDE systemsettings => Fix "Exec=" for KDE systemsettings

Alex Loginov 2014-11-27 21:08:53 CET

Summary: Fix "Exec=" for KDE systemsettings => Fix "Exec=" for KDE systemsettings and hide ? .desktop in other DEs

Comment 4 Luc Menut 2014-11-30 13:22:39 CET 
systemsettings.desktop fixed in svn (added OnlyShowIn=KDE b/c kdesystemsettings.desktop is used for other DEs).

Status: NEW => RESOLVED
CC: (none) => mageia
Hardware: i586 => All
Resolution: (none) => FIXED
Assignee: mageia => lmenut
Note You need to log in before you can comment on or make changes to this bug.