Code review comment for lp:~camptocamp/openobject-addons/7.0-fix-report_webkit-descriptor-20130211

Niels Huylebroeck (red15) wrote :

Worth noting it also checks the path environment variable to see if it can find the wkhtmltopdf binary.

On that note I would like to point out if you use the openerp.init script in the debian sub-dir of the server that the path does NOT include things like /usr/local/bin (where we usually like to install custom software on a production machine)

review: Approve

« Back to merge proposal