Page MenuHomeFreeBSD

devel/py-python-pcre: new port.
ClosedPublic

Authored by xmj on Jan 22 2015, 9:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 10 2024, 9:47 AM
Unknown Object (File)
Jan 16 2024, 6:30 PM
Unknown Object (File)
Dec 29 2023, 5:48 AM
Unknown Object (File)
Dec 25 2023, 4:21 PM
Unknown Object (File)
Dec 20 2023, 1:24 AM
Unknown Object (File)
Dec 10 2023, 4:14 AM
Unknown Object (File)
Nov 19 2023, 2:56 PM
Unknown Object (File)
Nov 19 2023, 2:56 PM
Subscribers
None

Details

Reviewers
swills
koobs
Summary

devel/py-python-pcre: new port.

A Python PCRE library

WWW: https://github.com/awahlig/python-pcre

PR: 196947
DiffRev: placeholder
Submitted by: Maxim Filimonov <che@bein.link>
Approved by: koobs (mentor)

Test Plan

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

xmj retitled this revision from to devel/py-python-pcre: new port..
xmj updated this object.
xmj edited the test plan for this revision. (Show Details)
xmj added reviewers: swills, koobs.
koobs requested changes to this revision.Jan 22 2015, 9:33 AM
koobs edited edge metadata.
koobs added inline comments.
devel/py-python-pcre/Makefile
10

Upstream description (setup.py) is

"Python PCRE bindings"

15

Is this also a LIB_DEPENDS, or only for headers/includes (BUILD_DEPENDS)

18

Upstream Trove Classifiers in setup.py is unclear as to whether this is 2 or 3, or 2/3 compatible. In the absence of these details, tests using Python 3.4 should confirm whether its 3 compatible. If not, set USES=python:2.7, unless it also requires the python2 symlink, in which case set USES=python:2

Bonus:

Submit upstream issue asking author to add Trove Classifiers

devel/py-python-pcre/files/patch-setup.py
9

See if you can use the following for the build_ext distutils command and the PYDISTUTILS_BUILDARGS / PYDISTUTILS_BUILD_TARGET python.mk variables:

  • --include-dirs (-I) list of directories to search for header files
  • --library-dirs (-L) directories to search for external C libraries
  • --libraries (-l) external C libraries to link with
  • --debug (-g) compile/link with debugging information
This revision now requires changes to proceed.Jan 22 2015, 9:33 AM
xmj edited edge metadata.

CRF: Comment, LIB_DEPENDS

swills edited edge metadata.

Approved

Holding off until upstream Issue 6 is fixed / closed.

devel/py-python-pcre/Makefile
18

Runs with both 2 and 3.

24

Note to self: replace with tagname once upstream issue [1] is resolved.
[1] https://github.com/awahlig/python-pcre/issues/6

devel/py-python-pcre/files/patch-setup.py
9

build_ext is run from within python, and neither adding --library-dirs nor --include-dirs changes things. (Actually, they make python setup.py error out...)

koobs edited edge metadata.
This revision is now accepted and ready to land.Jun 22 2015, 12:00 PM

Looks like we lost the files in this review. I've had my portstree resetted to scratch in between, so I'll recreate.