From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: nalaginrut Newsgroups: gmane.lisp.guile.devel Subject: Re: [PATCH] Add ".guile.sls" and ".sls" to the default %load-extensions Date: Tue, 06 Nov 2012 10:19:42 +0800 Organization: HFG Message-ID: <1352168382.2654.21.camel@Renee-desktop.suse> 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> <87liefsqe0.fsf@gnu.org> 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 1352168392 27175 80.91.229.3 (6 Nov 2012 02:19:52 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 6 Nov 2012 02:19:52 +0000 (UTC) Cc: Mark H Weaver , guile-devel@gnu.org To: Ludovic =?ISO-8859-1?Q?Court=E8s?= Original-X-From: guile-devel-bounces+guile-devel=m.gmane.org@gnu.org Tue Nov 06 03:20:01 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 1TVYlk-00042P-5F for guile-devel@m.gmane.org; Tue, 06 Nov 2012 03:20:00 +0100 Original-Received: from localhost ([::1]:60002 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVYlb-0007HE-8M for guile-devel@m.gmane.org; Mon, 05 Nov 2012 21:19:51 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:33617) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVYlZ-0007Gy-51 for guile-devel@gnu.org; Mon, 05 Nov 2012 21:19:50 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TVYlY-0000xz-5R for guile-devel@gnu.org; Mon, 05 Nov 2012 21:19:49 -0500 Original-Received: from mail-pb0-f41.google.com ([209.85.160.41]:42732) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TVYlW-0000xj-8U; Mon, 05 Nov 2012 21:19:46 -0500 Original-Received: by mail-pb0-f41.google.com with SMTP id rq2so24733pbb.0 for ; Mon, 05 Nov 2012 18:19:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:subject:from:to:cc:date:in-reply-to:references :organization:content-type:x-mailer:mime-version :content-transfer-encoding; bh=lK5yDiltDbrrh9s2z97b4rHxsfFy6wX2B+IMTPGlE1M=; b=bPKp75NeQ754V8GCCGjXCQ62W+w5agBUK5F0gwVKI7xRzs3v5uoG6EOFjLPQxFV0Xn /pMkbpHf0zbH2573JZxvuxeuk/+/Mfi9Q8Sidt8Nocqlf8QB2urnyPbzS4B21Gi+4v5g +nRIcG8rri4oseXbp9Id68p/BjWNQS8RBT9GIvomuExiVS0h/oBkL4qTCyYxE+C2xFg5 AmJZRklAdwygA3qfYaO5O/lgi/QBRV8gOMzfBisCHXsicEB78jU30heoqqs7w09TosDH V8o3i0YGMvwFZmeMfCyblX1v/eUW5EQQl/HzzKZK5cvSLsVo4axcsMNtDBH9brKP1qFi Ceyw== Original-Received: by 10.68.232.2 with SMTP id tk2mr36492943pbc.92.1352168385447; Mon, 05 Nov 2012 18:19:45 -0800 (PST) Original-Received: from [147.2.147.112] ([61.14.130.226]) by mx.google.com with ESMTPS id wf8sm11463168pbc.65.2012.11.05.18.19.42 (version=SSLv3 cipher=OTHER); Mon, 05 Nov 2012 18:19:44 -0800 (PST) In-Reply-To: <87liefsqe0.fsf@gnu.org> X-Mailer: Evolution 3.4.4 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 209.85.160.41 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:15102 Archived-At: hey guys! Should we consider add our specified name, like: .gl or just .guile? On Mon, 2012-11-05 at 22:15 +0100, Ludovic Courtès wrote: > 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’d 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", 0x7fff1f4aec20) = -1 ENOENT (No such file or directory) > stat("/home/ludo/.local/share/guile/site/2.0/ice-9/control", 0x7fff1f4aec20) = -1 ENOENT (No such file or directory) > stat("/home/ludo/soft/share/guile/site/2.0/ice-9/control.scm", 0x7fff1f4aec20) = -1 ENOENT (No such file or directory) > stat("/home/ludo/soft/share/guile/site/2.0/ice-9/control", 0x7fff1f4aec20) = -1 ENOENT (No such file or directory) > stat("/home/ludo/soft/share/guile/2.0/ice-9/control.scm", {st_mode=S_IFREG|0644, st_size=2961, ...}) = 0 > > (I’m 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’. >