Merge lp:~ken-vandine/content-hub/defaults_triplet into lp:content-hub

Proposed by Ken VanDine
Status: Merged
Approved by: Bill Filler
Approved revision: 73
Merged at revision: 72
Proposed branch: lp:~ken-vandine/content-hub/defaults_triplet
Merge into: lp:content-hub
Diff against target: 112 lines (+24/-13)
5 files modified
debian/control (+1/-1)
src/com/ubuntu/content/service/CMakeLists.txt (+3/-0)
src/com/ubuntu/content/service/com.ubuntu.content.hub.gschema.xml (+11/-11)
src/com/ubuntu/content/service/registry.cpp (+8/-1)
tests/CMakeLists.txt (+1/-0)
To merge this branch: bzr merge lp:~ken-vandine/content-hub/defaults_triplet
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+203841@code.launchpad.net

Commit message

Changed defaults to a triplet used to build the appid. Bumped build depends
for libupstart-app-launch2-dev needed for converting the triplet to an appid.

Description of the change

Changed defaults to a triplet used to build the appid. Bumped build depends
for libupstart-app-launch2-dev needed for converting the triplet to an appid.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

tested, approved. In CI Train

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-12-06 12:33:14 +0000
3+++ debian/control 2014-01-29 20:42:57 +0000
4@@ -11,7 +11,7 @@
5 libglib2.0-dev,
6 libgsettings-qt-dev,
7 libnih-dbus-dev,
8- libupstart-app-launch2-dev,
9+ libupstart-app-launch2-dev (>= 0.3+14.04.20140124),
10 qt5-default,
11 qtbase5-dev,
12 qtdeclarative5-dev,
13
14=== modified file 'src/com/ubuntu/content/service/CMakeLists.txt'
15--- src/com/ubuntu/content/service/CMakeLists.txt 2013-09-20 20:43:51 +0000
16+++ src/com/ubuntu/content/service/CMakeLists.txt 2014-01-29 20:42:57 +0000
17@@ -24,6 +24,7 @@
18 ${CMAKE_SOURCE_DIR}/src
19 ${CMAKE_SOURCE_DIR}/src/com/ubuntu/content
20 ${GSETTINGS_INCLUDE_DIRS}
21+ ${UPSTART_LAUNCH_INCLUDE_DIRS}
22 )
23
24 add_executable(
25@@ -41,6 +42,7 @@
26
27 content-hub
28 ${GSETTINGS_LDFLAGS}
29+ ${UPSTART_LAUNCH_LDFLAGS}
30 )
31
32 install(
33@@ -74,6 +76,7 @@
34
35 content-hub
36 ${GSETTINGS_LDFLAGS}
37+ ${UPSTART_LAUNCH_LDFLAGS}
38 )
39
40 install(
41
42=== modified file 'src/com/ubuntu/content/service/com.ubuntu.content.hub.gschema.xml'
43--- src/com/ubuntu/content/service/com.ubuntu.content.hub.gschema.xml 2013-12-10 15:59:15 +0000
44+++ src/com/ubuntu/content/service/com.ubuntu.content.hub.gschema.xml 2014-01-29 20:42:57 +0000
45@@ -1,17 +1,17 @@
46 <?xml version="1.0" encoding="UTF-8"?>
47 <schemalist>
48 <schema id="com.ubuntu.content.hub.default" path="/com/ubuntu/content/hub/peers/">
49- <key name="pictures" type="s">
50- <default>"gallery-app"</default>
51- </key>
52- <key name="music" type="s">
53- <default>"music-app"</default>
54- </key>
55- <key name="documents" type="s">
56- <default>"notes-app"</default>
57- </key>
58- <key name="contacts" type="s">
59- <default>"address-book-app"</default>
60+ <key name="pictures" type="as">
61+ <default>["com.ubuntu.gallery", "gallery", "current-user-version"]</default>
62+ </key>
63+ <key name="music" type="as">
64+ <default>[]</default>
65+ </key>
66+ <key name="documents" type="as">
67+ <default>[]</default>
68+ </key>
69+ <key name="contacts" type="as">
70+ <default>[]</default>
71 </key>
72 </schema>
73 <schema id="com.ubuntu.content.hub.all" path="/com/ubuntu/content/hub/peers/">
74
75=== modified file 'src/com/ubuntu/content/service/registry.cpp'
76--- src/com/ubuntu/content/service/registry.cpp 2013-09-20 19:54:51 +0000
77+++ src/com/ubuntu/content/service/registry.cpp 2014-01-29 20:42:57 +0000
78@@ -17,6 +17,7 @@
79 */
80
81 #include "registry.h"
82+#include <upstart-app-launch.h>
83
84 Registry::Registry() :
85 m_defaultPeers(new QGSettings("com.ubuntu.content.hub.default",
86@@ -32,7 +33,13 @@
87 {
88 qDebug() << Q_FUNC_INFO << type.id();
89 if (m_defaultPeers->keys().contains(type.id()))
90- return cuc::Peer(m_defaultPeers->get(type.id()).toString());
91+ {
92+ QStringList as(m_defaultPeers->get(type.id()).toStringList());
93+ std::string pkg = as[0].toStdString();
94+ std::string app = as[1].toStdString();
95+ std::string ver = as[2].toStdString();
96+ return cuc::Peer(QString::fromLocal8Bit(upstart_app_launch_triplet_to_app_id(pkg.c_str(), app.c_str(), ver.c_str())));
97+ }
98 else
99 return cuc::Peer();
100 }
101
102=== modified file 'tests/CMakeLists.txt'
103--- tests/CMakeLists.txt 2013-08-21 13:13:49 +0000
104+++ tests/CMakeLists.txt 2014-01-29 20:42:57 +0000
105@@ -24,6 +24,7 @@
106 include_directories (
107 ${CMAKE_SOURCE_DIR}/src
108 ${CMAKE_SOURCE_DIR}/src/com/ubuntu/content
109+ ${UPSTART_LAUNCH_INCLUDE_DIRS}
110 ${CMAKE_BINARY_DIR}/src
111 ${GMOCK_INCLUDE_DIR}
112 ${GTEST_INCLUDE_DIR}

Subscribers

People subscribed via source and target branches