[new port] : textproc/py-diff-match-patch

[rene]

Summary

The Diff Match and Patch libraries offer robust algorithms to perform
the operations required for synchronizing plain text.

Dependency for D1234

Test Plan

Completed poudriere testbuilds on 8,9,10 amd64/i386

[koobs]

textproc/py-diff-match-patch/Makefile

6

:|

Might we want to maybe make this 0.0.0.{datestamp} to protect from having to add/bump PORTEPOCH in the future?

18

test_suite is set in setup.py which points to the module/class that implements the test. no tests_require means no additional dependencies are needed.

Can remove this comment when regression-test: target is implemented

20

shebangfix is likely not necessary (but certainly check).

setuptools normally 'converts' these to the python version that was used to build/install them if they are declared correctly (as console_scripts)

See:

https://pythonhosted.org/setuptools/setuptools.html#automatic-script-creation

[rene]

Changed PORTVERSION to start with 0.0.0

[rene]

textproc/py-diff-match-patch/Makefile

18

There is a key test_suite in setup() in setup.py, can that somehow be invoked?

20

Somehow shebangfix is needed here, just look at the resulting files in the stage directory when not using it.

[koobs]

Diffs don't appear to be displaying correctly (or at all) in the reviews, so I cant accept what I can't see :)

I think it may be an ongoing phabric issue cc @eadler

textproc/py-diff-match-patch/Makefile

24

The canonical test invocation I use is:

regression-test: build
<tab>@cd ${WRKSRC} && ${PYTHON_CMD} ${PYDISTUTILS_SETUP} test

That *should* work, and if not, should be patched so it does and fixes sent upstream

[rene]

Added regression-test target, which works fine.

[rene]

In case phabricator does not work, a full svn diff is available at https://people.freebsd.org/~rene/py-diff-match-patch.diff

[koobs]

Looks good @rene!