Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 703450 - net-wireless/gnuradio - missing dependency on sci-libs/gsl:0
Summary: net-wireless/gnuradio - missing dependency on sci-libs/gsl:0
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Radio project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-12-20 23:29 UTC by Sławomir Nizio
Modified: 2020-09-21 19:24 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sławomir Nizio 2019-12-20 23:29:34 UTC
net-wireless/gnuradio-3.7.13.5, at least with some USE flags (I guess fec and wavelet) depends on sci-libs/gsl but only with USE=wavelet it is specified so in the ebuild.

So probably adding fec? ( sci-libs/gsl ) or so is needed.

evidence 1:
gnuradio-3.7.13.5/gr-fec/apps/CMakeLists.txt:#  ${GSL_LIBRARIES}
gnuradio-3.7.13.5/gr-fec/lib/CMakeLists.txt:  link_directories(${GSL_LIBRARY_DIRS})
gnuradio-3.7.13.5/gr-wavelet/lib/CMakeLists.txt:link_directories(${GSL_LIBRARY_DIRS})
gnuradio-3.7.13.5/gr-wavelet/lib/CMakeLists.txt:    ${GSL_LIBRARIES}

evidence 2:
/usr/lib64/libgnuradio-fec-3.7.13.5.so.0.0.0 links with libgsl.so.0

Reproducible: Always
Comment 1 Sławomir Nizio 2019-12-20 23:32:23 UTC
> So probably adding fec? ( sci-libs/gsl ) or so is needed.

by "or so" I mean, with wavelet it's like this:

    wavelet? (                                                                                             
        >=sci-libs/gsl-1.10
    )

but now I can see that the older version in Portage is 1.16 so it's not necessary even *if* it would be needed.
Comment 2 Sławomir Nizio 2019-12-20 23:36:02 UTC
> so it's not necessary even *if* it would be needed.

s/it's not necessary/>= is not necessary/
Comment 3 Chris Mayo 2019-12-22 17:56:23 UTC
Agreed. Updated PR for 3.8.0.0:
https://github.com/gentoo/gentoo/pull/13842

$ grep -rn gsl | grep include
lib/fec_mtrx_impl.h:27:#include <gsl/gsl_blas.h>
lib/fec_mtrx_impl.h:28:#include <gsl/gsl_linalg.h>
lib/fec_mtrx_impl.h:29:#include <gsl/gsl_matrix.h>
lib/fec_mtrx_impl.h:30:#include <gsl/gsl_permutation.h>
lib/fec_mtrx_impl.h:31:#include <gsl/gsl_randist.h>
lib/ldpc_G_matrix_impl.h:24:#include <gsl/gsl_blas.h>
lib/ldpc_G_matrix_impl.h:25:#include <gsl/gsl_linalg.h>
lib/ldpc_G_matrix_impl.h:26:#include <gsl/gsl_permutation.h>
lib/ldpc_G_matrix_impl.h:27:#include <gsl/gsl_randist.h>
Comment 4 Rick Farina (Zero_Chaos) gentoo-dev 2020-09-21 19:24:22 UTC
thanks for this, especially the keeping the PR up to date part.  This was merged when 3.8.x was merged, and 3.7 is sunsetting.