diff mbox

sane-backends: uses fork(), not available on noMMU systems

Message ID 1400319074-3056-1-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit 8989bfc8a6b88b56d7942a95d6e7517075ed8ffc
Headers show

Commit Message

Thomas Petazzoni May 17, 2014, 9:31 a.m. UTC
sane-backends currently doesn't build with the ADI external
toolchains, due to an internal compiler failure. However, using the
internal toolchain backend for Blackfin goes past this internal
compiler failure, but raises the problem that sane-backends uses
fork(). Sane-backends has a --enable-pthreads option that allows to
use threads instead of forking in some parts of sane, but there are
still some other areas of sane-backends that continue to use fork().

Since the probability of using sane-backends on a noMMU system is
fairly small, we'll just disable this package for now on such
architectures, until someone cares enough to investigate deeper.

Fixes:

  http://autobuild.buildroot.org/results/136/136100c3a2d843b7915fdd83d23d8b301fee577e/

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/sane-backends/Config.in | 1 +
 1 file changed, 1 insertion(+)

Comments

Peter Korsgaard May 17, 2014, 9:38 a.m. UTC | #1
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > sane-backends currently doesn't build with the ADI external
 > toolchains, due to an internal compiler failure. However, using the
 > internal toolchain backend for Blackfin goes past this internal
 > compiler failure, but raises the problem that sane-backends uses
 > fork(). Sane-backends has a --enable-pthreads option that allows to
 > use threads instead of forking in some parts of sane, but there are
 > still some other areas of sane-backends that continue to use fork().

 > Since the probability of using sane-backends on a noMMU system is
 > fairly small, we'll just disable this package for now on such
 > architectures, until someone cares enough to investigate deeper.

 > Fixes:

 >   http://autobuild.buildroot.org/results/136/136100c3a2d843b7915fdd83d23d8b301fee577e/

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Committed, thanks.
diff mbox

Patch

diff --git a/package/sane-backends/Config.in b/package/sane-backends/Config.in
index 04222fc..8156e85 100644
--- a/package/sane-backends/Config.in
+++ b/package/sane-backends/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_SANE_BACKENDS
 	bool "sane-backends"
+	depends on BR2_USE_MMU # fork()
 	help
 	  SANE - Scanner Access Now Easy