Package: jade
Version: 1.2.1-47.1
Severity: important
Tags: patch

Hello,

On hurd-i386, jade automatically enables the use of libthreads. This is
however not a good idea for at least two reasons:

- threads are not used on Linux, so the threaded code is not actually
  tested.
- libthreads is a lightweight threading library meant to be able to
  start thousands of threads, and thus the default stack size is tiny
  (64KiB), while jade apparently uses recursion quite a bit, leading to
  stack overflows, e.g. when building the libusb package.

The attached patch disables it. Of course re-autoconf will be needed.

Samuel

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (500, 'stable'), (1, 
'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.0.4 (SMP w/8 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages jade depends on:
ii  libc6       2.13-27
ii  libgcc1     1:4.7.0-1
ii  libsp1c2    1.3.4-1.2.1-47.1
ii  libstdc++6  4.7.0-1
ii  sgml-base   1.26+nmu1

jade recommends no packages.

Versions of packages jade suggests:
ii  doc-base   0.10.3
ii  sgml-data  2.0.6
ii  sp         1.3.4-1.2.1-47.1

-- no debconf information

-- 
Samuel Thibault <[email protected]>
"And the next time you consider complaining that running Lucid Emacs
19.05 via NFS from a remote Linux machine in Paraguay doesn't seem to
get the background colors right, you'll know who to thank."
(By Matt Welsh)
--- configure.in.orig   2012-04-16 02:48:01.000000000 +0200
+++ configure.in        2012-04-16 02:48:03.000000000 +0200
@@ -106,8 +106,8 @@
 
 dnl
 dnl Check for thread support.
-AC_CHECK_LIB(threads,cthread_fork,LIB_THREADS="-lthreads",LIB_THREADS="")
-AC_SUBST(LIB_THREADS)
+dnl AC_CHECK_LIB(threads,cthread_fork,LIB_THREADS="-lthreads",LIB_THREADS="")
+dnl AC_SUBST(LIB_THREADS)
 
 dnl 
 dnl  ANSI templates
--- spgrove/threads.h.orig      2012-04-16 03:23:16.000000000 +0200
+++ spgrove/threads.h   2012-04-16 03:23:19.000000000 +0200
@@ -141,7 +141,7 @@
 
 #endif /* _MSC_VER && _MT */
 
-#if (defined __MACH__) || (defined __GNU__)
+#if 0
 
 // For Mach, using C Threads. May or may not work as-is on your Mach-based OS.
 // Written by Raf Schietekat <[email protected]> on 1996-11-10.

Reply via email to