Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 756757 - net-wireless/gqrx-2.14: fails to build with USE=-pulseaudio
Summary: net-wireless/gqrx-2.14: fails to build with USE=-pulseaudio
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Rick Farina (Zero_Chaos)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-25 18:23 UTC by Daniel M. Weeks
Modified: 2020-12-21 17:35 UTC (History)
3 users (show)

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


Attachments
Patch to remove sed patching of missing file (gqrx-2.14-ebuild.patch,427 bytes, patch)
2020-11-27 01:49 UTC, David Turner
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel M. Weeks 2020-11-25 18:23:43 UTC
* ERROR: net-wireless/gqrx-2.14::gentoo failed (prepare phase):
 *   (no error message)
 * 
 * Call stack:
 *     ebuild.sh, line 125:  Called src_prepare
 *   environment, line 2003:  Called die
 * The specific snippet of code:
 *           sed -i 's/AUDIO_BACKEND = pulse/#AUDIO_BACKEND = pulse/' gqrx.pro || die;
 * 

Reproducible: Always
Comment 1 David Turner 2020-11-27 01:40:14 UTC
This is quite annoying since gqrx was bumped to v2.14 as v2.12.1-r1 fails to build as per bug #751082 ... and it appears there was no test that v2.14 builds when pulseaudio is disabled.

The issue in the ebuild is that it does a sed patch to the gqrx.pro file in that
case, but that file is no longer present in the codebase.

However, it should be safe to remove the sed patch for v2.14 as that pro file is only used for the older qmake build variant. cmake is now the only option which is what the ebuild uses anyway. See:
https://github.com/csete/gqrx/commit/bbdf77c09a3bbcce200f50cfe5f5332f4b6b3f93

I will try removing that sed patch from a local ebuild overlay on my pulseaudio free system and see if the result builds....
Comment 2 David Turner 2020-11-27 01:46:28 UTC
Yep. Works fine if those lines are removed. Will add a ebuild patch to this bug...
Comment 3 David Turner 2020-11-27 01:49:08 UTC
Created attachment 675232 [details, diff]
Patch to remove sed patching of missing file
Comment 4 David Turner 2020-11-27 01:50:04 UTC
To be clear, gqrx builds on my system with -pulseaudio and starts fine... I did a quick check on the FM band and seem fine.
Comment 5 Rick Farina (Zero_Chaos) gentoo-dev 2020-12-21 17:35:03 UTC
This was fixed a little while ago based on feedback from a different bug and some testing. Thanks for the report, hope it works better now.