Created attachment 259578 [details] Patch textproc/py-docstring_parser: update to 0.16 While here, switch to USE_PYTHON=pep517. Changelog: https://github.com/rr-/docstring_parser/blob/0.16/CHANGELOG.md QA: poudriere testport: OK ({142,134}{amd64,i386})
(In reply to Hiroki Tagato from comment #0) Hi, thanks for your patch. It looks good for me and I've only two hints/suggestions: - If you like, you can simplify the Makefile further by using "USE_PYTHON=pytest" and remove the "TEST_DEPENDS" and the "do-test" target. - One the patch lands into the ports tree it will break the port misc/py-instructor you maintain due its version requirements in its RUN_DEPENDS for py-docstring_parser. If you already have a patch for py-instructor and intend to commit it as soon as py-docstring_parser has been updated, it would make sense to do both in one go. If this is the case, you are welcome to commit the patch for py-docstring_parser yourself.
(In reply to Kai Knoblich from comment #1) Hi Kai, Thanks for the helpful suggestions. It does make sense to use USE_PYTHON=pytest for a simpler Makefile. Regarding the second comment, I have prepared a patch for py-instructor and would like to update the two ports at the same time. I will soon commit the changes. Thanks for your approval.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=3fbc3a4208ba5e20987d7c9f22f82a3acf9b141e commit 3fbc3a4208ba5e20987d7c9f22f82a3acf9b141e Author: Hiroki Tagato <tagattie@FreeBSD.org> AuthorDate: 2025-04-16 12:05:19 +0000 Commit: Hiroki Tagato <tagattie@FreeBSD.org> CommitDate: 2025-04-16 12:13:59 +0000 textproc/py-docstring_parser: update to 0.16 While here: - switch to USE_PYTHON=pep517 - switch to USE_PYTHON=pytest and remove TEST_DEPENDS and do-test target Changelog: https://github.com/rr-/docstring_parser/blob/0.16/CHANGELOG.md PR: 286110 Reviewed by: kai Approved by: kai textproc/py-docstring_parser/Makefile | 10 +++------- textproc/py-docstring_parser/distinfo | 6 +++--- 2 files changed, 6 insertions(+), 10 deletions(-)
Committed, thanks!