Bug 13949

Summary:	Fix "Exec=" for KDE systemsettings and hide ? .desktop in other DEs
Product:	Mageia	Reporter:	Alex Loginov <loginov_alex>
Component:	RPM Packages	Assignee:	Luc Menut <lmenut>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	balcaen.john, lmenut, mageia, rverschelde
Version:	Cauldron	Keywords:	Junior_job, Triaged
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	kdebase4-workspace	CVE:
Status comment:

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