Hi,

Naddy reported a build failure on py-neovim that looks like this:

---8<---
===>  Building for py3-neovim-0.4.0
Download error on https://pypi.org/simple/pytest-runner/: [Errno -5] no address 
associated with name...
--->8---

setuptools tries to download this dependency if it's not installed, and
presumably that only showed under dpb, where there's a pf rule to block
network access.

Obvious fix: add devel/py-test-runner as a BUILD_DEPEND.

However, as far as I can see, pytest-runner isn't used. If you comment
out the dependency, the build works fine, and the tests also don't seem
to require it.

So this change removes the dependency. Also an unrelated patch churned
from `make update-patches` -- should have done that in the update
yesterday.

No bumps needed, I think.

OK?


Index: patches/patch-setup_py
===================================================================
RCS file: patches/patch-setup_py
diff -N patches/patch-setup_py
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-setup_py      21 Nov 2019 10:14:46 -0000
@@ -0,0 +1,19 @@
+$OpenBSD$
+
+pytest-runner isn't actually required:
+https://github.com/neovim/pynvim/issues/417
+
+Index: setup.py
+--- setup.py.orig
++++ setup.py
+@@ -8,9 +8,7 @@ install_requires = [
+     'msgpack>=0.5.0',
+ ]
+ 
+-setup_requires = [
+-    'pytest-runner'
+-]
++setup_requires = []
+ 
+ tests_require = [
+     'pytest>=3.4.0',
Index: patches/patch-test_conftest_py
===================================================================
RCS file: /cvs/ports/editors/py-neovim/patches/patch-test_conftest_py,v
retrieving revision 1.2
diff -u -p -r1.2 patch-test_conftest_py
--- patches/patch-test_conftest_py      17 Mar 2019 13:15:34 -0000      1.2
+++ patches/patch-test_conftest_py      21 Nov 2019 10:07:38 -0000
@@ -8,9 +8,9 @@ version.
 Index: test/conftest.py
 --- test/conftest.py.orig
 +++ test/conftest.py
-@@ -0,0 +1,67 @@
-+import json
-+import os
+@@ -1,5 +1,72 @@
+ import json
+ import os
 +import textwrap
 +
 +import pynvim
@@ -76,3 +76,8 @@ Index: test/conftest.py
 +        editor = pynvim.attach('socket', path=listen_address)
 +
 +    return editor
++import json
++import os
+ 
+ import pytest
+ 

-- 
Best Regards
Edd Barrett

http://www.theunixzoo.co.uk

Reply via email to