Skip to contentSkip to conversation

package/nfs-utils: fix unmet direct dependency

PATCH New Jörg Krause 2 participants 2 messages Jul 20, 2015 – Jul 22, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/nfs-utils: fix unmet direct dependency

Commit 182520a8af734c735c4dbdff29dc07af1d615a0a adds rpcbind as a dependency without properly caring for the thread dependency.

rpcbind adds libtirpc as a dependendency which fails:
fatal error: pthread.h: No such file or directory

Since we probably don't care much about using nfs-utils in a non-threaded scenario, we simply disable nfs-utils for toolchains without thread support.

Fixes:
http://autobuild.buildroot.net/results/803/80321de8d4c6c0a00439b07289a5818962128b43/

---
 package/nfs-utils/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
package/nfs-utils/Config.in +2 -2
index 2806551..1273092 100644
@@ -1,10 +1,10 @@
 comment "nfs-utils needs a toolchain w/ threads"
 	depends on BR2_USE_MMU
-	depends on !(BR2_TOOLCHAIN_HAS_THREADS || BR2_TOOLCHAIN_HAS_NATIVE_RPC)
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
 
 config BR2_PACKAGE_NFS_UTILS
 	bool "nfs-utils"
-	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc
+	depends on BR2_TOOLCHAIN_HAS_THREADS # libtirpc, rpcbind
 	select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC
 	select BR2_PACKAGE_RPCBIND # runtime
 	depends on BR2_USE_MMU # fork()
Signature · 1 line
2.4.6
#1 lore raw

PATCH 1/1 package/nfs-utils: fix unmet direct dependency

Dear J?rg Krause,

Quoted 9 lines · J?rg Krause

Commit 182520a8af734c735c4dbdff29dc07af1d615a0a adds rpcbind as a dependency without properly caring for the thread dependency.

rpcbind adds libtirpc as a dependendency which fails:
fatal error: pthread.h: No such file or directory

Since we probably don't care much about using nfs-utils in a non-threaded scenario, we simply disable nfs-utils for toolchains without thread support.

Quoted 8 lines

Fixes:
http://autobuild.buildroot.net/results/803/80321de8d4c6c0a00439b07289a5818962128b43/

---
 package/nfs-utils/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com