Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 31 Aug 2021 01:35:50 +0000
From:      bugzilla-noreply@freebsd.org
To:        python@FreeBSD.org
Subject:   [Bug 257920] [NEW PORT] devel/py-pycapnp: Cython wrapping of the C++ Cap'n Proto library
Message-ID:  <bug-257920-21822-UKimADSMrj@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-257920-21822@https.bugs.freebsd.org/bugzilla/>
References:  <bug-257920-21822@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D257920

Kubilay Kocak <koobs@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[NEW PORT]                  |[NEW PORT]
                   |devel/py-pycapnp: Python    |devel/py-pycapnp: Cython
                   |bindings of the Cap'n Proto |wrapping of the C++ Cap'n
                   |library                     |Proto library
              Flags|maintainer-feedback+        |maintainer-feedback?(soboma
                   |                            |x@FreeBSD.org)
           Severity|Affects Some People         |Affects Only Me

--- Comment #5 from Kubilay Kocak <koobs@FreeBSD.org> ---
Review items:

- Match COMMENT to upstream titl,e modulo portlint needs. In this case: "Cy=
thon
wrapping of the C++ Cap'n Proto library"

- Package installs files into non Python version specific (shared) location=
s,
port needs to be made concurrent (installation) safe using
USE_PYTHON=3Dconcurrent [1]

- Upstream says it wants a C++14 compiler, probably worth using
USES=3Dcompiler:<lang> here. [2]

- Upstream mentions needing special compiler args for 32-bit, does this also
apply to us? [3]

- Using system (ports) LIB_DEPENDS is preferable, by default this uses the
bundled libpcap code. Use --force-system-libcapnp" added to
PYDISTUTILS_BUILDARGS [4]

- Add TEST_DEPENDS and (do-)test target when a package ships a test suite. =
In
this case TEST_DEPENDS=3Dpytest and do-test: cd ${WRKSRC} && ${PYTHON_CMD} =
-m
pytest -v -rs -o addopts=3D should get it to run (please confirm QA)

[1] "console_scripts": ["capnpc-cython =3D capnp._gen:main"]
[2]  C++14 supported compiler=20
[3] 32-bit Linux requires that capnproto be compiled with -fPIC. This is
usually set correctly unless you are compiling canproto yourself. This is a=
lso
called -DCMAKE_POSITION_INDEPENDENT_CODE=3D1 for cmake.
[4] setup.py: force_system_libcapnp =3D "--force-system-libcapnp" in sys.ar=
gv

--=20
You are receiving this mail because:
You are on the CC list for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-257920-21822-UKimADSMrj>