Code review comment for lp:ubuntu-docviewer-app

Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

I had a play with this on my amd64 wily laptop.
The ultimate goal being to have a fully-enclosed click package containing the necessary libreoffice binaries, rather than debian packages. So rather than install the debs from the ppa as Bjoern suggested, I grabbed the tarball directly from libreoffice.org

Followed these instructions:- https://wiki.documentfoundation.org/Installing_in_parallel/Linux
Steps 1-3 and 7 only, omitting the optional parts. I ended up with an unpackaged install of Libreoffice 5.x in ~/libreoffice/building/LibreOffice_5.0.0.1_Linux_x86-64_deb/DEBS/install so I edited line 71 in the attached code:-

std::unique_ptr<lok::Office> pOffice(lok::lok_cpp_init("/usr/lib/libreoffice/program/"));

To be:-

std::unique_ptr<lok::Office> pOffice(lok::lok_cpp_init("/home/alan/libreoffice/building/LibreOffice_5.0.0.1_Linux_x86-64_deb/DEBS/install/opt/libreoffice5.0/program/"));

Just to prove that this works with a libreoffice install which isn't installed in the usual places expected with a debian install.

This also means you don't need to replace the 4.x libreoffice shipped in ubuntu with the pre-release from a ppa, just for testing this.

« Back to merge proposal