Summary: | [NEW PORT] security/py-vici: Native Python interface for strongSwan's VICI protocol | ||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Dries Michiels <driesm> | ||||||||||||||||||||
Component: | Individual Port(s) | Assignee: | Mateusz Piotrowski <0mp> | ||||||||||||||||||||
Status: | Closed FIXED | ||||||||||||||||||||||
Severity: | Affects Only Me | CC: | 0mp, koobs, python, strongswan | ||||||||||||||||||||
Priority: | --- | Keywords: | feature, needs-qa | ||||||||||||||||||||
Version: | Latest | ||||||||||||||||||||||
Hardware: | Any | ||||||||||||||||||||||
OS: | Any | ||||||||||||||||||||||
Bug Depends on: | |||||||||||||||||||||||
Bug Blocks: | 243254 | ||||||||||||||||||||||
Attachments: |
|
Description
Dries Michiels
2020-01-11 15:17:39 UTC
Created attachment 210623 [details]
pyvici.diff
Attach the patch for real now ..
Created attachment 210624 [details]
pyciviv2.diff
Add py-vici under security/Makefile as new port.
Created attachment 210625 [details]
pyvici3.diff
Leave out other diffs I had locally.
Created attachment 210630 [details]
pyviciv4.diff
- Add tests
- Use GH to fetch source as PyPi does not include the test files
I created a PR to support running tests from setuptools with
@${CD} ${WRKSRC} && ${PYTHON_CMD} ${PYDISTUTILS_SETUP} test
and to update their supported python versions in setup.py.
Tests seem to run fine in all versions (2.7, 3.5+).
Created attachment 210631 [details]
pyvici5.diff
Thanks for the work thus far on the port. I tested the build quickly and seems good. Not sure if we should also add the 'net-vpn' category. strongSwan is listed under security and net-vpn. Oh yes, good idea. Will update the patch in a few hours with upstream fixes. Created attachment 210740 [details]
pyvici6.diff
Updated with upstream changes.
- Use PyPi for master site
- Use tox for tests
This is about as good as it gets :-)
All supported flavors build and run the tests fine.
portlint -ac = looks fine.
Created attachment 210899 [details] pyvici7.diff Thanks to strongswan@Nanoteq.com for PM me to catch some spaces where there should be tabs with the capital verison of portlint -AC. Created attachment 210909 [details]
Portlint results from py-vici port
I tested the py-vici port with portlint in conjunction with the strongSwan port.
Here is the portlint results.
Created attachment 210910 [details]
Poudriere testport results
This is the poudriere testport build results for py-vici.
Everything looks good to me on this port.
Take it! A commit references this bug: Author: 0mp Date: Fri Jan 31 16:26:27 UTC 2020 New revision: 524726 URL: https://svnweb.freebsd.org/changeset/ports/524726 Log: New port: security/py-vici: Native Python interface for strongSwan's VICI protocol The strongSwan VICI protocol allows external applications to monitor, configure and control the IKE daemon charon. This Python package provides a native client side implementation of the VICI protocol, well suited to script automated tasks in a reliable way. WWW: https://pypi.org/project/vici/ Event: Brussels DevSummit 2020 PR: 243270 Submitted by: Dries Michiels <driesm.michiels@gmail.com> Changes: head/security/py-vici/ head/security/py-vici/Makefile head/security/py-vici/distinfo head/security/py-vici/pkg-descr A commit references this bug: Author: 0mp Date: Fri Jan 31 16:28:52 UTC 2020 New revision: 524728 URL: https://svnweb.freebsd.org/changeset/ports/524728 Log: Hook security/py-vici to the build PR: 243270 Submitted by: Dries Michiels <driesm.michiels@gmail.com> Changes: head/security/Makefile Thanks! |