Merge lp:~dpm/reminders-app/fix-1289347-icon-deps into lp:reminders-app

[David Planella]

Add dependency to the Suru icons package to get the mobile icons installed on the desktop

Add dependency to the Suru icons package to get the mobile icons installed on the desktop

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:120
http://91.189.93.70:8080/job/reminders-app-ci/296/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-trusty/2467
    SUCCESS: http://91.189.93.70:8080/job/reminders-app-saucy-amd64-ci/296
    SUCCESS: http://91.189.93.70:8080/job/reminders-app-trusty-amd64-ci/296

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/reminders-app-ci/296/rebuild

review: Approve (continuous-integration)

[Daniel Holbach (dholbach)]

Looks good and is what weather and music app are doing as well.

review: Approve

[Alan Pope 🍺🐧🐱 🦄 (popey)]

I tried to build this on my desktop and it failed:-

E: reminders-app source: not-binnmuable-all-depends-any reminders-app-autopilot -> reminders-app
E: reminders-app: missing-dependency-on-libc needed by usr/bin/reminders

[Daniel Holbach (dholbach)]

These lintian errors are unrelated. Might be worth opening another bug for it. I'll take a look at the problem in the meantime.

[Daniel Holbach (dholbach)]

The lintian errors also don't block the build. It still passes.

[Daniel Holbach (dholbach)]

Pushed my fixes to lp:~dholbach/reminders-app/some-lintian-fixes and will open a separate MP for it.

=== modified file 'debian/control'
--- debian/control	2014-05-06 18:40:53 +0000
+++ debian/control	2014-05-06 21:25:49 +0000
@@ -24,7 +24,8 @@
          qtdeclarative5-ubuntu-ui-toolkit-plugin | qt-components-ubuntu,
          qtdeclarative5-localstorage-plugin,
          qtdeclarative5-qtquick2-plugin,
-         signon-plugin-oauth2 (>= 0.19-0~57~)
+         signon-plugin-oauth2 (>= 0.19-0~57~),
+         suru-icon-theme | ubuntu-mobile-icons,
 Description: Reminders application
  Note taking application powered by Evernote