Merge lp:~ken-vandine/content-hub/click_hook into lp:~content-hub-team/content-hub/trunk
Status: | Merged |
---|---|
Approved by: | Ken VanDine |
Approved revision: | 54 |
Merged at revision: | 59 |
Proposed branch: | lp:~ken-vandine/content-hub/click_hook |
Merge into: | lp:~content-hub-team/content-hub/trunk |
Diff against target: |
602 lines (+249/-55) 20 files modified
debian/content-hub.install (+2/-0) debian/control (+1/-0) debian/rules (+2/-2) src/com/ubuntu/content/detail/peer_registry.h (+4/-0) src/com/ubuntu/content/service/CMakeLists.txt (+9/-17) src/com/ubuntu/content/service/content-hub.hook.in (+2/-2) src/com/ubuntu/content/service/helper.cpp (+11/-6) src/com/ubuntu/content/service/hook.cpp (+59/-19) src/com/ubuntu/content/service/hook.h (+7/-4) src/com/ubuntu/content/service/registry.cpp (+31/-0) src/com/ubuntu/content/service/registry.h (+2/-0) tests/acceptance-tests/CMakeLists.txt (+22/-0) tests/acceptance-tests/app_hub_communication_default_peer.cpp (+2/-1) tests/acceptance-tests/app_hub_communication_handler.cpp (+2/-1) tests/acceptance-tests/app_hub_communication_known_peers.cpp (+2/-1) tests/acceptance-tests/app_hub_communication_stores.cpp (+2/-1) tests/acceptance-tests/app_hub_communication_transfer.cpp (+2/-1) tests/acceptance-tests/bad.json (+6/-0) tests/acceptance-tests/good.json (+6/-0) tests/acceptance-tests/test_hook.cpp (+75/-0) |
To merge this branch: | bzr merge lp:~ken-vandine/content-hub/click_hook |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Günter Schwann (community) | Approve | ||
PS Jenkins bot | continuous-integration | Approve | |
Review via email: mp+187341@code.launchpad.net |
Commit message
Added a click hook, which iterates installed known peers, removing them when they are no longer installed and adding peers when needed.
Description of the change
Added a click hook, which iterates installed known peers, removing them when they are no longer installed and adding peers when needed. This looks for JSON files installed as part of a click package in ${HOME}
Example syntax:
{
"source": [
"pictures",
"music"
]
}
Peers that provide an export handler for a type, should add this file to their click package and define it in their manifest with the content-hub click hook. Example from a manifest file:
"hooks": {
}
},
And add content_exchange to the apparmor policy groups.
FAILED: Continuous integration, rev:53 /code.launchpad .net/~ken- vandine/ content- hub/click_ hook/+merge/ 187341/ +edit-commit- message
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http:// jenkins. qa.ubuntu. com/job/ content- hub-ci/ 87/ jenkins. qa.ubuntu. com/job/ content- hub-saucy- amd64-ci/ 86/console jenkins. qa.ubuntu. com/job/ content- hub-saucy- armhf-ci/ 86/console jenkins. qa.ubuntu. com/job/ content- hub-saucy- i386-ci/ 86/console
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ content- hub-ci/ 87/rebuild
http://