dev-python/pymicro-vad needs BDEPEND=dev-python/pybind11 #4122

Closed
opened 2024-12-14 21:56:30 +01:00 by oo-stef · 1 comment

(see also #4121)
Hello,
When (re)compiling dev-python/pymicro-vad, I noticed that the ebuild does not mention the BDEPEND on pybind11.
I also got a QA error regarding the missing DISTUTILS_EXT.

Proposed fix for the ebuild (it might be missing test? (dev-python/pytest[${PYTHON_USEDEP}]) as well - I didn't run tests):

diff -u a/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild b/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild 
--- a/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild       2024-10-27 06:18:01.182461819 +0100
+++ b/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild       2024-12-14 21:51:34.904548001 +0100
@@ -4,6 +4,7 @@
 EAPI=8
 
 PYTHON_COMPAT=( python3_{11..13} )
+DISTUTILS_EXT=1
 DISTUTILS_USE_PEP517=setuptools
 inherit distutils-r1 pypi
 
@@ -17,6 +18,10 @@
 IUSE="test"
 RESTRICT="!test? ( test )"
 
+BDEPEND="
+        dev-python/pybind11[${PYTHON_USEDEP}]
+       "
+
 src_unpack() {
        unpack ${A}
        echo ${P}
(see also #4121) Hello, When (re)compiling dev-python/pymicro-vad, I noticed that the ebuild does not mention the BDEPEND on pybind11. I also got a QA error regarding the missing DISTUTILS_EXT. Proposed fix for the ebuild (it might be missing test? (dev-python/pytest[${PYTHON_USEDEP}]) as well - I didn't run tests): ``` diff -u a/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild b/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild --- a/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild 2024-10-27 06:18:01.182461819 +0100 +++ b/dev-python/pymicro-vad/pymicro-vad-1.0.1.ebuild 2024-12-14 21:51:34.904548001 +0100 @@ -4,6 +4,7 @@ EAPI=8 PYTHON_COMPAT=( python3_{11..13} ) +DISTUTILS_EXT=1 DISTUTILS_USE_PEP517=setuptools inherit distutils-r1 pypi @@ -17,6 +18,10 @@ IUSE="test" RESTRICT="!test? ( test )" +BDEPEND=" + dev-python/pybind11[${PYTHON_USEDEP}] + " + src_unpack() { unpack ${A} echo ${P} ```

funny, I was about to create the exact same issue!
Is it only a build dependency?

funny, I was about to create the exact same issue! Is it only a build dependency?
onkelbeh referenced this issue from a commit 2025-02-18 10:56:57 +01:00
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: onkelbeh/HomeAssistantRepository#4122