From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Newsgroups: gmane.lisp.guile.devel,gmane.lisp.guile.user Subject: Re: Loading a module before and after adding a load path Date: Mon, 21 Jan 2013 18:20:17 +0100 Message-ID: <87wqv68nce.fsf@gnu.org> References: <87bocmxzax.fsf@supernova.vialactea> <1358565467.2720.35.camel@Renee-desktop.suse> <87ip6tx0yf.fsf@supernova.vialactea> <877gn7bt4g.fsf@pobox.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Trace: ger.gmane.org 1358788835 16439 80.91.229.3 (21 Jan 2013 17:20:35 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 21 Jan 2013 17:20:35 +0000 (UTC) Cc: guile-user@gnu.org To: guile-devel@gnu.org Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Mon Jan 21 18:20:54 2013 Return-path: Envelope-to: guile-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TxL3G-0000MI-3A for guile-devel@m.gmane.org; Mon, 21 Jan 2013 18:20:54 +0100 Original-Received: from localhost ([::1]:60860 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxL2z-0004le-07 for guile-devel@m.gmane.org; Mon, 21 Jan 2013 12:20:37 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:38013) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxL2u-0004lG-OK for guile-devel@gnu.org; Mon, 21 Jan 2013 12:20:34 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TxL2t-0003N2-Df for guile-devel@gnu.org; Mon, 21 Jan 2013 12:20:32 -0500 Original-Received: from plane.gmane.org ([80.91.229.3]:45785) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TxL2t-0003Mq-76 for guile-devel@gnu.org; Mon, 21 Jan 2013 12:20:31 -0500 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1TxL37-0000Jp-3R for guile-devel@gnu.org; Mon, 21 Jan 2013 18:20:45 +0100 Original-Received: from 193.50.110.87 ([193.50.110.87]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 21 Jan 2013 18:20:45 +0100 Original-Received: from ludo by 193.50.110.87 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 21 Jan 2013 18:20:45 +0100 X-Injected-Via-Gmane: http://gmane.org/ Original-Followup-To: gmane.lisp.guile.devel Original-Lines: 39 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: 193.50.110.87 X-URL: http://www.fdn.fr/~lcourtes/ X-Revolutionary-Date: 2 =?utf-8?Q?Pluvi=C3=B4se?= an 221 de la =?utf-8?Q?R?= =?utf-8?Q?=C3=A9volution?= X-PGP-Key-ID: 0xEA52ECF4 X-PGP-Key: http://www.fdn.fr/~lcourtes/ludovic.asc X-PGP-Fingerprint: 83C4 F8E5 10A3 3B4C 5BEA D15D 77DD 95E2 EA52 ECF4 X-OS: x86_64-unknown-linux-gnu User-Agent: Gnus/5.130005 (Ma Gnus v0.5) Emacs/24.2 (gnu/linux) Cancel-Lock: sha1:14daerJaIpn2LalatEfY0s3Dfgo= X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 80.91.229.3 X-BeenThere: guile-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Developers list for Guile, the GNU extensibility library" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Original-Sender: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.lisp.guile.devel:15496 gmane.lisp.guile.user:9896 Archived-At: Andy Wingo skribis: > On Sat 19 Jan 2013 05:32, Noah Lavine writes: > >> In the long term, Andy and Ludo, what is the right way to fix this? > > This bug was introduced here: > > commit 0fb81f95b0222c5ba49efd3e36cf797df54c0863 > Author: Andy Wingo > Date: Wed Jun 3 09:48:16 2009 +0200 > > add exception_on_error optional arg to primitive-load-path > > Before, "didit" would be #f after a failed primitive-load-path, and > afterwards control proceeded to set it to #t. Good catch! > From 611e4d12abe5305b14ae24e4f135d5fc57ce0e9a Mon Sep 17 00:00:00 2001 > From: Andy Wingo > Date: Sun, 20 Jan 2013 19:33:42 +0100 > Subject: [PATCH] fix try-module-autoload, which did not detect failure to > find the file > > * libguile/load.c (scm_primitive_load_path): If the second argument is a > procedure, call it like a thunk. > > * doc/ref/api-evaluation.texi (Load Paths): Update docs. > > * module/ice-9/boot-9.scm (resolve-interface): Use `unless'. > (try-module-autoload): Use the new primitive-load-path to detect > failure to find an appropriate file. Fixes a bug reported by Diogo > F. S. Ramos. Thanks to Noah Lavine for tracking it down. Looks good to me. Thanks, Luod’.