Merge lp:~phablet-team/telephony-service/fix-hardcoded-paths into lp:telephony-service

Proposed by Roberto Mier Escandon
Status: Merged
Approved by: Tiago Salem Herrmann
Approved revision: 1215
Merged at revision: 1221
Proposed branch: lp:~phablet-team/telephony-service/fix-hardcoded-paths
Merge into: lp:telephony-service
Diff against target: 112 lines (+86/-2)
2 files modified
config.h.in (+4/-2)
snapcraft.yaml (+82/-0)
To merge this branch: bzr merge lp:~phablet-team/telephony-service/fix-hardcoded-paths
Reviewer Review Type Date Requested Status
Tiago Salem Herrmann (community) Approve
system-apps-ci-bot continuous-integration Approve
Review via email: mp+311716@code.launchpad.net

Commit message

prefixes snap env var value to paths in case service is into a snap package

Description of the change

prefixes snap env var value to paths in case service is into a snap package

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:1213
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/87/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2055/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2058
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1888/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1888/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1888/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1888
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1888/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/87/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:1214
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/88/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2057/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2060
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1890/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1890/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1890/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1890/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1890/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1890/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1890/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1890
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1890/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1890/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/88/rebuild

review: Needs Fixing (continuous-integration)
1215. By Roberto Mier Escandon

corrected bad var reference

Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:1215
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/91/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/2066
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/505
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2069
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1899/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1899
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1899/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-telephony-service-ci/91/rebuild

review: Approve (continuous-integration)
Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

Works fine. Tested telephony apps with ofono emulator and they are able to send messages and make calls.

review: Approve
1216. By Roberto Mier Escandon

set explicitly the architectures to build

1217. By Roberto Mier Escandon

rolledback to any architecture build

1218. By Renato Araujo Oliveira Filho

Trunk merged.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.h.in'
2--- config.h.in 2015-08-26 08:19:56 +0000
3+++ config.h.in 2016-12-05 15:15:21 +0000
4@@ -37,7 +37,9 @@
5 }
6
7 inline QString telephonyServiceDir() {
8- if (isRunningInstalled()) {
9+ if (qEnvironmentVariableIsSet("SNAP")) {
10+ return QString("%1/@CMAKE_INSTALL_PREFIX@/@TELEPHONY_SERVICE_DIR@/").arg(QString(qgetenv("SNAP")));
11+ } else if (isRunningInstalled()) {
12 return QString("@CMAKE_INSTALL_PREFIX@/@TELEPHONY_SERVICE_DIR@/");
13 } else {
14 return QString("@CMAKE_SOURCE_DIR@/");
15@@ -45,7 +47,7 @@
16 }
17
18 inline QString ubuntuPhonePluginPath() {
19- if (isRunningInstalled()) {
20+ if (isRunningInstalled() || qEnvironmentVariableIsSet("SNAP")) {
21 return QString::null;
22 } else {
23 return QString("@CMAKE_SOURCE_DIR@/");
24
25=== added directory 'snap'
26=== added directory 'snap/ubuntu-app-platform'
27=== added file 'snapcraft.yaml'
28--- snapcraft.yaml 1970-01-01 00:00:00 +0000
29+++ snapcraft.yaml 2016-12-05 15:15:21 +0000
30@@ -0,0 +1,82 @@
31+name: telephony-service
32+version: 0.1
33+summary: Telephony service components for Ubuntu - backend
34+description:
35+ Telephony service components for Ubuntu.
36+ .
37+ This package contains the backend components providing the telephony
38+ features such as calling and texting.
39+confinement: devmode
40+grade: devel
41+
42+apps:
43+ indicator:
44+ command: desktop-launch $SNAP/usr/bin/telephony-service-indicator
45+ plugs: [platform]
46+
47+ handler:
48+ command: desktop-launch $SNAP/usr/bin/telephony-service-handler
49+ plugs: [platform]
50+
51+ approver:
52+ command: desktop-launch $SNAP/usr/bin/telephony-service-approver
53+ plugs: [platform]
54+
55+plugs:
56+ platform:
57+ interface: content
58+ content: ubuntu-app-platform1
59+ target: ubuntu-app-platform
60+ default-provider: ubuntu-app-platform
61+
62+#slots:
63+# telephony:
64+# interface: telephony
65+
66+parts:
67+ telephony-service:
68+ plugin: cmake
69+ configflags: [-DCMAKE_INSTALL_PREFIX=/usr, -DCMAKE_CROSSCOMPILING=On]
70+ source: .
71+ build-packages:
72+ - cmake
73+ - dh-translations
74+ - dconf-cli
75+ - gnome-keyring
76+ - gsettings-ubuntu-schemas
77+ - libhistoryservice-dev
78+ - libicu-dev
79+ - libmessaging-menu-dev
80+ - libnotify-dev
81+ - libphonenumber-dev
82+ - libtelepathy-qt5-dev
83+ - libubuntu-application-api-dev
84+ - libprotobuf-dev
85+ - pkg-config
86+ - qt5-default
87+ - qtbase5-dev
88+ - qtdeclarative5-dev
89+ - qtdeclarative5-dev-tools
90+ - qtfeedback5-dev
91+ - qtdeclarative5-ubuntu-ui-toolkit-plugin
92+ - qtmultimedia5-dev
93+ - qtpim5-dev
94+ - libusermetricsinput1-dev
95+ - telepathy-mission-control-5
96+ - upstart
97+ stage-packages:
98+ - accountsservice-ubuntu-touch-schemas
99+ - gsettings-ubuntu-schemas
100+ - history-service
101+ - libnotify4
102+ - telepathy-mission-control-5
103+ - ubuntu-sounds
104+ - tone-generator
105+ - libusermetricsinput1
106+ - dconf-cli
107+ after: [desktop-ubuntu-app-platform]
108+
109+ environment:
110+ source: snap
111+ plugin: dump
112+

Subscribers

People subscribed via source and target branches

to all changes: