From xemacs-m  Tue Mar 25 01:17:46 1997
Received: from macon.informatik.uni-tuebingen.de (macon2.Informatik.Uni-Tuebingen.De [134.2.13.2])
	by xemacs.org (8.8.5/8.8.5) with ESMTP id BAA11272
	for <xemacs-beta@xemacs.org>; Tue, 25 Mar 1997 01:17:45 -0600 (CST)
Received: from modas.informatik.uni-tuebingen.de (modas.Informatik.Uni-Tuebingen.De [134.2.12.3]) by macon.informatik.uni-tuebingen.de (8.8.4/8.8.3/AIX-4.1/WSI-1.0) with SMTP id IAA29622 for <xemacs-beta@xemacs.org>; Tue, 25 Mar 1997 08:17:44 +0100
Received: by modas.informatik.uni-tuebingen.de (AIX 4.1/UCB 5.64/4.03)
          id AA18212; Tue, 25 Mar 1997 08:17:42 +0100
Sender: sperber@informatik.uni-tuebingen.de
To: xemacs-beta@xemacs.org
Subject: Re: file-remote-p broken
References: <y9lvi6h75ym.fsf@modas.informatik.uni-tuebingen.de> <m2hgi0a75r.fsf@altair.xemacs.org>
Mime-Version: 1.0 (generated by tm-edit 7.106)
Content-Type: text/plain; charset=US-ASCII
From: sperber@informatik.uni-tuebingen.de (Michael Sperber [Mr. Preprocessor])
Date: 25 Mar 1997 08:17:41 +0100
In-Reply-To: Steven L Baur's message of 24 Mar 1997 21:10:24 -0800
Message-Id: <y9lend4784q.fsf@modas.informatik.uni-tuebingen.de>
Lines: 38
X-Mailer: Gnus v5.4.33/XEmacs 20.1(beta9)

>>>>> "sb" == Steven L Baur <steve@miranova.com> writes:

sb> I'm trying to get this patch squared away before applying, bear with
sb> me.

You're right.

sb> Michael Sperber writes:

>> At least in 20.1-b9.  (Has noone used this?)  Its interface is also a
>> bit too generous.  I hope this applies to 19.15, also.  My next 19.15
>> beta arrives in three hours ...

>> ;; Suggested by Michael Kifer <kifer@CS.SunySB.EDU>
>> ! (defun file-remote-p (file-name)
>> !   "Test whether FILE-NAME resides on the local system.

>> The special value 'unknown is returned if no remote file access package
>> has been loaded."

sb> Michael K. has already pointed out those lines no longer apply.

>> !   (cond ((featurep 'ange-ftp) (and (ange-ftp-ftp-path file-name) t))
>> (t (require 'efs)
sb>                         ^^^^

sb> Shouldn't this be 'efs-auto?

Actually, it should be 'efs-cu.  Really actually, efs-ftp-path should
autoload.  My workspace (among 1 or 2 other lines of autoload
additions) has this already.

sb> And we should probably add a guard on `allow-remote-paths' before
sb> doing it.

Yes.

Cheers =8-} Mike

