[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-21 Thread noreply
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support/+merge/259230 --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-21 Thread Alan Pope 
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support/+merge/25923

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-19 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:140 http://91.189.93.70:8080/job/ubuntu-docviewer-app-ci/221/ Executed test runs: SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2770 deb: http://91.189.93.70:8080/job/generic-mediumtests-utop

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Stefano Verzegnassi
Can we say that AppArmor rules have not been updated when installing the new package? If so, I remove the debug lines from the code and top-approve it after Jenkins' approval. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support/+merge/259230 Your team Ubun

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope 
Review: Approve It works! I can see and open documents from both the SD card and the internet storage now. phablet@ubuntu-phablet:~/.cache/upstart$ tail -f application-click-com.ubuntu.docviewer_docviewer_0.3.latest.log "/var/lib/apport" "/etc/default/apport" "/var/lib/urfkill" "/var/lib/misc

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Ubuntu Phone Apps Jenkins Bot
Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:138 http://91.189.93.70:8080/job/ubuntu-docviewer-app-ci/219/ Executed test runs: FAILURE: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2762/console FAILURE: http://91.189.93.70:8080/job/ubuntu-doc

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope 
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app has been updated. Commit Message changed to: SD Card support For more details, see: https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support/+merg

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Ubuntu Phone Apps Jenkins Bot
Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:138 No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself): https://code.launchpad.net/~verzegnas

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Stefano Verzegnassi
I checked out this last night, I downloaded this branch and run it on a vivid-proposed Nexus 5 (hammerhead). The only denial I got is related to the loading of the QML file. I've pused a new revision, with some debug output. I got the list of all the mounted volumes with no problem (a list of abo

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-16 Thread Stefano Verzegnassi
Those denials are caused by the QStorageInfo class, which is used to get the list of the mounted media on the device, so that we can build the full path to the Documents folder in an SD card (see lines 3964-3986 of the diff). The issue happens before any file get parsed and added to the documents

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-15 Thread Ubuntu Phone Apps Jenkins Bot
Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:137 No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself): https://code.launchpad.net/~verzegnas

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-15 Thread Alan Pope 
(proposing so I can comment) Tested this on bq device running vivid. Put spread documents between ~/Documents and /media/phablet/2541-1C26/Documents/ $ ls /media/*/*/[Dd][Oo][Cc][Uu][Mm][Ee][Nn][Tt][Ss]/ Manual_Aquaris_E4.1.5_ubuntu_EN.pdf Manual_Aquaris_E4.5_ubuntu_EN.pdf Manual_Aquaris_E4 (cop