Gnus development mailing list
 help / color / mirror / Atom feed
From: Reiner Steib <reinersteib+gmane@imap.cc>
To: ding@gnus.org, emacs-devel@gnu.org
Subject: Re: Problems with `require' in recent changes
Date: Wed, 12 Dec 2007 21:45:32 +0100	[thread overview]
Message-ID: <v9ir33vdzn.fsf@marauder.physik.uni-ulm.de> (raw)
In-Reply-To: <jwvr6hseyrd.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Wed, 12 Dec 2007 10:09:29 -0500")

On Wed, Dec 12 2007, Stefan Monnier wrote:

>> Now only password-cache.el exists in the Gnus trunk.  However,
>> password.el is still in ecrypto that is an official XEmacs Lisp
>> package.  

I guess even when XEmacs renames password.el as well, there is problem
when a user installs a new Gnus package but has an elder ecrypto
package.

If XEmacs renames it, the following code in `tramp.el' doesn't DTRT:

;; As long as password.el is not part of (X)Emacs, it shouldn't
;; be mandatory
(if (featurep 'xemacs)
    (load "password" 'noerror)
  (or (require 'password-cache nil 'noerror)
      (require 'password nil 'noerror))) ; from No Gnus, also in tar ball

>> So, it will override password-cache.el if some program requires the
>> feature `password'.  I'm not sure what it brings, but it seems that
>> password-cache.el had better provide also the feature `password'.
>> Though it might cause a trouble in programs that use password
>> functions, I guess it is easy to be solved.
>
> I really wish we had kept the name "password.el".  Such name changes are
> just a waste of time and energy.

I agree.

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

  reply	other threads:[~2007-12-12 20:45 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-11 12:04 Dave Goldberg
2007-12-12  3:12 ` Katsumi Yamaoka
2007-12-12  3:15   ` Glenn Morris
2007-12-12  5:39     ` Katsumi Yamaoka
2007-12-12 15:09       ` Stefan Monnier
2007-12-12 20:45         ` Reiner Steib [this message]
2007-12-14  0:07           ` Katsumi Yamaoka
2007-12-16 18:32             ` Reiner Steib
2007-12-16 20:22               ` Miles Bader
2007-12-16 20:51                 ` Reiner Steib
2007-12-17  2:25                   ` Glenn Morris
2007-12-17 18:44                     ` Glenn Morris
2007-12-18  0:29                       ` Katsumi Yamaoka
2007-12-28 19:34                         ` Reiner Steib
2007-12-17 21:45                   ` Dave Goldberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=v9ir33vdzn.fsf@marauder.physik.uni-ulm.de \
    --to=reinersteib+gmane@imap.cc \
    --cc=Reiner.Steib@gmx.de \
    --cc=ding@gnus.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 NNTP newsgroup(s).