* [Andy Wingo] Re: [PATCH] Fix for `submodules' in (ice-9 session) (closes #30062)
@ 2010-09-03 5:27 Andy Wingo
0 siblings, 0 replies; only message in thread
From: Andy Wingo @ 2010-09-03 5:27 UTC (permalink / raw)
To: guile-devel
[-- Attachment #1: Type: message/rfc822, Size: 1603 bytes --]
From: Andy Wingo <wingo@pobox.com>
To: "Jose A. Ortega Ruiz" <jao@gnu.org>
Subject: Re: [PATCH] Fix for `submodules' in (ice-9 session) (closes #30062)
Date: Thu, 02 Sep 2010 22:01:16 -0700
Message-ID: <m3bp8fo1yb.fsf@unquote.localdomain>
Hi,
Sorry for the delay. The reason I was holding off was because my
backwards-compatibility shims should have been working, but they
weren't, and so I wanted to commit the fix to the back-compat shims,
then have your test-case go in (to verify back-compatibility), then have
your fix to the submodules function go in (to avoid using deprecated
features).
Specifically I wanted to test that one could resolve modules via the
value namespace only in Guile 2.0. Your fix is needed regardless to
work with Guile built without support for deprecated code.
I have now pushed my back-compat fix, then applied your
patch. Everything is working fine, and I have pushed.
However why does adding the test:
(pass-if "a leaf module: ice-9 q" (find-mod-name '(ice-9 q)))
Why does that fail? Is it because ice-9 q is not loaded?
Thanks for the patch, and the debugging, and for putting up with the
back-and-forth :)
Cheers,
Andy
--
http://wingolog.org/
[-- Attachment #2: Type: text/plain, Size: 27 bytes --]
--
http://wingolog.org/
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2010-09-03 5:27 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-03 5:27 [Andy Wingo] Re: [PATCH] Fix for `submodules' in (ice-9 session) (closes #30062) Andy Wingo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).