When instanciating Session, it's not clear to me what is expected as root path. Seems only used to find a VERSION.txt file that I can't find anyway.
| "session = Session(%s, %r)" % (
|| self._relativitize(self.config_path),
|| self.jailroot_buildout_dir or self.buildout_dir),
When deploying from /root into /root/jailroot, do you expect jailroot-buildout-dir to be '/' which is the root from inside the jail, or '/root/jailroot' which is the path of the jail.
In the later case, that path was already deducted in the method _relativitize in server.py without having to define a new variable.
At the moment, your changes do not work with my current setup. If you can clarify your approach, I'm willing to refactor your MP to make it work with my setup against latest 1.8 serie. I'd be happy to finalize this branch and get it merged.
Hello Stefan,
I tried your latest branch lp:~therp-nl/anybox.recipe.openerp/8.2-jbaudoux-relative_paths_explicit_jailroot_support to test the jailroot. buildout- dir" option.
It's not clear for me what you expect in the new "jailroot-
When instanciating Session, it's not clear to me what is expected as root path. Seems only used to find a VERSION.txt file that I can't find anyway. ize(self. config_ path), buildout_ dir or self.buildout_dir),
| "session = Session(%s, %r)" % (
|| self._relativit
|| self.jailroot_
When deploying from /root into /root/jailroot, do you expect jailroot- buildout- dir to be '/' which is the root from inside the jail, or '/root/jailroot' which is the path of the jail.
In the later case, that path was already deducted in the method _relativitize in server.py without having to define a new variable.
At the moment, your changes do not work with my current setup. If you can clarify your approach, I'm willing to refactor your MP to make it work with my setup against latest 1.8 serie. I'd be happy to finalize this branch and get it merged.