Too much dependencies for dev-python/pyowm #238

Closed
opened 2 months ago by zentoo · 2 comments
zentoo commented 2 months ago

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 2 months ago
onkelbeh added the
cleanup
label 2 months ago
Owner

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

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

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.
onkelbeh referenced this issue from a commit 2 months ago
onkelbeh closed this issue 2 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.