diff mbox

package/dvblast: fix static linking

Message ID 1467645076-13092-1-git-send-email-yann.morin.1998@free.fr
State Accepted
Headers show

Commit Message

Yann E. MORIN July 4, 2016, 3:11 p.m. UTC
dvblast uconditionally believes that any Linux system will have support
for clock nanosleep().

However, clock_nanosleep() is only availabe with NPTL.

Fixing dvblast is not really complicated, but not trivial either. So we
jsut make it depend on NPTL.

Fixes:
    http://autobuild.buildroot.org/?reason=dvblast-3.0
    http://autobuild.buildroot.org/results/dbe/dbe91dd6543b4b88fbf1e5b09d4997d2dc730747/
    http://autobuild.buildroot.org/results/32b/32bee5071f46b86de61db7f35a2cd04e270bc73c/
    [and many more...]

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/dvblast/Config.in | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Thomas Petazzoni July 4, 2016, 3:24 p.m. UTC | #1
Hello,

On Mon,  4 Jul 2016 17:11:16 +0200, Yann E. MORIN wrote:
> dvblast uconditionally believes that any Linux system will have support
> for clock nanosleep().
> 
> However, clock_nanosleep() is only availabe with NPTL.
> 
> Fixing dvblast is not really complicated, but not trivial either. So we
> jsut make it depend on NPTL.

jsut -> just

Also, the title was wrong: you're not fixing static linking, but build
on non-NPTL platforms. So I fixed that up and applied.

Thanks,

Thomas
diff mbox

Patch

diff --git a/package/dvblast/Config.in b/package/dvblast/Config.in
index 1ea8f93..b9c5c14 100644
--- a/package/dvblast/Config.in
+++ b/package/dvblast/Config.in
@@ -4,13 +4,13 @@  config BR2_PACKAGE_DVBLAST
 	select BR2_PACKAGE_LIBEV
 	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	depends on !BR2_bfin  # libev
-	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL # clock_nanosleep()
 	help
 	  DVBlast is a simple and powerful MPEG-2/TS demux and
 	  streaming application.
 
 	  http://www.videolan.org/projects/dvblast.html
 
-comment 'dvblast needs a toolchain w/ threads'
+comment 'dvblast needs a toolchain w/ NPTL'
 	depends on !BR2_bfin  # libev
-	depends on !BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_TOOLCHAIN_HAS_THREADS_NPTL