Merge lp:~robert-ancell/indicator-datetime/libecal-3.16 into lp:indicator-datetime/15.10

Proposed by Robert Ancell on 2015-06-02
Status: Rejected
Rejected by: Iain Lane on 2015-07-20
Proposed branch: lp:~robert-ancell/indicator-datetime/libecal-3.16
Merge into: lp:indicator-datetime/15.10
Diff against target: 63 lines (+5/-4)
5 files modified
CMakeLists.txt (+1/-1)
debian/control (+1/-1)
src/engine-eds.cpp (+1/-0)
tests/CMakeLists.txt (+1/-1)
tests/run-eds-test.sh (+1/-1)
To merge this branch: bzr merge lp:~robert-ancell/indicator-datetime/libecal-3.16
Reviewer Review Type Date Requested Status
Charles Kerr (community) 2015-06-02 Approve on 2015-06-02
PS Jenkins bot continuous-integration Needs Fixing on 2015-06-02
Review via email: mp+260789@code.launchpad.net

Description of the Change

When wily updates to evolution-data-server 3.16 there is a small API change required.

To post a comment you must log in.
Robert Ancell (robert-ancell) wrote :

Updated is tracked in bug 1460271.

Charles Kerr (charlesk) wrote :

Robert Ancell, patch LGTM, what is the timeline for landing libecal-3.16?

Feel free to silo this together with the ecal upgrade.

review: Approve
Robert Ancell (robert-ancell) wrote :

We don't have a timeline currently. Currently tracking down all the build-dependencies so we can do the migration. If no side-effects I would expect to land 3.16 as soon as we have resolved those.

Iain Lane (laney) wrote :
Download full text (3.8 KiB)

Been blatting bits into ppa:ci-train-ppa-service/landing-028

Testing that silo, I don't see any appointments in i-datetime any more. Running it with G_MESSAGES_DEBUG=all shows

laney@raleigh> pkill -f indicator-datetime-service; G_MESSAGES_DEBUG=all /usr/lib/x86_64-linux-gnu/indicator-datetime/indicator-datetime-service ~
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/timezone-file.cpp:90 Monitoring timezone file '/etc/timezone'
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/timezone-file.cpp:90 Monitoring timezone file '/etc/timezone'
(process:12781): Indicator-Datetime-DEBUG: PlannerMonth 0x1f3d5a0 setting calendar month range: [2015-06-01 00:00:00..2015-06-30 23:59:59]
(process:12781): Indicator-Datetime-DEBUG: rebuilding because the date range changed
(process:12781): Indicator-Datetime-DEBUG: 0x1f3e1c0 setting date range to [2015-06-17 00:00:00..2015-07-17 00:00:00]
(process:12781): Indicator-Datetime-DEBUG: rebuilding because the date range changed
(process:12781): Indicator-Datetime-DEBUG: 0x1f45010 setting date range to [2015-06-17 00:00:00..2015-07-17 00:00:00]
(process:12781): Indicator-Datetime-DEBUG: rebuilding because the date range changed
(process:12781): Indicator-Datetime-DEBUG: planner has 0 appointments in it
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/formatter.cpp:104 guint unity::indicator::datetime::{anonymous}::calculate_seconds_until_next_fifteen_minutes(GDateTime*) the next timestamp rebuild will be at 2015-06-18 17:15:00
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/formatter.cpp:104 guint unity::indicator::datetime::{anonymous}::calculate_seconds_until_next_fifteen_minutes(GDateTime*) the next timestamp rebuild will be at 2015-06-18 17:15:00
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/formatter.cpp:104 guint unity::indicator::datetime::{anonymous}::calculate_seconds_until_next_fifteen_minutes(GDateTime*) the next timestamp rebuild will be at 2015-06-18 17:15:00
(process:12781): Indicator-Datetime-DEBUG: /build/buildd/indicator-datetime-13.10.0+15.10.20150515/src/formatter.cpp:104 guint unity::indicator::datetime::{anonymous}::calculate_seconds_until_next_fifteen_minutes(GDateTime*) the next timestamp rebuild will be at 2015-06-18 17:15:00
(process:12781): Indicator-Datetime-DEBUG: bus acquired: com.canonical.indicator.datetime
(process:12781): Indicator-Datetime-DEBUG: from geoclue, setting timezone to 'Europe/London'
(process:12781): Indicator-Datetime-DEBUG: getting all appointments from [2015-06-01 00:00:00 ... 2015-06-30 23:59:59]
(process:12781): Indicator-Datetime-DEBUG: default_timezone is 0x1f6db78
(process:12781): Indicator-Datetime-DEBUG: RangePlanner 0x1f3cda0 got 0 appointments
(process:12781): Indicator-Datetime-DEBUG: time to delete task 0x1f6add0
(process:12781): Indicator-Datetime-DEBUG: gett...

