From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Aidan Gauland Newsgroups: gmane.emacs.diffs,gmane.emacs.devel Subject: Re: /srv/bzr/emacs/trunk r112148: Added eshell-tramp module Date: Thu, 28 Mar 2013 21:18:55 +1300 Message-ID: <87hajw6iog.fsf@no8wireless.co.nz> References: <87wqst2vw3.fsf@no8wireless.co.nz> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Trace: ger.gmane.org 1364458773 4879 80.91.229.3 (28 Mar 2013 08:19:33 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Thu, 28 Mar 2013 08:19:33 +0000 (UTC) Cc: emacs-diffs@gnu.org, emacs-devel@gnu.org To: Stefan Monnier Original-X-From: emacs-diffs-bounces+gnu-emacs-diffs=m.gmane.org@gnu.org Thu Mar 28 09:19:59 2013 Return-path: Envelope-to: gnu-emacs-diffs@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 1UL83y-0003Iw-CN for gnu-emacs-diffs@m.gmane.org; Thu, 28 Mar 2013 09:19:58 +0100 Original-Received: from localhost ([::1]:37767 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UL83a-0004Zp-8F for gnu-emacs-diffs@m.gmane.org; Thu, 28 Mar 2013 04:19:34 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:56544) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UL83U-0004ZN-MC for emacs-diffs@gnu.org; Thu, 28 Mar 2013 04:19:31 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UL83R-000633-9r for emacs-diffs@gnu.org; Thu, 28 Mar 2013 04:19:28 -0400 Original-Received: from jersey.rurallink.co.nz ([114.134.15.197]:44991 helo=smtp.rurallink.co.nz) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UL83G-0005zz-VJ; Thu, 28 Mar 2013 04:19:15 -0400 Original-Received: from 114-134-6-169.rurallink.co.nz ([114.134.6.169] helo=pad8) by smtp.rurallink.co.nz with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72) (envelope-from ) id 1UL834-00020s-R6; Thu, 28 Mar 2013 21:19:04 +1300 X-Rurallink-Authenticated-User: aidalgol@no8wireless.co.nz X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 114.134.15.197 X-BeenThere: emacs-diffs@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Mailing list for Emacs changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-diffs-bounces+gnu-emacs-diffs=m.gmane.org@gnu.org Original-Sender: emacs-diffs-bounces+gnu-emacs-diffs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.diffs:120062 gmane.emacs.devel:158342 Archived-At: Stefan Monnier writes: > So IIUC the way it's enabled is by loading the package and then calling > eshell-tramp-initialize. I think the modules' `initialize' functions are called by whatever part of Eshell loads the modules, but I'm not certain. This is one of the yet-to-be-documented parts of Eshell. > This said, IIUC just loading eshell-tramp.el will also enable it (tho > not "completely"). That's a problem (and yes, this is not a new > problem, it's just a problem in most/all eshell modules). Yes, if the feature `em-tramp' is loaded but the module is not enabled, then it is just taking up memory and doing nothing. And, yes, this is a problem with Eshell modules in general, and ERC modules, from what I can tell. I really have no idea what a better solution would be. (I CC'd this message to emacs-devel, because it seems more general now and less about one particular commit.) --Aidan