Page MenuHomeFreeBSD

[NEW PORT] finance/py-ofxparse: OFX file format parser for Python
AbandonedPublic

Authored by vishwin on Jun 13 2019, 7:35 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 26 2024, 3:26 AM
Unknown Object (File)
Dec 21 2023, 11:07 PM
Unknown Object (File)
Dec 9 2023, 5:08 PM
Unknown Object (File)
Nov 9 2023, 12:42 PM
Unknown Object (File)
Nov 8 2023, 6:10 PM
Unknown Object (File)
Nov 8 2023, 9:12 AM
Unknown Object (File)
Nov 8 2023, 6:15 AM
Unknown Object (File)
Nov 7 2023, 12:46 PM
Subscribers

Details

Reviewers
koobs
Group Reviewers
Python
Summary

ofxparse is a parser for Open Financial Exchange (.ofx) format files, written in Python.

PR: 242855
Sponsored by: Wahgwan Industry Limited

Test Plan
  • testport: OK
  • portlint: OK

Tests included.

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 24836
Build 23588: arc lint + arc unit

Event Timeline

Is there a PR to go with this? If so please add it to the revision summary (PR: XXXX)

If not, please create one, adding this review url to the bugs URL field. Without a PR, reviews can 'fall through the cracks'

finance/py-ofxparse/Makefile
25

This isn't necessary when specifying do-test as the target

koobs requested changes to this revision.Jun 13 2019, 11:36 AM

Could you split the 'testport' results in the TEST PLAN section into explicit py27, py3x poudriere tests. Many packages package differently when built with different versions

Also, does maketest: Pass? Add the summary output to the TEST PLAN sectio0n too

finance/py-ofxparse/Makefile
28

The package defines test_suite in setup.py which enables the setuptools 'test' command.

When packages define this, use ${PYTHON_CMD} ${PYDISTUTILS_SETUP} test unless there is a compelling temporary reason not to (say for example, that invocation method is currently broken)

Failing that method working, the next best method is:

${PYTHON_CMD} -m <module> where target could be nose or pytest et al.

Many packages depend on, or are limited to specific versions of test dependencies, and doing the above, allows you to use the 'package name and version' style TEST_DEPENDS declaration, which the 'binary name' method doesn't allow.

So when changing the do-test invocation, switch the TEST_DEPENDS on nose from the binary to packagename style

This revision now requires changes to proceed.Jun 13 2019, 11:36 AM

Minor formatting nit, the canonical and preferred format for a new port is:

[NEW PORT] category/portname: COMMENT

<pkg-descr>

Feel free to re-add me as reviewer after updating

use setuptools test_suite

vishwin added inline comments.
finance/py-ofxparse/Makefile
28

The documentation said to run nose directly, but it turns out test_suite works too. Switched.

vishwin retitled this revision from New port: finance/py-ofxparse OFX file format parser for Python to [NEW PORT] finance/py-ofxparse OFX file format parser for Python.Aug 5 2019, 3:27 AM
vishwin set the repository for this revision to rP FreeBSD ports repository.
vishwin marked an inline comment as done.

Hook into INDEX

vishwin retitled this revision from [NEW PORT] finance/py-ofxparse OFX file format parser for Python to [NEW PORT] finance/py-ofxparse: OFX file format parser for Python.Dec 24 2019, 10:43 AM
vishwin edited the summary of this revision. (Show Details)
vishwin set the repository for this revision to rP FreeBSD ports repository.