Bug 242065
| Summary: | editors/py-pynvim: Update to 0.4.0 | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Thomas Merkel <tm> | ||||||
| Component: | Individual Port(s) | Assignee: | Tobias C. Berner <tcberner> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | joneum, meka, python, tcberner | ||||||
| Priority: | --- | Keywords: | needs-patch, needs-qa | ||||||
| Version: | Latest | ||||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| See Also: |
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=241767 https://github.com/neovim/pynvim/pull/418 |
||||||||
| Attachments: |
|
||||||||
Comment on attachment 209236 [details]
patch for ports to update py-pynvim to version 0.4.0
^Triage: Reporter is MAINTAINER, set attachment maintainer-approval flag (to +) to signify maintainer approval
Heya Thomas,
are you sure about this?
> -RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}msgpack>=0.5.0:devel/py-msgpack@${PY_FLAVOR} \
>+RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}msgpack05>=0.5.0:devel/py-msgpack05@${PY_FLAVOR} \
This is an error on this change:
[00:00:03] Warning: (editors/py-pynvim): Error: editors/py-pynvim depends on nonexistent origin 'devel/py-msgpack05' (moved to devel/py-msgpack); Please contact maintainer of the port to fix this.
Created attachment 209995 [details]
patch for ports to update py-pynvim to version 0.4.0 include fix py-msgpack
It's totally true that the last patch wasn't valid based on the requirement for py-msgpack. I've attached a new patch without these changes!
(In reply to Thomas Merkel from comment #3) Hi Thomas, with this new patch, there is another error: http://joneumbox.org/data/121i386-ports/2019-12-20_12h28m39s/logs/errors/py37-pynvim-0.4.0.log (In reply to Jochen Neumeister from comment #4) pytest-runner is unfortunately included as an unconditional setup_requires in the upstream code, when instead it should be conditional on setup.py invocation arguments including 'test' or similar. See bug 241767 and ports r519582 I've created a PR upstream: https://github.com/neovim/pynvim/pull/418 You can either apply the patch from that PR, or patch out pytest-runner from setup_requires (making sure to add it as a TEST_DEPENDS in the port) *** Bug 244763 has been marked as a duplicate of this bug. *** Committed. Thanks. A commit references this bug: Author: tcberner Date: Sat Mar 14 08:38:33 UTC 2020 New revision: 528402 URL: https://svnweb.freebsd.org/changeset/ports/528402 Log: editors/py-pynvim: update to 0.4.0 [1] https://github.com/neovim/pynvim/commit/f04853121de564ac03b981a984595ea33331ad3e PR: 242065 Submitted by: Thomas Merkel <tm@NetBSD.org> (maintainer), koobs [1] Changes: head/editors/py-pynvim/Makefile head/editors/py-pynvim/distinfo |
Created attachment 209236 [details] patch for ports to update py-pynvim to version 0.4.0 Upstream provided a new version of py-pynvim. A patch has been attached to upgrade the port to 0.4.0.