Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 430348 - net-wireless/gnuradio-3.6.1-r1 USE="doc" - some documentation is installed outside /usr/share/doc/${PF}
Summary: net-wireless/gnuradio-3.6.1-r1 USE="doc" - some documentation is installed ou...
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Radio project
Keywords: PATCH
Depends on:
Reported: 2012-08-07 19:50 UTC by Chris Mayo
Modified: 2016-01-05 03:01 UTC (History)
4 users (show)

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

Patch to ebuild (gnuradio-3.6.1-r1.ebuild-doc.patch,1.16 KB, patch)
2012-08-07 19:50 UTC, Chris Mayo
Details | Diff
Allow GR_PKG_DOC_DIR to be defined by cmake (gnuradio-,586 bytes, patch)
2015-07-20 18:19 UTC, Chris Mayo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Mayo 2012-08-07 19:50:40 UTC
Created attachment 320646 [details, diff]
Patch to ebuild

As noted in the gnuradio-3.6.1-r1 ebuild.

Attached patch fixes this for me and:
- doesn't compress examples
- doesn't install xml version of docs (which also got compressed)

Tested with USE="doc examples".
Comment 1 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-08-07 21:33:16 UTC
I'd prefer something that could be upstreamed eventually. Which would probably mean to set the documentation via cmake parameter.
Comment 2 Chris Reffett (RETIRED) gentoo-dev Security 2012-08-13 23:02:49 UTC
Would passing -DGR_PKG_DOC_DIR=${GR_DOC_DIR}/${CMAKE_PROJECT_NAME}-${PVF} work? Or maybe not quite that, the intent is to have portage expand PVF but leave the other two variables alone.
Comment 3 Rick Farina (Zero_Chaos) gentoo-dev 2015-07-10 19:32:24 UTC
kinda pathetic that it took me this long to get to testing this, but meh, doc location bug.

turns out proper quoting is our friend


thanks Chris
Comment 4 Chris Mayo 2015-07-20 18:19:18 UTC
Created attachment 407286 [details, diff]
Allow GR_PKG_DOC_DIR to be defined by cmake

Doesn't seem to work for me, testing with USE="-doc examples":

$ ls /usr/share/doc/gnuradio-3.7.7*
README.analog.bz2  README.blocks.bz2  README.filter.bz2   README.qtgui.bz2   README.bz2         README.fft.bz2      README.hacking.bz2

examples  README.building-boost.bz2  README.bz2  README.hacking.bz2

GR_PKG_DOC_DIR is set by ./CMakeLists.txt, overwriting any value defined by cmake. Attached patch changes that (easy to use thanks to epatch_user).

But with the patch:
$ qlist gnuradio | grep README

${GR_DOC_DIR}/${CMAKE_PROJECT_NAME} doesn't get expanded, and ${PVF} doesn't exist?

-DGR_PKG_DOC_DIR="${EPREFIX}"/usr/share/doc/"${PF}" does work for me.

It appears the patch will be unnecessary with 3.7.8 (August 8):
Comment 5 Rick Farina (Zero_Chaos) gentoo-dev 2016-01-04 20:44:45 UTC
pretty confident that it's the opposite of better:

zero@gato ~ % qlist gnuradio | grep README
Comment 6 Rick Farina (Zero_Chaos) gentoo-dev 2016-01-05 03:01:20 UTC
+	sed -i "s:\${DOCVER}:${PVR}:" CMakeLists.txt || die

Not really sure that this line is needed so I'm not using it.  Please let me know if I'm entirely mistaken.