Bug 376068 - kscreenlocker has unneeded link dependency on libXcursor
Summary: kscreenlocker has unneeded link dependency on libXcursor
Status: RESOLVED FIXED
Alias: None
Product: kscreenlocker
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-05 17:05 UTC by Viktor Ostashevskyi
Modified: 2017-02-08 17:12 UTC (History)
4 users (show)

See Also:
Latest Commit: https://commits.kde.org/kscreenlocker/dd9a089cfaefb24f012a137618e00005a48b6009
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Viktor Ostashevskyi 2017-02-05 17:05:01 UTC 
Grepping in sources doesn't show any usage except one line in main CMakeLists.txt:

target_link_libraries(KScreenLocker                                                                                                   
PUBLIC                                                                                                                                
   Qt5::Core                                                                                                                          
   Qt5::X11Extras                                                                                                                     
PRIVATE                                                                                                                               
   KF5::I18n                                                                                                                          
   KF5::IdleTime                                                                                                                      
   KF5::GlobalAccel                                                                                                                   
   KF5::Notifications                                                                                                                 
   KF5::CoreAddons                                                                                                                    
   KF5::ConfigGui                                                                                                                     
   KF5::WindowSystem                                                                                                                  
   ${X11_LIBRARIES}                                                                                                                   
   ${X11_Xcursor_LIB}                                                                                                                 
   XCB::XCB                                                                                                                           
   XCB::KEYSYMS                                                                                                                       
   KF5::WaylandServer                                                                                                                 
   Wayland::Server                                                                                                                    
)

I have a strong feeling it is not needed in CMakeLists.txt
Comment 1 Kai Uwe Broulik 2017-02-08 17:12:43 UTC 
Git commit dd9a089cfaefb24f012a137618e00005a48b6009 by Kai Uwe Broulik.
Committed on 08/02/2017 at 17:12.
Pushed by broulik into branch 'master'.

Stop linking X11_Xcursor_LIB in KScreenLocker

It's not used.

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

M  +0    -1    CMakeLists.txt

https://commits.kde.org/kscreenlocker/dd9a089cfaefb24f012a137618e00005a48b6009