OpenERP buildout recipe

Code review comment for lp:~jbaudoux/anybox.recipe.openerp/20130908_relative_paths

  1. 20130908_relative_paths
  2. Merge into trunk
Revision history for this message
Jacques-Etienne Baudoux (jbaudoux) wrote :

As the openerp.cfg is generated by importing openerp.tools.config from buildout, and the path is generated by openerp with os.path.abspath(os.path.expanduser(os.path.expandvars(os.path.dirname(openerp.__file__))))
I don't see anyway to have relative path here.
And as having full path in openerp.cfg is not blocking, so I'm gonna leave it like this.

Nevertheless, for python, relative path is required for the python module loader if you deploy in chroot jail. So, I suggest you merge this MP.

Reply

« Back to merge proposal