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 Subject: Re: [PATCH] Add ".guile.sls" and ".sls" to the default %load-extensions Date: Mon, 05 Nov 2012 22:15:19 +0100 Message-ID: <87liefsqe0.fsf@gnu.org> References: <87fw4qgi1i.fsf@tines.lan> <87k3u16rak.fsf@gnu.org> <87k3u1knqn.fsf@delenn.lan> <87bofd6j24.fsf@gnu.org> <87k3u0kiny.fsf@delenn.lan> <87pq3s9j2b.fsf@gnu.org> <874nl3epxw.fsf@tines.lan> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1352150136 11360 80.91.229.3 (5 Nov 2012 21:15:36 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 5 Nov 2012 21:15:36 +0000 (UTC) Cc: guile-devel@gnu.org To: Mark H Weaver Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Mon Nov 05 22:15:46 2012 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 1TVU1I-0006Uc-JE for guile-devel@m.gmane.org; Mon, 05 Nov 2012 22:15:44 +0100 Original-Received: from localhost ([::1]:49749 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVU19-0000U6-GH for guile-devel@m.gmane.org; Mon, 05 Nov 2012 16:15:35 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:34433) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVU15-0000TT-Os for guile-devel@gnu.org; Mon, 05 Nov 2012 16:15:32 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TVU14-0008UZ-L5 for guile-devel@gnu.org; Mon, 05 Nov 2012 16:15:31 -0500 Original-Received: from [2a01:e0b:1:123:ca0a:a9ff:fe03:271e] (port=45783 helo=xanadu.aquilenet.fr) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVU14-0008TL-F8 for guile-devel@gnu.org; Mon, 05 Nov 2012 16:15:30 -0500 Original-Received: from localhost (localhost [127.0.0.1]) by xanadu.aquilenet.fr (Postfix) with ESMTP id 600F4A616; Mon, 5 Nov 2012 22:15:20 +0100 (CET) Original-Received: from xanadu.aquilenet.fr ([127.0.0.1]) by localhost (xanadu.aquilenet.fr [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id s1lXSHWgZeMJ; Mon, 5 Nov 2012 22:15:20 +0100 (CET) Original-Received: from pluto (reverse-83.fdn.fr [80.67.176.83]) by xanadu.aquilenet.fr (Postfix) with ESMTPSA id 952E3A53A; Mon, 5 Nov 2012 22:15:19 +0100 (CET) X-URL: http://www.fdn.fr/~lcourtes/ X-Revolutionary-Date: 15 Brumaire an 221 de la =?utf-8?Q?R=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 In-Reply-To: <874nl3epxw.fsf@tines.lan> (Mark H. Weaver's message of "Mon, 05 Nov 2012 15:48:43 -0500") User-Agent: Gnus/5.130005 (Ma Gnus v0.5) Emacs/24.2 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 2a01:e0b:1:123:ca0a:a9ff:fe03:271e 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:15098 Archived-At: Hi! Mark H Weaver skribis: > I don't think the command-line switch is a good solution. The problem > is that even if the main program is not written in R6RS, and even if the > main program does not use any R6RS libraries directly, this option might > still be needed. Right, good point. > I see only two sane options: > > * Add these load extensions by default, to enable users and Guildhall > repository maintainers to easily import R6RS libraries as-is. > > * Reject the dominant R6RS library naming convention, and force users > and guildhall repository maintainers to rename the files to .scm and > move *.guile.sls to *.scm. > > I can live with either of these options, but given the existence of > guildhall, I see no good compromise position. IMO, we need to make a > decision to either fully embrace or fully reject the R6RS library naming > convention. Yes, makes sense. Looking at your initial patch, the two extensions were actually searched My current inclination would be to fully reject it, then. The problem is that we=E2=80=99d be adding two stats for each file. Currently there are two stat(2) for each entry in $GUILE_LOAD_PATH: stat("/home/ludo/.local/share/guile/site/2.0/ice-9/control.scm", 0x7fff1f4a= ec20) =3D -1 ENOENT (No such file or directory) stat("/home/ludo/.local/share/guile/site/2.0/ice-9/control", 0x7fff1f4aec20= ) =3D -1 ENOENT (No such file or directory) stat("/home/ludo/soft/share/guile/site/2.0/ice-9/control.scm", 0x7fff1f4aec= 20) =3D -1 ENOENT (No such file or directory) stat("/home/ludo/soft/share/guile/site/2.0/ice-9/control", 0x7fff1f4aec20) = =3D -1 ENOENT (No such file or directory) stat("/home/ludo/soft/share/guile/2.0/ice-9/control.scm", {st_mode=3DS_IFRE= G|0644, st_size=3D2961, ...}) =3D 0 (I=E2=80=99m not sure if searching for a file name with no extension is intentional. Ideas?) That would double that. This is likely to become noticeable with large $GUILE_LOAD_PATH values and/or numerous module loads. WDYT? Ludo=E2=80=99.