Merge lp:~camptocamp/ocb-addons/ocb-addons-fix-report_webkit-descriptor-20130211 into lp:ocb-addons

Proposed by Alexandre Fayolle - camptocamp
Status: Merged
Merged at revision: 8696
Proposed branch: lp:~camptocamp/ocb-addons/ocb-addons-fix-report_webkit-descriptor-20130211
Merge into: lp:ocb-addons
Diff against target: 15 lines (+3/-2)
1 file modified
report_webkit/__openerp__.py (+3/-2)
To merge this branch: bzr merge lp:~camptocamp/ocb-addons/ocb-addons-fix-report_webkit-descriptor-20130211
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) Approve
Alexandre Fayolle - camptocamp spellchecking Approve
Review via email: mp+148402@code.launchpad.net

Description of the change

[FIX] report_webkit - change description for new webkit lib path parameter

To post a comment you must log in.
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM

review: Approve (spellchecking)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

In my original proposal I suggested to use the bug tracker as the glue between the official branches and the upstream, because I want to track that all code submitted to backports is also submitted to upstream. It is arguably overkill in this case to create a bug report, but is there an alternative procedure that we can standardize on for cases such as this one?

IMHO this is blocking for the merge, just to keep things organized.

review: Needs Information
Revision history for this message
Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c) wrote :

Well, I must say I agree with you Stefan : it's a bit overkill. But well, if this is the way to follow the "divergence" with Official branch, I think we should report a bug for this.

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

Here is the bug report, it was just missing in the MP.

https://bugs.launchpad.net/openobject-addons/+bug/1113418

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks for the info!

review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Oh, for even better tracking the bug should be set to also affect project ocb-addons, then target to series. I did it in this case because I already merged the fix just now.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'report_webkit/__openerp__.py'
2--- report_webkit/__openerp__.py 2012-11-29 22:26:45 +0000
3+++ report_webkit/__openerp__.py 2013-02-14 10:20:28 +0000
4@@ -62,8 +62,9 @@
5 PDF. Version 0.9.9 or later is necessary, and can be found at
6 http://code.google.com/p/wkhtmltopdf/ for Linux, Mac OS X (i386) and Windows (32bits).
7
8-After installing the library on the OpenERP Server machine, you need to set the
9-path to the ``wkthtmltopdf`` executable file on each Company.
10+After installing the library on the OpenERP Server machine, you need to set
11+the path to the ``wkthtmltopdf`` executable file in a system parameter named ``webkit_path``
12+in Settings -> Customization -> Low Level Object -> System Parameters
13
14 If you are experiencing missing header/footer problems on Linux, be sure to
15 install a 'static' version of the library. The default ``wkhtmltopdf`` on