D17817: Build without D-Bus on Android

2019-01-01 Thread Volker Krause
This revision was automatically updated to reflect the committed changes. Closed by commit R289:8c3fa065fb1e: Build without D-Bus on Android (authored by vkrause). REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17817?vs=48282&id=48469 REVISION DETAIL

D17817: Build without D-Bus on Android

2018-12-30 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Given the discussion I think this should go in. REPOSITORY R289 KNotifications BRANCH arcpatch-D17817 REVISION DETAIL https://phabricator.kde.org/D17817 To: vkrause, dhaumann Cc

D17817: Build without D-Bus on Android

2018-12-30 Thread Volker Krause
vkrause added a comment. D17868 adds kwindowsystem to the Android CI, which is (still) a dependency for knotifications, I'll add knotifications itself too once the necessary patches have landed. REPOSITORY R289 KNotifications REVISION DETAIL https://

D17817: Build without D-Bus on Android

2018-12-29 Thread Ben Cooksley
bcooksley added a comment. Android CI builds are only enabled for the following repositories: - "frameworks/extra-cmake-modules" - "frameworks/karchive" - "frameworks/attica" - "frameworks/breeze-icons" - "frameworks/kcodecs" - "frameworks/kconfig" - "frameworks/k

D17817: Build without D-Bus on Android

2018-12-29 Thread Aleix Pol Gonzalez
apol added a comment. In D17817#383403 , @vkrause wrote: > In D17817#383354 , @dhaumann wrote: > > > +1, although it will likely break quickly since this branch is not tested via CI. Then again, thi

D17817: Build without D-Bus on Android

2018-12-28 Thread Volker Krause
vkrause added a comment. In D17817#383354 , @dhaumann wrote: > +1, although it will likely break quickly since this branch is not tested via CI. Then again, this is also vslid for e.g. the KSyntaxHighlighting branches that were added for standal

D17817: Build without D-Bus on Android

2018-12-28 Thread Dominik Haumann
dhaumann added a comment. +1, although it will likely break quickly since this branch is not tested via CI. Then again, this is also vslid for e.g. the KSyntaxHighlighting branches that were added for standalone deployment. Another +1 would be appreciated. REPOSITORY R289 KNotificatio

D17817: Build without D-Bus on Android

2018-12-28 Thread Volker Krause
vkrause updated this revision to Diff 48282. vkrause added a comment. Remove accidentally added block that is not supposed to be part of this commit. REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17817?vs=48241&id=48282 BRANCH arcpatch-D17817 R

D17817: Build without D-Bus on Android

2018-12-27 Thread Volker Krause
vkrause created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. vkrause requested review of this revision. REVISION SUMMARY This is probably the most invasive part of the Android support, the remainder is implementing the native Android backe