<div dir="ltr">Hi Martin,<div><br></div><div>This is great! I've gone through the commits, and I think it looks good. It is a pity that the default python `easy_install` does not provide a complete uninstall feature, and I understand that you are reluctant to have an extra dependency on `pip` just to manage the package.</div><div><br></div><div>As for the README.rst, it wasn't much there from the beginning, and it makes sense to have it part of the main vici README.md. Essentially, only the vici data structures need an explanation of how they map to python `dict`, `list` etc.</div><div><br></div><div><br></div><div>Cheers,</div><div>Björn</div><div><br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-02-25 17:08 GMT+01:00 Martin Willi <span dir="ltr"><<a href="mailto:martin@strongswan.org" target="_blank">martin@strongswan.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Björn,<br>
<span class=""><br>
> and at the same time elaborate the best options for an integration.<br>
<br>
</span>I've integrated your python library into the strongSwan code base by<br>
migrating your commits to the vici-python branch [1]. On top of that,<br>
I've tried to integrate it into our build system, using a very similar<br>
approach as with the ruby gem. Your README.rst is not yet part of it,<br>
probably we should integrate that to the vici README.md [2].<br>
<br>
The branch introduces a --enable-python-eggs option, and an optional<br>
--with-pythoneggdir= path option. If vici is built, a python egg gets<br>
created with the make target and installed with the make install target<br>
using easy_install.<br>
<br>
This approach allows us to install the python library along with<br>
strongSwan. Version numbering is analog to strongSwan.<br>
<br>
I've not planned to push that extension to PyPI or the like; but of<br>
course you are free to continue doing so.<br>
<br>
I'm really no python expert, so any comment is welcome :-).<br>
<br>
Best regards<br>
Martin<br>
<br>
[1]<a href="http://git.strongswan.org/?p=strongswan.git;a=shortlog;h=refs/heads/vici-python" target="_blank">http://git.strongswan.org/?p=strongswan.git;a=shortlog;h=refs/heads/vici-python</a><br>
[2]<a href="https://github.com/strongswan/strongswan/blob/master/src/libcharon/plugins/vici/README.md" target="_blank">https://github.com/strongswan/strongswan/blob/master/src/libcharon/plugins/vici/README.md</a><br>
<br>
</blockquote></div><br></div></div></div></div>