Ubuntu Document Viewer App

Merge lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/workaround-fix-1432412 into lp://staging/ubuntu-docviewer-app/trunk

workaround-fix-1432412
Merge into trunk

Proposed by Stefano Verzegnassi on 2015-03-25

Status:

Merged

Approved by:

Stefano Verzegnassi on 2015-03-26

Approved revision:

107

Merged at revision:

107

Proposed branch:

lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/workaround-fix-1432412

Merge into:

lp://staging/ubuntu-docviewer-app/trunk

Diff against target:

48 lines (+10/-4)

2 files modified

po/com.ubuntu.docviewer.pot (+3/-3)
src/plugin/poppler-qml-plugin/pdfdocument.cpp (+7/-1)

To merge this branch:

bzr merge lp://staging/~verzegnassi-stefano/ubuntu-docviewer-app/workaround-fix-1432412

Medium

Triaged

Link a bug report

Reviewer	Review Type	Date Requested	Status
Alan Pope 🍺🐧🐱 🦄 (community)		2015-03-25	Approve on 2015-03-26
Ubuntu Phone Apps Jenkins Bot	continuous-integration		Approve on 2015-03-25
Review via email: mp+254117@code.staging.launchpad.net

Commit message

Workaround for multi-core support on ARM SoCs

Description of the change

Workaround for multi-core support on ARM SoCs

Revision history for this message

Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote on 2015-03-25:

PASSED: Continuous integration, rev:107
http://91.189.93.70:8080/job/ubuntu-docviewer-app-ci/185/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2425
        deb: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2425/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-docviewer-app-utopic-amd64-ci/82
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-docviewer-app-vivid-amd64-ci/88

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

review: Approve (continuous-integration)

Revision history for this message

Alan Pope 🍺🐧🐱 🦄 (popey) wrote on 2015-03-26:

Tested on bq device and works fine. We may need to re-visit on meizu device (which runs vivid, and a newer kernel) because it seems to never spin more than 1 core even under heavy load in this app.

For now, I say approve as bq device is only one shipping, and we can analyse meizu/vivid issue separately.

review: Approve

Revision history for this message

Stefano Verzegnassi (verzegnassi-stefano) wrote on 2015-03-26:

Ok, I top-approve it then.