Gnus development mailing list
 help / color / mirror / Atom feed
* pop3.el itegration with netrc.el
@ 2004-10-13 15:44 Ted Zlatanov
  2004-10-14 18:21 ` Simon Josefsson
  0 siblings, 1 reply; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-13 15:44 UTC (permalink / raw)


pop3.el seems to be separate from the rest of Gnus, so I'm not sure if
I should add netrc.el (authinfo) support to it - then pop3.el would
also depend on gnus-encrypt.el.  Please advise.

Thanks
Ted



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-13 15:44 pop3.el itegration with netrc.el Ted Zlatanov
@ 2004-10-14 18:21 ` Simon Josefsson
  2004-10-14 18:37   ` Ted Zlatanov
  2004-10-14 18:49   ` Reiner Steib
  0 siblings, 2 replies; 25+ messages in thread
From: Simon Josefsson @ 2004-10-14 18:21 UTC (permalink / raw)


"Ted Zlatanov" <tzz@lifelogs.com> writes:

> pop3.el seems to be separate from the rest of Gnus, so I'm not sure if
> I should add netrc.el (authinfo) support to it - then pop3.el would
> also depend on gnus-encrypt.el.  Please advise.

There are many incompatible forks of pop3.el, the copy in Gnus CVS
used to be one of them.  I see now that the copies in Gnus CVS and
Emacs CVS are in sync.  I'm not sure how that happened, but it is a
good thing.  So perhaps we can make this one the canonical copy, and
add new features to it.  I'd say go ahead.

Btw, is it difficult to modularize gnus-encrypt.el into, say,
encrypt.el, that doesn't use a lot of Gnus functions?  I recall some
discussions about that, though.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 18:21 ` Simon Josefsson
@ 2004-10-14 18:37   ` Ted Zlatanov
  2004-10-14 18:56     ` Simon Josefsson
  2004-10-14 18:49   ` Reiner Steib
  1 sibling, 1 reply; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-14 18:37 UTC (permalink / raw)
  Cc: ding, emacs-devel

On Thu, 14 Oct 2004, jas@extundo.com wrote:

> "Ted Zlatanov" <tzz@lifelogs.com> writes:
> 
>> pop3.el seems to be separate from the rest of Gnus, so I'm not sure if
>> I should add netrc.el (authinfo) support to it - then pop3.el would
>> also depend on gnus-encrypt.el.  Please advise.
> 
> There are many incompatible forks of pop3.el, the copy in Gnus CVS
> used to be one of them.  I see now that the copies in Gnus CVS and
> Emacs CVS are in sync.  I'm not sure how that happened, but it is a
> good thing.  So perhaps we can make this one the canonical copy, and
> add new features to it.  I'd say go ahead.

I've cc-ed the emacs-devel list in case anyone there has suggestions.
This is regarding pop3.el and its use of netrc (~/.authinfo) files.

> Btw, is it difficult to modularize gnus-encrypt.el into, say,
> encrypt.el, that doesn't use a lot of Gnus functions?  I recall some
> discussions about that, though.

This may be a good time to make that break.  The current
gnus-encrypt.el can be turned into encrypt.el with very few changes
except for the dependency on password.el, which is in Gnus.
gnus-encrypt.el will remain as a wrapper for Gnus, using password.el
and allowing closer integration with other Gnus libraries.  This is
not the real problem, though.

The root problem is that netrc.el is a part of Gnus I can't break out
(I think), so the Gnus version of pop3.el will have a dependency we
don't want.  Maybe we should have gnus-pop3.el to implement what I'm
proposing, so pop3.el remains clean of Gnus dependencies?

Ted



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 18:21 ` Simon Josefsson
  2004-10-14 18:37   ` Ted Zlatanov
@ 2004-10-14 18:49   ` Reiner Steib
  2004-10-14 19:01     ` Simon Josefsson
  1 sibling, 1 reply; 25+ messages in thread
From: Reiner Steib @ 2004-10-14 18:49 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 1649 bytes --]

On Thu, Oct 14 2004, Simon Josefsson wrote:

> "Ted Zlatanov" <tzz@lifelogs.com> writes:
>
>> pop3.el seems to be separate from the rest of Gnus, so I'm not sure if
>> I should add netrc.el (authinfo) support to it - then pop3.el would
>> also depend on gnus-encrypt.el.  Please advise.
>
> There are many incompatible forks of pop3.el, the copy in Gnus CVS
> used to be one of them.  I see now that the copies in Gnus CVS and
> Emacs CVS are in sync.  I'm not sure how that happened, but it is a
> good thing.

I've merged the changes from Emacs to Gnus initially and Miles keeps
them in sync now.  The most important thing was
`pop3-leave-mail-on-server'.  I'm not sure if this feature (introduced
in Emacs) was a good idea in the first place (and I don't know if it
works reliably [1]).  But if upgrading from Gnus 5.9 to 5.11 would
suddenly delete the mails on the server after pop (when
`pop3-leave-mail-on-server' is ignored as in Gnus 5.10.6), people
won't be happy ("Gnus deleted all my mail!!!1").

> So perhaps we can make this one the canonical copy, and add new
> features to it.  I'd say go ahead.

Dave Love pointed out that `pop3.el' could be used outside Gnus and
suggested to remove the nn-whatever dependencies.  If we think that
this is a valuable goal, we should avoid adding further dependencies.

,----[ http://thread.gmane.org/gmane.emacs.pretest.bugs/3974 ]
| pop3.el shouldn't use nnheader-accept-process-output, so it can be
| used outside Gnus.
`----

Bye, Reiner.

[1] I think I'll add a warning, see the attached patch.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: rs-pop-leave.patch --]
[-- Type: text/x-patch, Size: 2058 bytes --]

Index: lisp/pop3.el
===================================================================
RCS file: /usr/local/cvsroot/gnus/lisp/pop3.el,v
retrieving revision 6.23.2.2
diff -u -r6.23.2.2 pop3.el
--- lisp/pop3.el	22 Aug 2004 21:48:11 -0000	6.23.2.2
+++ lisp/pop3.el	30 Aug 2004 16:43:52 -0000
@@ -55,7 +55,14 @@
 values are 'apop.")
 
 (defvar pop3-leave-mail-on-server nil
-  "*Non-nil if the mail is to be left on the POP server after fetching.")
+  "*Non-nil if the mail is to be left on the POP server after fetching.
+
+If the `pop3-leave-mail-on-server' is non-`nil' the mail is to be
+left on the POP server after fetching.  Note that POP servers
+maintain no state information between sessions, so what the
+client believes is there and what is actually there may not match
+up.  If they do not, then the whole thing can fall apart and
+leave you with a corrupt mailbox.")
 
 (defvar pop3-timestamp nil
   "Timestamp returned when initially connected to the POP server.
Index: texi/gnus.texi
===================================================================
RCS file: /usr/local/cvsroot/gnus/texi/gnus.texi,v
retrieving revision 6.603.2.4
diff -u -r6.603.2.4 gnus.texi
--- texi/gnus.texi	26 Aug 2004 09:59:57 -0000	6.603.2.4
+++ texi/gnus.texi	30 Aug 2004 16:43:53 -0000
@@ -13528,10 +13528,16 @@
 
 @end table
 
+@vindex pop3-movemail
+@vindex pop3-leave-mail-on-server
 If the @code{:program} and @code{:function} keywords aren't specified,
 @code{pop3-movemail} will be used.  If the
 @code{pop3-leave-mail-on-server} is non-@code{nil} the mail is to be
-left on the POP server after fetching.
+left on the @acronym{POP} server after fetching when using
+@code{pop3-movemail}.  Note that POP servers maintain no state
+information between sessions, so what the client believes is there and
+what is actually there may not match up.  If they do not, then the whole
+thing can fall apart and leave you with a corrupt mailbox.
 
 Here are some examples.  Fetch from the default @acronym{POP} server,
 using the default user name, and default fetcher:

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 18:37   ` Ted Zlatanov
@ 2004-10-14 18:56     ` Simon Josefsson
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Josefsson @ 2004-10-14 18:56 UTC (permalink / raw)
  Cc: emacs-devel

"Ted Zlatanov" <tzz@lifelogs.com> writes:

>> Btw, is it difficult to modularize gnus-encrypt.el into, say,
>> encrypt.el, that doesn't use a lot of Gnus functions?  I recall some
>> discussions about that, though.
>
> This may be a good time to make that break.  The current
> gnus-encrypt.el can be turned into encrypt.el with very few changes
> except for the dependency on password.el, which is in Gnus.

We can install password.el in Emacs.

> The root problem is that netrc.el is a part of Gnus I can't break out
> (I think)

Why not?  Doing so seem like a good idea, to me.  netrc.el doesn't
call any Gnus functions, except for gnus-encrypt, as far as I can
tell.  And if we (you) move gnus-encrypt.el to encrypt.el, all will be
fine, I think.

> so the Gnus version of pop3.el will have a dependency we don't want.

pop3.el would depend on netrc.el, which would depend on encrypt.el,
which would depend on password.el.  All small files, and no Gnus
dependencies.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 18:49   ` Reiner Steib
@ 2004-10-14 19:01     ` Simon Josefsson
  2004-10-14 19:52       ` Reiner Steib
  0 siblings, 1 reply; 25+ messages in thread
From: Simon Josefsson @ 2004-10-14 19:01 UTC (permalink / raw)


Reiner Steib <reinersteib+gmane@imap.cc> writes:

>> So perhaps we can make this one the canonical copy, and add new
>> features to it.  I'd say go ahead.
>
> Dave Love pointed out that `pop3.el' could be used outside Gnus and
> suggested to remove the nn-whatever dependencies.  If we think that
> this is a valuable goal, we should avoid adding further dependencies.
>
> ,----[ http://thread.gmane.org/gmane.emacs.pretest.bugs/3974 ]
> | pop3.el shouldn't use nnheader-accept-process-output, so it can be
> | used outside Gnus.
> `----

I agree, but I wouldn't consider netrc.el, password.el nor encrypt.el
as something part of Gnus, even though they technically are.  They are
(or will be) small standalone libraries.

But the nn* dependency is bad, using nn* will lead to loading parts of
Gnus.  Can't we write a pop3-accept-process-output, and drop the
nnheader dependency?  Seems simple.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 19:01     ` Simon Josefsson
@ 2004-10-14 19:52       ` Reiner Steib
  2004-10-14 22:39         ` Simon Josefsson
  0 siblings, 1 reply; 25+ messages in thread
From: Reiner Steib @ 2004-10-14 19:52 UTC (permalink / raw)


On Thu, Oct 14 2004, Simon Josefsson wrote:

> I agree, but I wouldn't consider netrc.el, password.el nor encrypt.el
> as something part of Gnus, even though they technically are.  They are
> (or will be) small standalone libraries.

Ah, sorry.  I didn't know that `[gnus-]encrypt.el' (Ted wrote "pop3.el
would also depend on gnus-encrypt.el") has no real Gnus dependencies.

> But the nn* dependency is bad, using nn* will lead to loading parts of
> Gnus.  Can't we write a pop3-accept-process-output, and drop the
> nnheader dependency?  Seems simple.

I didn't look at it in detail.  We'd probably have to duplicate
`nnheader-accept-process-output' and the strange
`nnheader-read-timeout' thingie.  I would be glad if you could fix
this.

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




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 19:52       ` Reiner Steib
@ 2004-10-14 22:39         ` Simon Josefsson
  2004-10-15 17:08           ` Ted Zlatanov
  2004-10-15 17:48           ` pop3.el itegration with netrc.el Reiner Steib
  0 siblings, 2 replies; 25+ messages in thread
From: Simon Josefsson @ 2004-10-14 22:39 UTC (permalink / raw)


Reiner Steib <reinersteib+gmane@imap.cc> writes:

>> But the nn* dependency is bad, using nn* will lead to loading parts of
>> Gnus.  Can't we write a pop3-accept-process-output, and drop the
>> nnheader dependency?  Seems simple.
>
> I didn't look at it in detail.  We'd probably have to duplicate
> `nnheader-accept-process-output' and the strange
> `nnheader-read-timeout' thingie.  I would be glad if you could fix
> this.

Done.  I hope it will be merged automatically to Emacs CVS (that
gateway is quite neat, btw!).

Perhaps emacs/lisp/gnus/pop3.el should be moved to emacs/lisp/net/?
People seem to be afraid of using files from gnus/.  And now pop3.el
is standalone.  What do you think?




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 22:39         ` Simon Josefsson
@ 2004-10-15 17:08           ` Ted Zlatanov
  2004-10-15 17:43             ` Simon Josefsson
  2004-10-15 18:07             ` Reiner Steib
  2004-10-15 17:48           ` pop3.el itegration with netrc.el Reiner Steib
  1 sibling, 2 replies; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-15 17:08 UTC (permalink / raw)
  Cc: ding

[-- Attachment #1: Type: text/plain, Size: 516 bytes --]

I'm attaching the new encrypt.el (mostly gnus-encrypt.el minus cruft
and with a search&replace done) and the related netrc.el (just
search&replace of gnus-encrypt with encrypt).

Let me know if it looks OK before I put it into Gnus.  gnus-encrypt.el
will become a shallow wrapper with no options for now if this
conversion happens.

If password.el, netrc.el, encrypt.el, and pop3.el move to Emacs,
that's OK but I don't have CVS access to Emacs source code so it will
be harder for me to maintain those files.

Ted


[-- Attachment #2: netrc.el --]
[-- Type: application/emacs-lisp, Size: 6523 bytes --]

[-- Attachment #3: encrypt.el --]
[-- Type: application/emacs-lisp, Size: 9201 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:08           ` Ted Zlatanov
@ 2004-10-15 17:43             ` Simon Josefsson
  2004-10-15 19:07               ` Ted Zlatanov
  2004-10-15 19:16               ` Reiner Steib
  2004-10-15 18:07             ` Reiner Steib
  1 sibling, 2 replies; 25+ messages in thread
From: Simon Josefsson @ 2004-10-15 17:43 UTC (permalink / raw)


"Ted Zlatanov" <tzz@lifelogs.com> writes:

> I'm attaching the new encrypt.el (mostly gnus-encrypt.el minus cruft
> and with a search&replace done) and the related netrc.el (just
> search&replace of gnus-encrypt with encrypt).
>
> Let me know if it looks OK before I put it into Gnus.

Looks fine (untested though).

Perhaps there should be some autoload cookies in encrypt.el, to avoid
the autoload declarations inside netrc.el.

> gnus-encrypt.el will become a shallow wrapper with no options for
> now if this conversion happens.

Would gnus-encrypt.el still be useful?

> If password.el, netrc.el, encrypt.el, and pop3.el move to Emacs,
> that's OK but I don't have CVS access to Emacs source code so it will
> be harder for me to maintain those files.

I'd say we can maintain them in Gnus CVS, and sync the version in
Emacs from time to time.

When you have installed the files in Gnus HEAD, I'll write a proposal
to install them in Emacs CVS on emacs-devel, and install them if
nobody object.  Since it is new functionality, I'm assuming they
shouldn't go into Gnus 5.10.  Hopefully by moving the files away from
emacs/lisp/gnus/ the auto-sync gateway won't merge the files back into
Gnus 5.10.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-14 22:39         ` Simon Josefsson
  2004-10-15 17:08           ` Ted Zlatanov
@ 2004-10-15 17:48           ` Reiner Steib
  2004-10-15 17:58             ` Simon Josefsson
  2004-10-17 23:22             ` Miles Bader
  1 sibling, 2 replies; 25+ messages in thread
From: Reiner Steib @ 2004-10-15 17:48 UTC (permalink / raw)


[ Resending this message.  The first one didn't make it to the list,
  it seems.  At least it's not on Gmane.  Sorry for any duplicates. ]

On Fri, Oct 15 2004, Simon Josefsson wrote:

> Done.

Thanks.

> I hope it will be merged automatically to Emacs CVS

When installed in v5-10 (i.e. if it is a bug fix), it should be
automatically synced to Gnus trunk and to Emacs.

> (that gateway is quite neat, btw!).

ACK.  Miles also syncs from Emacs to Gnus, but he said that Gnus ->
Emacs is easier for him.  I don't know if it is inconvenient for Miles
if we install a change in v5-10 and trunk at the same time.

Maybe we should have MAINTAINANCE file (or README.developer) in Gnus
CVS to explain these non-obvious things, WDYT?

Other items in this file could be:

- ./etc/gnus-news.texi -> make -> GNUS-NEWS

- Avoiding dependencies:

  - message / gnus

  - nn*

  - mm*

  - rfc*

  - tla netrc pop3 dig dns ...

- Compatibility with XEmacs and older Emacs versions in v5-10 and in
  the trunk.

> Perhaps emacs/lisp/gnus/pop3.el should be moved to emacs/lisp/net/?
> People seem to be afraid of using files from gnus/.  And now pop3.el
> is standalone.  What do you think?

I'd agree.  Also for some other files: dig.el dns.el ...  If I recall
the discussions on emacs-devel correctly, this implies that the file
would be officially maintained by the Emacs developers.  Thus Richard
has to decide about it.  Please suggest the moving on emacs-devel.

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



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:48           ` pop3.el itegration with netrc.el Reiner Steib
@ 2004-10-15 17:58             ` Simon Josefsson
  2004-10-15 18:55               ` Reiner Steib
  2004-10-17 23:22             ` Miles Bader
  1 sibling, 1 reply; 25+ messages in thread
From: Simon Josefsson @ 2004-10-15 17:58 UTC (permalink / raw)


Reiner Steib <reinersteib+gmane@imap.cc> writes:

> ACK.  Miles also syncs from Emacs to Gnus, but he said that Gnus ->
> Emacs is easier for him.  I don't know if it is inconvenient for Miles
> if we install a change in v5-10 and trunk at the same time.
>
> Maybe we should have MAINTAINANCE file (or README.developer) in Gnus
> CVS to explain these non-obvious things, WDYT?

Yes, please.  I know I will forget about them unless they are written
down somewhere.

> Other items in this file could be:

Neat.  Looks like a "Gnus Coding Style" file, which indeed would be
useful.

>> Perhaps emacs/lisp/gnus/pop3.el should be moved to emacs/lisp/net/?
>> People seem to be afraid of using files from gnus/.  And now pop3.el
>> is standalone.  What do you think?
>
> I'd agree.  Also for some other files: dig.el dns.el ...  If I recall
> the discussions on emacs-devel correctly, this implies that the file
> would be officially maintained by the Emacs developers.  Thus Richard
> has to decide about it.  Please suggest the moving on emacs-devel.

But the files are in emacs/lisp/gnus, so in theory that already means
they are maintained by the Emacs developers?

dns.el has a mm-util dependency, but dig.el should be moved to
emacs/lisp/net/.  Any other files?  Perhaps: starttls.el, sha1.el,
sieve*.el, format-spec.el.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:08           ` Ted Zlatanov
  2004-10-15 17:43             ` Simon Josefsson
@ 2004-10-15 18:07             ` Reiner Steib
  2004-10-15 19:14               ` encrypt.el (was: pop3.el itegration with netrc.el) Ted Zlatanov
  1 sibling, 1 reply; 25+ messages in thread
From: Reiner Steib @ 2004-10-15 18:07 UTC (permalink / raw)


On Fri, Oct 15 2004, Ted Zlatanov wrote:

> If password.el, netrc.el, encrypt.el, and pop3.el move to Emacs,
> that's OK but I don't have CVS access to Emacs source code so it will
> be harder for me to maintain those files.

netrc.el has already been "moved to Emacs" a long time ago.  This
doesn't mean that you cannot maintain the files anymore.  Changes in
v5-10/lisp/netrc.el should still be merged into Emacs (follow the URL
for more):

,----[ http://thread.gmane.org/20041010225828.GA31829@fencepost ]
| From: Miles Bader <miles@lsi.nec.co.jp>
| Subject: Re: [rmail-mbox-branch]: mail-utils
| Date: Tue, 12 Oct 2004 11:09:11 +0900
| Message-ID: <buoy8ick7qw.fsf@mctpc71.ucom.lsi.nec.co.jp>
| 
| Richard Stallman <rms@gnu.org> writes:
| >     Do you mean that a patch from a gnus maintainer would have to be
| >     posted to the mailing list to be put into emacs (unlike Gnus
| >     changes)?
| >
| > I mean they should treat these files the same way they treat all the
| > other Lisp files that are not part of Gnus.  That does not mean ALL
| > changes have to be discussed.  They can install local uncontroversial
| > bug fixes without discussion, just as any of us would.  However,
| > beyond that, they ought to post on this list.
| 
| Ok, that sounds reasonable, and workable in practice for my Gnus-syncing.
`----

password.el and encrypt.el (tla, ...) are new in No Gnus.  So these
files will not be considered now, I think.

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




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:58             ` Simon Josefsson
@ 2004-10-15 18:55               ` Reiner Steib
  0 siblings, 0 replies; 25+ messages in thread
From: Reiner Steib @ 2004-10-15 18:55 UTC (permalink / raw)


On Fri, Oct 15 2004, Simon Josefsson wrote:

> Reiner Steib <reinersteib+gmane@imap.cc> writes:
[...]
>>> Perhaps emacs/lisp/gnus/pop3.el should be moved to emacs/lisp/net/?
>>> People seem to be afraid of using files from gnus/.  And now pop3.el
>>> is standalone.  What do you think?
>>
>> I'd agree.  Also for some other files: dig.el dns.el ...  If I recall
>> the discussions on emacs-devel correctly, this implies that the file
>> would be officially maintained by the Emacs developers.  Thus Richard
>> has to decide about it.  Please suggest the moving on emacs-devel.
>
> But the files are in emacs/lisp/gnus, so in theory that already means
> they are maintained by the Emacs developers?

In _my_ understanding, Richard considers the files in emacs/lisp/gnus
(and the corresponding manuals) as "maintained by the Gnus
developers".  If a file is useful for other Emacs packages and doesn't
depend on Gnus he want it to be moved to a different directory.  (But
I might be misunderstanding him.)

> dns.el has a mm-util dependency, but dig.el should be moved to
> emacs/lisp/net/.  Any other files?  Perhaps: starttls.el, sha1.el,
> sieve*.el, format-spec.el.

pgg*.el?

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




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:43             ` Simon Josefsson
@ 2004-10-15 19:07               ` Ted Zlatanov
  2004-10-15 19:16               ` Reiner Steib
  1 sibling, 0 replies; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-15 19:07 UTC (permalink / raw)
  Cc: ding

On Fri, 15 Oct 2004, jas@extundo.com wrote:

> Perhaps there should be some autoload cookies in encrypt.el, to
> avoid the autoload declarations inside netrc.el.

Done, I think (I didn't remove the autoload declarations in netrc.el
yet though).

All the changes are in CVS now.

> Would gnus-encrypt.el still be useful?

I don't think so, so I've mentally marked it for tentative removal.
If no complaints arise I'll remove it next week.

> I'd say we can maintain them in Gnus CVS, and sync the version in
> Emacs from time to time.
> 
> When you have installed the files in Gnus HEAD, I'll write a proposal
> to install them in Emacs CVS on emacs-devel, and install them if
> nobody object.  Since it is new functionality, I'm assuming they
> shouldn't go into Gnus 5.10.  Hopefully by moving the files away from
> emacs/lisp/gnus/ the auto-sync gateway won't merge the files back into
> Gnus 5.10.

I have a feeling a lot of people will have something to say about
encrypt.el - it's a touchy topic for a lot of developers, and it's
hard to do it right.

Ted



^ permalink raw reply	[flat|nested] 25+ messages in thread

* encrypt.el (was: pop3.el itegration with netrc.el)
  2004-10-15 18:07             ` Reiner Steib
@ 2004-10-15 19:14               ` Ted Zlatanov
       [not found]                 ` <iluwtxrvkfb.fsf@latte.josefsson.org>
  0 siblings, 1 reply; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-15 19:14 UTC (permalink / raw)
  Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 675 bytes --]

On Fri, 15 Oct 2004, reinersteib+gmane@imap.cc wrote:

> password.el and encrypt.el (tla, ...) are new in No Gnus.  So these
> files will not be considered now, I think.

encrypt.el should be general-purpose enough that it can be outside
Gnus itself.  I would like it to be part of Emacs, not just Gnus,
because encryption is important to users and Emacs doesn't come with
much built-in for handling encryption.  I'm attaching a copy, the same
(as of now) as the one in the Gnus CVS HEAD.  Can anyone comment on
the likelihood of inclusion in Emacs?

The current list of encryption methods is minimal but can be easily
expanded.  The API is IMHO more important.

Thanks
Ted


[-- Attachment #2: encrypt.el --]
[-- Type: application/emacs-lisp, Size: 9233 bytes --]

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:43             ` Simon Josefsson
  2004-10-15 19:07               ` Ted Zlatanov
@ 2004-10-15 19:16               ` Reiner Steib
  2004-10-17 23:13                 ` Miles Bader
  1 sibling, 1 reply; 25+ messages in thread
From: Reiner Steib @ 2004-10-15 19:16 UTC (permalink / raw)


On Fri, Oct 15 2004, Simon Josefsson wrote:

> I'd say we can maintain them in Gnus CVS, and sync the version in
> Emacs from time to time.
>
> When you have installed the files in Gnus HEAD, I'll write a
> proposal to install them in Emacs CVS on emacs-devel, and install
> them if nobody object.  Since it is new functionality, I'm assuming
> they shouldn't go into Gnus 5.10.

[ I haven't looked at Ted's encryption code; please disregard my remarks
if irrelevant. ]

We intended to keep "Gnus in Emacs trunk" and Gnus 5.10.x (v5-10
branch) in sync in order to avoid merging problems in the future.

If it is new functionality that will most probably not make the
current Gnus instable, Lars may agree to add it in v5-10.  The storing
of plain-text passwords in ~/.authinfo may even qualify as a bug that
needs to be fixed in the stable series.

> Hopefully by moving the files away from emacs/lisp/gnus/ the
> auto-sync gateway won't merge the files back into Gnus 5.10.

It would surely require some additional work for Miles.  It's not a
fully automated process, AFAIK.

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




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 19:16               ` Reiner Steib
@ 2004-10-17 23:13                 ` Miles Bader
  2004-10-18 18:30                   ` Ted Zlatanov
  0 siblings, 1 reply; 25+ messages in thread
From: Miles Bader @ 2004-10-17 23:13 UTC (permalink / raw)


Reiner Steib <reinersteib+gmane@imap.cc> writes:
>> Hopefully by moving the files away from emacs/lisp/gnus/ the
>> auto-sync gateway won't merge the files back into Gnus 5.10.
>
> It would surely require some additional work for Miles.  It's not a
> fully automated process, AFAIK.

The sync process ignores file location, it uses the arch "id-tags" you
can see at the bottom of each source file.  So if you move a file in one
tree, I'll have to intervene to make sure that it does/doesn't move in
the other tree (whichever is appropriate), but subsequent changes to the
moved file will still propagate.

If you want the association between files to be completely broken (I
don't think you usually do), then you can remove the arch-tag: line at
the bottom of one copy (it will get a new one automatically).  It's
_not_ desirable in general to do this; if a particular file needs
special care with regard to propagating changes, just tell me, and I'll
watch out for anything funny.

The main problem is ChangeLogs:  if you add entries to lisp/ChangeLog
(in Gnus) or lisp/gnus/ChangeLog (in Emacs), it's clear that they should
simply be propagated to the corresponding file in the other tree,
because the changes they describe almost certainly will be as well.

However, if files from Gnus are present in other Emacs directories, then
this easy 1-to-1 correspondence is lost, so it will likely require me to
manually deal with such ChangeLog entries.  This problem already exists
with texi/ChangeLog, but it's not that heavily updated, so it hasn't
proved much of burden; hopefully the bulk of Gnus changes will still
remain inside the main gnus source even if these "generic" files are
moved.

But anyway, dealing with ChangeLogs is a relatively minor hassle becuase
it's only 1 or 2 files, and if it gets too annoying I'll write a script
that deals with it.

-Miles
-- 
o The existentialist, not having a pillow, goes everywhere with the book by
  Sullivan, _I am going to spit on your graves_.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-15 17:48           ` pop3.el itegration with netrc.el Reiner Steib
  2004-10-15 17:58             ` Simon Josefsson
@ 2004-10-17 23:22             ` Miles Bader
  1 sibling, 0 replies; 25+ messages in thread
From: Miles Bader @ 2004-10-17 23:22 UTC (permalink / raw)


Reiner Steib <reinersteib+gmane@imap.cc> writes:
> I don't know if it is inconvenient for Miles if we install a change in
> v5-10 and trunk at the same time.

The automatic merge usually deals with ignoring completely identical
changes, so they're not generally a problem.  If the code actually has to
be different for the trunk and v5-10 then there will be a conflict, but
I'll usually just notice that it's essentially the same code, and drop the
merge for that hunk -- which is easier than actually having to fix the code
myself!

So far it hasn't been much of a problem; as usual (see my previous post...)
the main annoyance is ChangeLogs:  I try to eliminate redundant ChangeLog
entries, but often this takes searching around.

The main thing that would help me is if when you install in both places,
try to make sure the ChangeLog text is _identical_, as much as possible.

Thanks,

-Miles
-- 
=====
(^o^;
(()))
*This is the cute octopus virus, please copy it into your sig so it can spread.




^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-17 23:13                 ` Miles Bader
@ 2004-10-18 18:30                   ` Ted Zlatanov
  2004-10-21 18:35                     ` Ted Zlatanov
  0 siblings, 1 reply; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-18 18:30 UTC (permalink / raw)
  Cc: ding

On Mon, 18 Oct 2004, miles@gnu.org wrote:

> The main problem is ChangeLogs: if you add entries to lisp/ChangeLog
> (in Gnus) or lisp/gnus/ChangeLog (in Emacs), it's clear that they should
> simply be propagated to the corresponding file in the other tree,
> because the changes they describe almost certainly will be as well.
> 
> However, if files from Gnus are present in other Emacs directories, then
> this easy 1-to-1 correspondence is lost, so it will likely require me to
> manually deal with such ChangeLog entries.  This problem already exists
> with texi/ChangeLog, but it's not that heavily updated, so it hasn't
> proved much of burden; hopefully the bulk of Gnus changes will still
> remain inside the main gnus source even if these "generic" files are
> moved.
> 
> But anyway, dealing with ChangeLogs is a relatively minor hassle becuase
> it's only 1 or 2 files, and if it gets too annoying I'll write a script
> that deals with it.

If you could make the default add-change-log-entry-other-window to
mention the arch-tag (if available at the end of the file), I think
your life and ours will be much easier.  This would require changes to
add-log.el though.

I love the idea of a general identifier for all files in the VCS.
That makes the CVS annoyance of directory mis-management much easier
to reconcile with TLA.  Thanks for helping with all the Gnus <-> Emacs
sync issues.

Ted



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: pop3.el itegration with netrc.el
  2004-10-18 18:30                   ` Ted Zlatanov
@ 2004-10-21 18:35                     ` Ted Zlatanov
  0 siblings, 0 replies; 25+ messages in thread
From: Ted Zlatanov @ 2004-10-21 18:35 UTC (permalink / raw)
  Cc: ding

On 18 Oct 2004, tzz@lifelogs.com wrote:

> If you could make the default add-change-log-entry-other-window to
> mention the arch-tag (if available at the end of the file), I think
> your life and ours will be much easier.  This would require changes to
> add-log.el though.

Would you like me to try this or is it better raised on emacs-devel?

It seems like a very useful feature, if the arch-tags are generally
used for Emacs development.

Ted



^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: encrypt.el
       [not found]                           ` <4nr7m9j1ah.fsf@lifelogs.com>
@ 2004-12-01 22:12                             ` Reiner Steib
  2004-12-02 16:36                               ` encrypt.el Ted Zlatanov
  0 siblings, 1 reply; 25+ messages in thread
From: Reiner Steib @ 2004-12-01 22:12 UTC (permalink / raw)
  Cc: ding

On Wed, Dec 01 2004, Ted Zlatanov wrote:

> I need to clarify, however, how encrypt.el will coexist with the
> Gnus version.  Should I remove encrypt.el from Gnus, or leave it in
> as a duplicate of the Emacs tree version?

[I think you mean Gnus repository here, right?]

No Gnus is supposed to run on Emacs 21.1 (and up) and XEmacs 21.4 (and
up).  Wouldn't removing encrypt.el break running Gnus on versions
before Emacs 21.4?

Bye, Reiner.

[ Cc-ing Ding, trimming xemacs-beta ]
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: encrypt.el
  2004-12-01 22:12                             ` encrypt.el Reiner Steib
@ 2004-12-02 16:36                               ` Ted Zlatanov
  2004-12-02 20:56                                 ` encrypt.el Reiner Steib
  2004-12-07 17:50                                 ` encrypt.el Ted Zlatanov
  0 siblings, 2 replies; 25+ messages in thread
From: Ted Zlatanov @ 2004-12-02 16:36 UTC (permalink / raw)
  Cc: ding

On Wed, 01 Dec 2004, reinersteib+gmane@imap.cc wrote:

On Wed, Dec 01 2004, Ted Zlatanov wrote:
> 
>> I need to clarify, however, how encrypt.el will coexist with the
>> Gnus version.  Should I remove encrypt.el from Gnus, or leave it in
>> as a duplicate of the Emacs tree version?
> 
> [I think you mean Gnus repository here, right?]

Gnus CVS is synchronized with Emacs these days, so it would be both.

> No Gnus is supposed to run on Emacs 21.1 (and up) and XEmacs 21.4 (and
> up).  Wouldn't removing encrypt.el break running Gnus on versions
> before Emacs 21.4?

Yes, and I'll leave it in on yours and Simon Josefsson's advice.  So I
will only add encrypt.el to the Emacs lisp/ tree without removing it
from anywhere else.

I do wish that CVS supported symlinks :)

Ted

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: encrypt.el
  2004-12-02 16:36                               ` encrypt.el Ted Zlatanov
@ 2004-12-02 20:56                                 ` Reiner Steib
  2004-12-07 17:50                                 ` encrypt.el Ted Zlatanov
  1 sibling, 0 replies; 25+ messages in thread
From: Reiner Steib @ 2004-12-02 20:56 UTC (permalink / raw)
  Cc: ding

On Thu, Dec 02 2004, Ted Zlatanov wrote:

> On Wed, 01 Dec 2004, reinersteib+gmane@imap.cc wrote:
[...]
>> [I think you mean Gnus repository here, right?]
>
> Gnus CVS is synchronized with Emacs these days, so it would be both.

I think that Miles can do the syncing independent of the location, e.g.

  [v5-10]gnus/lisp/netrc.el <--> emacs/lisp/net/net/netrc.el

BTW, should `encrypt.el' go in the stable branch (v5-10), too?  Maybe
we already discussed this, but I forgot.

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

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: encrypt.el
  2004-12-02 16:36                               ` encrypt.el Ted Zlatanov
  2004-12-02 20:56                                 ` encrypt.el Reiner Steib
@ 2004-12-07 17:50                                 ` Ted Zlatanov
  1 sibling, 0 replies; 25+ messages in thread
From: Ted Zlatanov @ 2004-12-07 17:50 UTC (permalink / raw)
  Cc: ding

I'll hold off on the encrypt.el migration until after the next Emacs
release, as requested.

Thanks for all the help
Ted

^ permalink raw reply	[flat|nested] 25+ messages in thread

end of thread, other threads:[~2004-12-07 17:50 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-10-13 15:44 pop3.el itegration with netrc.el Ted Zlatanov
2004-10-14 18:21 ` Simon Josefsson
2004-10-14 18:37   ` Ted Zlatanov
2004-10-14 18:56     ` Simon Josefsson
2004-10-14 18:49   ` Reiner Steib
2004-10-14 19:01     ` Simon Josefsson
2004-10-14 19:52       ` Reiner Steib
2004-10-14 22:39         ` Simon Josefsson
2004-10-15 17:08           ` Ted Zlatanov
2004-10-15 17:43             ` Simon Josefsson
2004-10-15 19:07               ` Ted Zlatanov
2004-10-15 19:16               ` Reiner Steib
2004-10-17 23:13                 ` Miles Bader
2004-10-18 18:30                   ` Ted Zlatanov
2004-10-21 18:35                     ` Ted Zlatanov
2004-10-15 18:07             ` Reiner Steib
2004-10-15 19:14               ` encrypt.el (was: pop3.el itegration with netrc.el) Ted Zlatanov
     [not found]                 ` <iluwtxrvkfb.fsf@latte.josefsson.org>
     [not found]                   ` <E1CIozF-00034v-V7@fencepost.gnu.org>
     [not found]                     ` <4n4qkrvrwu.fsf@lifelogs.com>
     [not found]                       ` <E1CJx6h-0000Ns-QW@fencepost.gnu.org>
     [not found]                         ` <4nsm87vrgk.fsf@lifelogs.com>
     [not found]                           ` <4nr7m9j1ah.fsf@lifelogs.com>
2004-12-01 22:12                             ` encrypt.el Reiner Steib
2004-12-02 16:36                               ` encrypt.el Ted Zlatanov
2004-12-02 20:56                                 ` encrypt.el Reiner Steib
2004-12-07 17:50                                 ` encrypt.el Ted Zlatanov
2004-10-15 17:48           ` pop3.el itegration with netrc.el Reiner Steib
2004-10-15 17:58             ` Simon Josefsson
2004-10-15 18:55               ` Reiner Steib
2004-10-17 23:22             ` Miles Bader

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).