Review Request 123076: Remove KNotifications dependency from libkdeconnectcore

Aleix Pol Gonzalez aleixpol at kde.org
Tue Mar 24 11:26:59 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123076/
-----------------------------------------------------------

(Updated March 24, 2015, 11:26 a.m.)


Status
------

This change has been marked as submitted.


Review request for kdeconnect.


Changes
-------

Submitted with commit 8f777040f7044d72bc491ce0e34fa6ef9fad2b03 by Aleix Pol to branch master.


Repository: kdeconnect-kde


Description
-------

Moves the implementation into the actual daemon. This opens the possibility for different ways to expose these notifications depending on where the libkdeconnect will be deployed.


Diffs
-----

  CMakeLists.txt 05a1b2c 
  core/CMakeLists.txt 2026317 
  core/daemon.h 1817bdf 
  core/daemon.cpp 4e9cec4 
  core/device.cpp 96c62b5 
  core/kdeconnectconfig.cpp 1356f43 
  daemon/CMakeLists.txt 8847c32 
  daemon/kdeconnectd.cpp cb7de79 
  plugins/battery/CMakeLists.txt f6a6c5b 
  plugins/notifications/CMakeLists.txt 565bf7f 
  plugins/ping/CMakeLists.txt 15daa70 
  plugins/screensaver-inhibit/CMakeLists.txt 8936930 
  plugins/sftp/CMakeLists.txt cecedf8 
  plugins/share/CMakeLists.txt fed160c 
  plugins/telephony/CMakeLists.txt 9ec3027 

Diff: https://git.reviewboard.kde.org/r/123076/diff/


Testing
-------

I unpaired myself and paired back.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150324/7ee3d31e/attachment-0001.html>


More information about the KDEConnect mailing list