[Bug 257131] [NEW PORT] devel/py-cadquery-pywrap: C++ binding generator for Python

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=257131

Kubilay Kocak <koobs@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|needs-patch                 |
              Flags|maintainer-feedback+        |maintainer-feedback?(ports@
                   |                            |nicandneal.net)

--- Comment #6 from Kubilay Kocak <koobs@FreeBSD.org> ---
(In reply to Neal Nelson from comment #5)

Thanks Neal

  - Upstream ships a LICENSE file, add LICENSE_FILE

  - If jinja2 is no longer a requirement, should it be removed from RUN_DEPENDS
?

  - Use %%VARIABLE%% substitution in post-patch, or use SUB_FILES / SUBLIST

  - There's a strong preference for not hardcoding compiler versions. Does
pywrap support only LLVM 10 or some other version set? If it's something like
>=X, use something like the following: 

    https://cgit.freebsd.org/ports/tree/devel/libclc/Makefile

  - setup.py declares PyPI clang package as a dependency, whats the nature of
this dependency or why is it not included or necessary here? My assumption
would be that `import clang` would fail if its missing. Do we need to port it?

  - Packages that install files into non-versioned (shared) locations need to
be made concurrent-installation- safe (using USE_PYTHON=concurrent). This
package does: entry_points={'console_scripts':

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