On 2018/08/09 13:09, Elias M. Mariani wrote:
> 3 - devel/py-test-forked 0.2 NEW
> https://marc.info/?l=openbsd-ports&m=153222412422291
> ++++++++++++++++++++++++
This one needs
TEST_DEPENDS = ${RUN_DEPENDS}
Otherwise OK sthen@ for someone to commit.
> 4 - pytest suite update:
> This includes:
> - devel/py-py to 1.5.4 UPDATE
> - devel/py-pluggy 0.7.1 UPDATE
> - devel/py-hypothesis to 3.66.30 UPDATE
> - devel/py-test to 3.7.1 UPDATE
> - devel/py-test-httpbin to 0.3.0 UPDATE
> - devel/py-test-localserver to 0.4.1 UPDATE
> - devel/py-test-mock to 1.10.0 UPDATE
> - devel/py-test-runner to 4.2 UPDATE
> - devel/py-test-xdist to 1.22.5 UPDATE
> This update depends on devel/py-pathlib2 (2) and devel/py-test-forked (3)
> https://marc.info/?l=openbsd-ports&m=153369310008767
> ++++++++++++++++++++++++
With devel/py-pathlib2 already committed we just need
devel/py-test-forked to unblock the above. Diff reads OK and your
testing seems good but I or anyone else who commits should do a bit of
their own testing too.
> 5.1 - devel/spyder/py-spyder-kernels NEW
> 5.2 - devel/spyder/spyder NEW
> https://marc.info/?l=openbsd-ports&m=153298181816263
In devel/spyder/spyder/patches/patch-spyder___init___py you have
+DOCPATH = '${LOCALBASE}/share/doc/spyder/'
- as it's referring to "a file from this port", it should be ${TRUEPREFIX}
In devel/spyder/Makefile the ,python3 versions should be listed too.
These also need
TEST_DEPENDS = ${RUN_DEPENDS}
(if you try 'make test' and don't have deps installed, it tries to fetch
them from the network, which fails if you're using the recommended setup
with PORTS_PRIVSEP=yes and a PF rule to block _pbuild from making
connections e.g. "block return quick proto {tcp udp} user _pbuild").
The test for py-spyder-kernels needs pandas, is that ready yet? If not,
then maybe set NO_TEST with a comment next to it like "# disabled until
py-pandas is imported"?
