Ubuntu Document Viewer App

Merge lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/get-click-deps into lp://staging/ubuntu-docviewer-app

  1. get-click-deps
  2. Merge into lo-viewer
Proposed by Stefano Verzegnassi
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 🦄
Approved revision: 179
Merged at revision: 192
Proposed branch: lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/get-click-deps
Merge into: lp://staging/ubuntu-docviewer-app
Diff against target: 697 lines (+638/-6)
6 files modified
CMakeLists.txt (+25/-0)
docviewer-libs.json (+94/-0)
docviewer.apparmor (+1/-0)
get-click-deps (+451/-0)
src/plugin/libreofficetoolkit-qml-plugin/config.h (+65/-5)
src/plugin/libreofficetoolkit-qml-plugin/lodocument.cpp (+2/-1)
To merge this branch: bzr merge lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/get-click-deps
Reviewer Review Type Date Requested Status
Alan Pope 🍺🐧🐱 🦄 (community) Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+275848@code.staging.launchpad.net

Commit message

Automatically fetch dependencies for LibreOffice viewer and put them inside the .click package.

Description of the change

Automatically fetch dependencies for LibreOffice viewer and put them inside the .click package.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

PASSED: Continuous integration, rev:179
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-ci/51/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-vivid-amd64-ci/51

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-docviewer-app-reboot-ci/51/rebuild

review: Approve (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

I'll re-review this once LibreOffice 5.0.3 has built in the community ppa and I have tested it.

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

Tested with latest libreoffice 5.0.3. Works brilliantly, thanks Stefano!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches