Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to newest version 0.12.4 WKHTMLtoPDF. #1

Open
wants to merge 2 commits into
base: patch-2
Choose a base branch
from

Conversation

chris001
Copy link
Owner

Install the newest version 0.12.4 (Nov 2016) of WKHTMLtoPDF.
Fixes many bugs over the previously used 0.12.1 (released 2014) which is obsolete for years now.
https://github.com/wkhtmltopdf/wkhtmltopdf/releases

* Script can now run again and again without corrupting the installation.
* Script will install to `/home/odoo/` by default, instead of `/odoo/`, this is best practice, because `/home/odoo` will be backed up on typical servers, and `/odoo` will be restricted or refused access on typical servers.
* Script will halt on major errors to more easily locate any errors.
* Cleaned up, combined many `apt-get` and `pip` calls into fewer calls.
* All output goes to `~/install_log` for easier posting to github for help.
* `./install_log` is cleared at the start of each run.
* All work (downloads, building the `conf` file and service file) takes place in home dir.
* Fixed bug, was creating `addons` dir with ownership of `root`.
* Add configurable option to add user `odoo` to group `sudo` or not.
* Show proof that service is running at the end of the run.
* More to do, (put each step into its own function, and create reverse of each function for uninstaller) but this is a great start.
Install the newest version 0.12.4 (Nov 2016) of WKHTMLtoPDF.
Fixes many bugs over the previously used 0.12.1 (released 2014) which is obsolete for years now.
https://github.com/wkhtmltopdf/wkhtmltopdf/releases
@jpaulin
Copy link

jpaulin commented Aug 27, 2018

@chris001 Is this 0.12.4 actually already within the script? I took a look at some source and found the version number particularly familiar looking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants