Gnus development mailing list
 help / color / mirror / Atom feed
From: David Engster <deng@randomsample.de>
To: ding@gnus.org
Subject: Re: auth-source.el and eieio
Date: Tue, 15 Feb 2011 09:41:54 +0100	[thread overview]
Message-ID: <m2r5b9lmel.fsf@randomsample.de> (raw)
In-Reply-To: <87ei79d8fd.fsf@marauder.physik.uni-ulm.de> (Reiner Steib's message of "Tue, 15 Feb 2011 09:11:02 +0100")

Reiner Steib writes:
> On Mon, Feb 14 2011, Ted Zlatanov wrote:
>
>> On Mon, 14 Feb 2011 14:20:53 -0800 Lars Ingebrigtsen <larsi@gnus.org> wrote: 
>>
>> LI> Ted Zlatanov <tzz@lifelogs.com> writes:
>>>> OK, so we can't drop 23.1 and older.  Are we bundling EIEIO with Gnus or
>>>> not?  If not, I'll fix auth-source.el to degrade semi-gracefully.
>>
>> LI> Would bundling EIEIO with Gnus have any other negative impact?  
>> LI> That is, are there incompatibilities in EIEIO itself that may
>> LI> freak out various XEmacs/Emacs instances that may (or may not)
>> LI> have other versions of EIEIO installed?
>>
>> LI> If this can be done cleanly (i.e., a lib directory that dgnushack.el
>> LI> figures out whether to add to the load path (or not)), then I think
>> LI> including it would be the way to go.
>
> It is likely to lead to load-path shadows with weird effects.
> [ Just imagine compiling Gnus from git on a system with several Emacs
> versions (e.g. Debian-based systems).  The user will probably compile
> with the oldest Emacs and run the compiled version of Gnus with all
> versions.  With the newer Emacs, Gnus' EIEIO files will probably
> shadow the newer ones from Emacs. ]

Installing an Emacs package behind the user's back is never a good
idea. Can't Gnus determine at run-time whether it has to put its own
EIEIO in the load-path or not?

>> That would work for me, assuming there are no version issues.  But I
>> don't know enough about EIEIO to say.  Reiner, WDYT?  Anyone else?
>
> I don't object, but it may lead to trouble.

I've PMed Eric and he doesn't see much of a problem. EIEIO hasn't seen
significant changes in roughly two years, so it's pretty stable. He's
fine with whatever we do with it.

-David



  reply	other threads:[~2011-02-15  8:41 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-14  1:25 Katsumi Yamaoka
2011-02-14  1:45 ` Lars Ingebrigtsen
2011-02-14 15:33   ` Ted Zlatanov
2011-02-14 15:40     ` Richard Riley
2011-02-14 16:49       ` Ted Zlatanov
2011-02-14 17:39         ` David Engster
2011-02-14 18:02           ` Ted Zlatanov
2011-02-14 20:16             ` Lars Ingebrigtsen
2011-02-14 20:44               ` David Engster
2011-02-14 20:45               ` Reiner Steib
2011-02-14 21:09                 ` Lars Ingebrigtsen
2011-02-14 22:01                   ` Steinar Bang
2011-02-16  9:20                   ` Reiner Steib
2011-02-16 19:10                     ` Ted Zlatanov
2011-02-16 20:59                       ` David Engster
2011-02-14 21:07               ` Ted Zlatanov
2011-02-14 21:18                 ` Lars Ingebrigtsen
2011-02-14 21:26                   ` David Engster
2011-02-14 21:39                     ` Ted Zlatanov
2011-02-14 21:53                       ` David Engster
2011-02-14 21:57                         ` Lars Ingebrigtsen
2011-02-14 22:07                           ` Ted Zlatanov
2011-02-14 22:20                             ` Lars Ingebrigtsen
2011-02-14 22:33                               ` Ted Zlatanov
2011-02-15  8:11                                 ` Reiner Steib
2011-02-15  8:41                                   ` David Engster [this message]
2011-02-15 13:43                                   ` Ted Zlatanov
2011-02-15 14:11                                     ` Steinar Bang
2011-02-16 20:28                                     ` Reiner Steib
2011-02-16 20:50                                       ` Ted Zlatanov
2011-02-16 22:16                                         ` Ted Zlatanov
2011-02-17  0:31                                           ` Katsumi Yamaoka
2011-02-17  0:40                                             ` Matt Lundin
2011-02-17 14:02                                           ` Steinar Bang
2011-02-17 17:06                                             ` Ted Zlatanov
2011-02-17 18:46                                               ` Steinar Bang
2011-02-17 22:01                                                 ` Steinar Bang
2011-02-17 22:39                                                   ` Lars Ingebrigtsen
2011-02-18  8:48                                                     ` Steinar Bang
2011-02-18 10:52                                                       ` Steinar Bang
2011-02-18 12:22                                                         ` Steinar Bang
2011-02-18 20:26                                                     ` Ted Zlatanov
2011-02-18 21:33                                                       ` Charles Philip Chan
2011-02-19 18:00                                                         ` Charles Philip Chan
2011-02-21 16:53                                                           ` Charles Philip Chan
2011-02-18 22:11                                                       ` Lars Ingebrigtsen
2011-02-22 21:53                                                         ` Ted Zlatanov
2011-02-14 22:46                               ` David Engster
2011-02-15  8:37                             ` Steinar Bang
2011-02-14 22:18                           ` David Engster
2011-02-14 22:04                   ` Steinar Bang
2011-02-15  8:13                     ` Jochen Hein
2011-02-16 20:15                       ` Jochen Hein
2011-02-14  9:49 ` Michael Albinus
2011-02-14 15:26   ` Ted Zlatanov
2011-02-14 15:47     ` Michael Albinus
2011-02-14 16:54       ` Ted Zlatanov

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=m2r5b9lmel.fsf@randomsample.de \
    --to=deng@randomsample.de \
    --cc=ding@gnus.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).