Merge lp:~nskaggs/ubuntu-calendar-app/revert-workaround-code into lp:ubuntu-calendar-app

Proposed by Nicholas Skaggs
Status: Merged
Approved by: Nicholas Skaggs
Approved revision: 301
Merged at revision: 303
Proposed branch: lp:~nskaggs/ubuntu-calendar-app/revert-workaround-code
Merge into: lp:ubuntu-calendar-app
Diff against target: 23 lines (+0/-7)
1 file modified
calendar.qml (+0/-7)
To merge this branch: bzr merge lp:~nskaggs/ubuntu-calendar-app/revert-workaround-code
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Kunal Parmar Approve
Renato Araujo Oliveira Filho Pending
Review via email: mp+222256@code.launchpad.net

Commit message

Revert accidentally merged code

Description of the change

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

This is likely to be rejected by jenkins because of the regression that's in trunk:

https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1326963

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

looks like its caused by unmet dependency for qtdeclarative5-ubuntu-syncmonitor0.1

The following packages have unmet dependencies:
 calendar-app : Depends: qtdeclarative5-ubuntu-syncmonitor0.1 but it is not installable
E: Unable to correct problems, you have held broken packages.

Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

looks good to me

review: Approve
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Yea, syncmonitor doesn't exist in trunk for trusty, so :-)

As expected, MismatchError: After 10.0 seconds test on NewEventEntryField.activeFocus failed: True != dbus.Boolean(False, variant_level=1).

Revision history for this message
David Planella (dpm) wrote :

I've now build qtdeclarative5-ubuntu-syncmonitor0.1 in the core apps PPA for trusty, so the unmet dependency failure should be fixed.

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'calendar.qml'
2--- calendar.qml 2014-06-05 12:08:18 +0000
3+++ calendar.qml 2014-06-05 20:02:21 +0000
4@@ -2,19 +2,12 @@
5 import Ubuntu.Components 0.1
6 import Ubuntu.Components.Popups 0.1
7 import Ubuntu.SyncMonitor 0.1
8-import QtQuick.Window 2.0
9-
10 import "dateExt.js" as DateExt
11
12 MainView {
13 id: mainView
14 useDeprecatedToolbar: false
15
16- // Work-around until this branch lands:
17- // https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/optIn-tabsDrawer/+merge/212496
18- property bool windowActive: typeof window != 'undefined'
19- onWindowActiveChanged: window.title = i18n.tr("Calendar")
20-
21 // Argument during startup
22 Arguments {
23 id: args;

Subscribers

People subscribed via source and target branches

to status/vote changes: