Bug 382460 - Exec line in kglobalaccel5 service file must not have any path prefix on Windows
Summary: Exec line in kglobalaccel5 service file must not have any path prefix on Windows
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kglobalaccel
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.36.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Ralf Habacker
URL:
Keywords:
Depends on:
Blocks: 380136
  Show dependency treegraph
 
Reported: 2017-07-18 09:03 UTC by Ralf Habacker
Modified: 2018-01-12 09:07 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.43.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Habacker 2017-07-18 09:03:12 UTC
The Exec line in kglobalaccel/src/runtime/org.kde.kglobalaccel.service.in
[D-BUS Service]
Name=org.kde.kglobalaccel
Exec=@CMAKE_INSTALL_PREFIX@/bin/kglobalaccel5

and ./kglobalaccel/src/runtime/dbus/org.kde.kglobalaccel.service.in
[D-BUS Service]
Name=org.kde.kglobalaccel
Exec=@CMAKE_INSTALL_PREFIX@/bin/kglobalaccel5

will not work on Windows (see bug 382458 for details)

Instead the prefix in the Exec line needs to be completely removed on windows e.g.

Exec=@SOME_PREFIX@kglobalaccel5

with SOME_PREFIX set to empty on Windows and to ${CMAKE_INSTALL_PREFIX}/bin/ otherwise

Also on Windows kglobalaccel5 needs to be installed in bindir
Comment 1 Ralf Habacker 2017-12-20 10:00:04 UTC
review request https://phabricator.kde.org/D9424
Comment 2 Ralf Habacker 2018-01-12 09:07:16 UTC
Git commit 205b69533bb54ee196cf50b3af1e4224d46ca94b by Ralf Habacker.
Committed on 12/01/2018 at 09:06.
Pushed by habacker into branch 'master'.

Use cmake function 'kdbusaddons_generate_dbus_service_file' from kdbusaddons to generate dbus service file

Summary:
FIXED-IN:5.43.0

Test Plan:
compiled against installed build from kdbusaddons git repo master branch

Reviewers: dfaure, bcooksley, apol

Reviewed By: dfaure

Subscribers: kfunk, vonreth, #windows, apol, #frameworks

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D9424

M  +1    -4    src/runtime/CMakeLists.txt
D  +0    -3    src/runtime/org.kde.kglobalaccel.service.in

https://commits.kde.org/kglobalaccel/205b69533bb54ee196cf50b3af1e4224d46ca94b