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

Florian Reinhard florian.reinhard at googlemail.com
Fri Dec 28 23:45:17 UTC 2012


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

Review request for Telepathy.


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
-----

  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 
  src/accounts-list-delegate.cpp 31ee0804f4747d2823b3054fa8fbdd25a9d3a117 
  src/add-account-assistant.cpp 07eae584091e7e1a47ca7c63b1652d5ba545ae94 
  src/kcm-telepathy-accounts.h c944042710ef75edfc58408bcfc384d64acb0ce0 
  src/kcm-telepathy-accounts.cpp 52565c5b69f047cd8a32f8f0d4e672d307412640 
  src/main-widget.ui 4983c71c4d7d183525dd705fc5859f38d1f64d0b 

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/20121228/5ae66c8e/attachment.html>


More information about the KDE-Telepathy mailing list