From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Thien-Thi Nguyen Newsgroups: gmane.lisp.guile.bugs Subject: Re: [PATCH] Declare `GC_dump' ourselves if doesn't. Date: Tue, 12 Jan 2010 19:51:49 +0100 Message-ID: <87d41fma0q.fsf@ambire.localdomain> References: <87zl4nmmpl.fsf@ambire.localdomain> <878wc45m5w.fsf@gnu.org> <87k4vodjga.fsf@ambire.localdomain> <874omro1b1.fsf@gnu.org> <87hbqrmj2d.fsf@ambire.localdomain> <87my0jfdfz.fsf@gnu.org> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1263322920 17460 80.91.229.12 (12 Jan 2010 19:02:00 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 12 Jan 2010 19:02:00 +0000 (UTC) To: bug-guile@gnu.org Original-X-From: bug-guile-bounces+guile-bugs=m.gmane.org@gnu.org Tue Jan 12 20:01:53 2010 Return-path: Envelope-to: guile-bugs@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1NUlzs-0002Kp-Hs for guile-bugs@m.gmane.org; Tue, 12 Jan 2010 20:01:44 +0100 Original-Received: from localhost ([127.0.0.1]:57232 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NUlzt-0006Jt-Bv for guile-bugs@m.gmane.org; Tue, 12 Jan 2010 14:01:45 -0500 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NUltE-0000eh-Ve for bug-guile@gnu.org; Tue, 12 Jan 2010 13:54:53 -0500 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NUlt3-0000Oq-Gg for bug-guile@gnu.org; Tue, 12 Jan 2010 13:54:52 -0500 Original-Received: from [199.232.76.173] (port=48161 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NUlt3-0000Of-A5 for bug-guile@gnu.org; Tue, 12 Jan 2010 13:54:41 -0500 Original-Received: from smtp-out01.alice.it ([85.33.2.12]:4897) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NUlt2-00079k-L7 for bug-guile@gnu.org; Tue, 12 Jan 2010 13:54:41 -0500 Original-Received: from FBCMMO04.fbc.local ([192.168.184.135]) by smtp-out01.alice.it with Microsoft SMTPSVC(6.0.3790.3959); Tue, 12 Jan 2010 19:54:32 +0100 Original-Received: from FBCMCL01B08.fbc.local ([192.168.171.46]) by FBCMMO04.fbc.local with Microsoft SMTPSVC(6.0.3790.3959); Tue, 12 Jan 2010 19:54:18 +0100 Original-Received: from ambire.localdomain ([95.237.68.156]) by FBCMCL01B08.fbc.local with Microsoft SMTPSVC(6.0.3790.3959); Tue, 12 Jan 2010 19:54:18 +0100 Original-Received: from ttn by ambire.localdomain with local (Exim 4.63) (envelope-from ) id 1NUlqH-0003lN-GW for bug-guile@gnu.org; Tue, 12 Jan 2010 19:51:49 +0100 In-Reply-To: <87my0jfdfz.fsf@gnu.org> ("Ludovic =?utf-8?Q?Court=C3=A8s=22'?= =?utf-8?Q?s?= message of "Tue, 12 Jan 2010 18:19:44 +0100") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.91 (gnu/linux) X-OriginalArrivalTime: 12 Jan 2010 18:54:18.0359 (UTC) FILETIME=[A4EAB070:01CA93B8] X-detected-operating-system: by monty-python.gnu.org: Windows 2000 SP4, XP SP1+ X-BeenThere: bug-guile@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Bug reports for GUILE, GNU's Ubiquitous Extension Language" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: bug-guile-bounces+guile-bugs=m.gmane.org@gnu.org Errors-To: bug-guile-bounces+guile-bugs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.lisp.guile.bugs:4452 Archived-At: () ludo@gnu.org (Ludovic Court=C3=A8s) () Tue, 12 Jan 2010 18:19:44 +0100 > | dnl See note for PKG_CHECK_MODULES in aclocal.m4. > | PKG_PROG_PKG_CONFIG > | if test "$BDW_GC_CFLAGS" || test "$BDW_GC_LIBS" ; then : > | dnl We don't need to declare those env vars precious; > | dnl PKG_CHECK_MODULES does that. > | else > | PKG_CHECK_MODULES([BDW_GC], [bdw-gc]) > | fi This patch shouldn=E2=80=99t be necessary since it duplicates what =E2=80=98PKG_CHECK_MODULES=E2=80=99 does. Not exactly. If there is no bdw-gc.pc on the system (as is the case here on a Debian Etch installation), PKG_CHECK_MODULES dies, and so does the configure script. The original README recommends in such case to munge three variables. Two of them are valid: BDW_GC_CFLAGS and BDW_GC_LIBS. The other (PKG_CONFIG) is bogus because munging it interferes with "make installcheck". ... without reading =E2=80=98README=E2=80=99, so you basically shoot you= rself in the foot and there=E2=80=99s not much we can do. :-) I don't know where you got that idea. Here's what actually happened: I read the README, applied the bogus advice, noticed the breakage, and then worked on the patch so that the now-updated README gives good advice, supported by the now-updated configure script. Lastly, i posted the patch for review. There IS something you can do: take a stand against pkg-config and DTRT (the Autoconf Way). I know there are other packages that are heavily reliant on pkg-config, but Guile doesn't seem to be one of them. You can do this by helping identify what are the features of libgc (whatever version) that Guile requires. With that info, it would not be hard to write a test for the configure script. So, please help answer that question (if you can). thi