Would it be possible to make LLVM an optional build flag?
Hi, if I understand you correctly your bug is a request to make the dep to llvm (or clang) optional. However, from the ebuild itself I don't see a dependency on llvm/clang: https://gitweb.gentoo.org/repo/gentoo.git/tree/net-wireless/gnuradio/gnuradio-3.10.4.0.ebuild Can you please add the output of the following two commands on your system to this bug (e.g. as attachments)? - 'emerge -pvt net-wireless/gnuradio' - 'emerge --info' Thanks, Nils
Hi Nils, gnuradio depends on pygccxml, which depends on castxml, which in turn depends on clang/llvm. I guess the question should be if it is possible to make pygccxml optional for gnuradio. Regards, Laszlo
I noticed this bug today while bumping gnuradio and I've asked the gnuradio team about it. No promises on what comes of this, but I'm exploring.
Thus far I have confirmed that modtool requires this, and it breaks without it, so gentoo seems to be doing things "right" by requiring llvm here https://github.com/gnuradio/gnuradio/issues/5329 Since pygccxml is only used for modtool, I'm moving the dep under the modtool use flag. Based on conversations in #gnuradio, if pygccxml isn't installed, modtool will sometimes work, and sometimes not work. As such, I'm keeping it as a requirement for modtool and have opened an upstream bug to maybe have a more useful "you should install pygccxml" instead of just failing with no obvious recourse.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c8d732e10e7cc0564e13c6d1d39a5dc383029128 commit c8d732e10e7cc0564e13c6d1d39a5dc383029128 Author: Rick Farina <zerochaos@gentoo.org> AuthorDate: 2022-12-19 19:31:56 +0000 Commit: Rick Farina <zerochaos@gentoo.org> CommitDate: 2022-12-19 19:32:25 +0000 net-wireless/gnuradio: adjust deps more correctly Closes: https://bugs.gentoo.org/883143 Signed-off-by: Rick Farina <zerochaos@gentoo.org> .../gnuradio/{gnuradio-3.10.5.0.ebuild => gnuradio-3.10.5.0-r1.ebuild} | 2 +- net-wireless/gnuradio/gnuradio-9999.ebuild | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
To be super explicit, right now USE=modtool still needs llvm, but you can disable modtool to avoid the dep if you like. I'll do better depending on upstream's resolution of https://github.com/gnuradio/gnuradio/issues/6441
