From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Neil Jerram Newsgroups: gmane.lisp.guile.devel Subject: Re: [PATCH] Fix hanging of popen.test Date: Sun, 04 Jul 2010 21:33:27 +0100 Message-ID: <874ogfrnqf.fsf@ossau.uklinux.net> References: <1276210496-6477-1-git-send-email-neil@ossau.uklinux.net> <87vd9p5p07.fsf@ossau.uklinux.net> <871vbq25if.fsf@ossau.uklinux.net> <87d3v9irhy.fsf@ossau.uklinux.net> <87sk442l0n.fsf@ossau.uklinux.net> <87bpasvzra.fsf@ossau.uklinux.net> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: dough.gmane.org 1278276019 32184 80.91.229.12 (4 Jul 2010 20:40:19 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Sun, 4 Jul 2010 20:40:19 +0000 (UTC) Cc: guile-devel@gnu.org To: Andy Wingo Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Sun Jul 04 22:40:17 2010 Return-path: Envelope-to: guile-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1OVVz6-0008Ed-2W for guile-devel@m.gmane.org; Sun, 04 Jul 2010 22:40:16 +0200 Original-Received: from localhost ([127.0.0.1]:51218 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OVVz4-0004A8-Ij for guile-devel@m.gmane.org; Sun, 04 Jul 2010 16:40:14 -0400 Original-Received: from [140.186.70.92] (port=59932 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OVVto-0000sw-G3 for guile-devel@gnu.org; Sun, 04 Jul 2010 16:34:57 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OVVt8-0007md-So for guile-devel@gnu.org; Sun, 04 Jul 2010 16:34:08 -0400 Original-Received: from mail3.uklinux.net ([80.84.72.33]:60936) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OVVt8-0007eR-M2 for guile-devel@gnu.org; Sun, 04 Jul 2010 16:34:06 -0400 Original-Received: from arudy (unknown [92.29.72.223]) by mail3.uklinux.net (Postfix) with ESMTP id 4CC631F68DC; Sun, 4 Jul 2010 21:33:29 +0100 (BST) Original-Received: from arudy (unknown [192.168.11.8]) by arudy (Postfix) with ESMTP id 98D8E3801D; Sun, 4 Jul 2010 21:33:28 +0100 (BST) In-Reply-To: (Andy Wingo's message of "Thu, 01 Jul 2010 11:48:49 +0100") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.4-2.6 X-BeenThere: guile-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Developers list for Guile, the GNU extensibility library" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Errors-To: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.lisp.guile.devel:10612 Archived-At: Andy Wingo writes: > This has been an on-and-off issue: > 02fcbf78b27788c03563e5c3d297a4cd469ce562, and > 04af4c4c5221c082905d52eb5ad3829ed681d097. Right. The commit (4f99a499197b592a9a3060de2205531852f4f94) that Patrick McCarty identified (with git bisect) is another very similar case. Even if we fix this particular one, it seems likely that similar cases will keep arising, so a more general fix would be better. The general problem is that any #:use-module or (@ ...) that indirectly pulls in srfi-1, in any code that is run as part of Guile startup, will cause the build to fail when doing the snarf-check-and-output-texi, because libguile-srfi-srfi-13-14-v-4 hasn't been built at that point. But if the developer concerned happens to have a compatible libguile-srfi-srfi-13-14-v-4 and libguile installed in /usr/lib or /usr/local/lib, the build may pick those up and so mask the problem. Is there a reason why we don't just move all the SRFI C code into the core libguile? I think that would be a general fix. Alternatively we could try making snarf-check-and-output-texi happen later in the build. But I had a quick go at that and found it tricky, because of the dependency on all the .doc files, and those being listed and generated by libguile/Makefile.am; if we moved the snarf-check-and-output-texi step to, say, doc/Makefile.am, we'd lose that dependency. > I don't really get it. I thought this was fixed. There have been a > couple threads about this, even, if you search the archives in recent > months. I've checked the one starting at http://lists.gnu.org/archive/html/guile-devel/2010-06/msg00029.html. But I think it just covers one particular case, that you fixed. Regards, Neil