Review Request: check in AddAccountAssistant for installed connection managers and optional install them

Florian Reinhard florian.reinhard at googlemail.com
Sat Dec 29 00:04:41 UTC 2012


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

(Updated Dec. 29, 2012, 12:04 a.m.)


Review request for Telepathy.


Changes
-------

previous patch contained changes that were not supposed to end up here (yet)


Description
-------

Check in SimpleProfileSelectWidget if required connection managers are installed
    
* new cmake option -DPACKAGE_INSTALLATION=[apt,packagekit]
* default: no package installation enabled just promt a message
* on the first page of AddAccountAssistant profiles with no CM installed will be disabled


Diffs (updated)
-----

  src/KCMTelepathyAccounts/CMakeLists.txt e19f3accf465f852012d251f69bbfa2ff372bb7f 
  src/KCMTelepathyAccounts/package-install-action.h PRE-CREATION 
  src/KCMTelepathyAccounts/package-install-action.cpp PRE-CREATION 
  src/KCMTelepathyAccounts/profile-list-model.h 8b313d19765b63071408047b5d183ecc419500de 
  src/KCMTelepathyAccounts/profile-list-model.cpp 12752928f377552a323a502557d2717819c257de 
  src/KCMTelepathyAccounts/simple-profile-select-widget.h 52eede1c4d5b7a39143b71c19f33dcb965827bf9 
  src/KCMTelepathyAccounts/simple-profile-select-widget.cpp 5f9a516e74a17c8cf86a5f5382d5071baff1c8a8 
  src/KCMTelepathyAccounts/simple-profile-select-widget.ui 5ad8e79a7bd4fd6d52dac61cbd88becb0cd569ae 

Diff: http://git.reviewboard.kde.org/r/107995/diff/


Testing
-------

* remove telepathy-haze
* start the kcm
* result see screenshot


Screenshots
-----------

telepathy-haze not installed
  http://git.reviewboard.kde.org/r/107995/s/937/


Thanks,

Florian Reinhard

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121229/3fec2bbf/attachment-0001.html>


More information about the KDE-Telepathy mailing list