Read more...

Iain Lane (laney) wrote :

OK, never mind, I'm an idiot who forgot to upload it. :)

Do have an extra commit to fix the tests though - want to pull lp:~laney/indicator-datetime/eds316-fix-tests into here?

413. By Robert Ancell on 2015-06-18

Fix tests

Iain Lane (laney) wrote :

I have proposed https://code.launchpad.net/~laney/indicator-datetime/eds-3.16/+merge/265287 which supersedes this (they refactored the tests a bit in trunk, it's otherwise the same)

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2014-08-19 04:03:38 +0000
3+++ CMakeLists.txt 2015-06-18 21:40:28 +0000
4@@ -36,7 +36,7 @@
5 glib-2.0>=2.36
6 gio-unix-2.0>=2.36
7 libical>=0.48
8- libecal-1.2>=3.5
9+ libecal-1.2>=3.16
10 libedataserver-1.2>=3.5
11 gstreamer-1.0>=1.2
12 libnotify>=0.7.6
13
14=== modified file 'debian/control'
15--- debian/control 2015-04-06 14:25:59 +0000
16+++ debian/control 2015-06-18 21:40:28 +0000
17@@ -9,7 +9,7 @@
18 libglib2.0-dev (>= 2.35.4),
19 libnotify-dev (>= 0.7.6),
20 libgstreamer1.0-dev,
21- libecal1.2-dev (>= 3.5),
22+ libecal1.2-dev (>= 3.16),
23 libical-dev (>= 1.0),
24 libedataserver1.2-dev (>= 3.5),
25 liburl-dispatcher1-dev,
26
27=== modified file 'src/engine-eds.cpp'
28--- src/engine-eds.cpp 2015-04-06 19:58:34 +0000
29+++ src/engine-eds.cpp 2015-06-18 21:40:28 +0000
30@@ -263,6 +263,7 @@
31 g_debug("%s connecting a client to source %s", G_STRFUNC, source_uid);
32 e_cal_client_connect(source,
33 source_type,
34+ -1,
35 self->m_cancellable,
36 on_client_connected,
37 gself);
38
39=== modified file 'tests/CMakeLists.txt'
40--- tests/CMakeLists.txt 2015-04-06 00:19:01 +0000
41+++ tests/CMakeLists.txt 2015-06-18 21:40:28 +0000
42@@ -75,7 +75,7 @@
43 find_program(GVFSD gvfsd PATHS /usr/lib/gvfs/)
44 OPTION(EVOLUTION_SOURCE_SERVICE_NAME "DBus name for source registry")
45 if(NOT EVOLUTION_SOURCE_SERVICE_NAME)
46- set(EVOLUTION_SOURCE_SERVICE_NAME "org.gnome.evolution.dataserver.Sources3")
47+ set(EVOLUTION_SOURCE_SERVICE_NAME "org.gnome.evolution.dataserver.Sources4")
48 endif()
49
50 function(add_eds_test_by_name name)
51
52=== modified file 'tests/run-eds-test.sh'
53--- tests/run-eds-test.sh 2015-04-06 20:09:16 +0000
54+++ tests/run-eds-test.sh 2015-06-18 21:40:28 +0000
55@@ -43,7 +43,7 @@
56
57 # run dbus-test-runner
58 $1 --keep-env --max-wait=90 \
59---task $2 --task-name $3 --wait-until-complete --wait-for=org.gnome.evolution.dataserver.Calendar4 \
60+--task $2 --task-name $3 --wait-until-complete --wait-for=org.gnome.evolution.dataserver.Calendar7 \
61 --task $4 --task-name "evolution" --wait-until-complete -r
62 #--task $6 --task-name "source-registry" --wait-for=org.gtk.vfs.Daemon -r \
63 #--task $7 --task-name "gvfsd" -r

Subscribers

People subscribed via source and target branches