Too much dependencies for dev-python/pyowm #238

New Issue

zentoo · 2021-10-04T13:50:29+02:00

zentoo commented

2021-10-04 13:50:29 +02:00

dev-python/pyowm bring 42 ebuild dependencies and most of are BDEPEND ones but I'm almost sure that these dependencies are only for documentation generation (sphinx). So it could good to add a "doc" use flag that permit to unselect theses extra dependencies.

Upstream github make a distinction between requirements and dev requirements where these BDEPEND seems to come from.

I'm using openweathermap that need dev-python/pyowm and without openweathermap use flag on homeassistant I got only two python package in deps directory: pyowm and geojson.

dev-python/pyowm bring 42 ebuild dependencies and most of are BDEPEND ones but I'm almost sure that these dependencies are only for documentation generation (sphinx). So it could good to add a "doc" use flag that permit to unselect theses extra dependencies. Upstream github make a distinction between requirements and dev requirements where these BDEPEND seems to come from. I'm using openweathermap that need dev-python/pyowm and without openweathermap use flag on homeassistant I got only two python package in deps directory: pyowm and geojson.

onkelbeh self-assigned this 2021-10-05 07:03:25 +02:00

onkelbeh added the

Cleanup

label 2021-10-05 07:14:55 +02:00

onkelbeh commented

2021-10-05 07:18:51 +02:00

Will take some time, need an empty box for a dep test.

zentoo commented

2021-10-06 12:15:03 +02:00

I've tried to reduce it following requirements.txt from upstream.
So I ended with only the following dependencies and I did a depclean to be sure to purge all remaining packages before restart homeassistant:

RDEPEND=">=dev-python/requests-2.20.0[${PYTHON_USEDEP}]
    >=dev-python/geojson-2.3.0[${PYTHON_USEDEP}]
    >=dev-python/PySocks-1.7.1[${PYTHON_USEDEP}]"
BDEPEND="${REDEPEND}
    test? (
        dev-python/nose[${PYTHON_USEDEP}]
        dev-python/pytest[${PYTHON_USEDEP}]
    )"

So far everything is working perfectly, openweathermap data don't produce warning or errors while no new packages came in deps directory.

So we need to respect only requirements.txt and not dev-requirements.txt.

Note: may be there is other ebuilds that use another xxx-requirements.txt for dev.

I've tried to reduce it following requirements.txt from upstream. So I ended with only the following dependencies and I did a depclean to be sure to purge all remaining packages before restart homeassistant: ``` RDEPEND=">=dev-python/requests-2.20.0[${PYTHON_USEDEP}] >=dev-python/geojson-2.3.0[${PYTHON_USEDEP}] >=dev-python/PySocks-1.7.1[${PYTHON_USEDEP}]" BDEPEND="${REDEPEND} test? ( dev-python/nose[${PYTHON_USEDEP}] dev-python/pytest[${PYTHON_USEDEP}] )" ``` So far everything is working perfectly, openweathermap data don't produce warning or errors while no new packages came in deps directory. So we need to respect only requirements.txt and not dev-requirements.txt. Note: may be there is other ebuilds that use another xxx-requirements.txt for dev.

👍 1

onkelbeh referenced this issue from a commit

2021-10-07 07:02:32 +02:00

pyowm-3.2.0: cleanup deps, fixes #238

onkelbeh closed this issue

2021-10-07 07:02:32 +02:00

onkelbeh referenced this issue from a commit

2021-10-07 07:02:32 +02:00

pyowm-3.2.0: remove coverage, fixes #238

Sign in to join this conversation.