Pterodactyl Gnus v0.91 is released

Pterodactyl Gnus v0.91 is released

[Lars Magne Ingebrigtsen]

MIME variable cleanup and agent active fixes.

Get it from <URL:http://quimby.gnus.org/gnus/dist/pgnus.tar.gz>
or "/ftp@quimby.gnus.org:/pub/gnus/".  The patch is available as
<URL:ftp://quimby.gnus.org/pub/gnus/ding-patches/pgnus-0.90-0.91.diff.gz>.

ChangeLog since last release:

Sun Jul  4 06:31:01 1999  Lars Magne Ingebrigtsen  <larsi@menja.ifi.uio.no>

	* gnus.el: Pterodactyl Gnus v0.91 is released.

1999-07-04 04:35:28  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* gnus-agent.el (gnus-agent-save-active-1): New function.
	(gnus-agent-save-active): use it.
	(gnus-agent-save-groups): Ditto.

	* gnus-cache.el (gnus-cache-write-active): Use it.

	* gnus-agent.el (gnus-agent-write-active): Use it.

	* gnus-util.el (gnus-write-active-file): New function.

	* gnus-agent.el (gnus-agent-write-active): New function to keep
	lower boundaries and canceled groups.
	(gnus-agent-save-groups): Use it.
	(gnus-agent-save-active): Use it.
	(gnus-agent-save-group-info): Only write active files.
	(gnus-agent-expire): Update active file.

	* mm-decode.el (mm-inlinable-part-p): Removed.
	(mm-user-display-methods): Default to nil.
	(mm-user-display-methods): Removed.
	(add-mime-display-method): Removed.
	(mm-automatic-display): Renamed.
	(mm-automatic-display-p): Use it.
	(mm-inlined-types): New variable.
	(mm-inlined-p): New function.

	* message.el (message-reply): Bind message-this-is-mail.

1999-07-03 13:16:31  Michael Klingbeil  <mklingbeil@knuut.de>

	* smiley.el (smiley-buffer): Fix for NT.

1999-07-03 11:26:47  Lars Magne Ingebrigtsen  <larsi@gnus.org>

	* mm-encode.el (mm-encode-buffer): Check whether we have 7bit.

	* message.el (message-check-news-header-syntax): Protect against
	nil froms.

	* mm-util.el (mm-auto-mode-alist): New.

	* mml.el (mml-generate-mime-1): Ditto.

	* gnus.el: Use mm-insert-file-contents throughout instead of
	nnheader. 

	* mm-util.el (mm-insert-file-contents): New function.



-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen

Re: Pterodactyl Gnus v0.91 is released

[Michael Klingbeil]

Lars Magne Ingebrigtsen <larsi@ifi.uio.no> writes:

> 1999-07-03 13:16:31  Michael Klingbeil  <mklingbeil@knuut.de>
> 
> 	* smiley.el (smiley-buffer): Fix for NT.

I think this is from Fabrice, not from me;)

-- 

"Auch die Bretter, die man vor dem Kopf hat, koennen die Welt bedeuten"
 (Werner Finck)

Re: Pterodactyl Gnus v0.91 is released

[Neil Crellin]

Lars Magne Ingebrigtsen <larsi@ifi.uio.no> writes:
> MIME variable cleanup and agent active fixes.

Some change in 0.90 or 0.91 changed the invocation of xv on image
mime-subparts for me so it no longer works.  I used to enter
clari.living.comics.bizarro for example, press b and up would pop the
window in xv.  Now it offers to save the jpg to a file for me.

Re: Pterodactyl Gnus v0.91 is released

[Neil Crellin]

Neil Crellin <neilc@wallaby.cc> writes:
> Lars Magne Ingebrigtsen <larsi@ifi.uio.no> writes:
> > MIME variable cleanup and agent active fixes.
> 
> Some change in 0.90 or 0.91 changed the invocation of xv on image
> mime-subparts for me so it no longer works.  I used to enter
> clari.living.comics.bizarro for example, press b and up would pop the
> window in xv.  Now it offers to save the jpg to a file for me.

Reverting the s/string-match/string=/ patch to around line 490 of
mailcap.el provides the expected old behaviour.  Looking more closely
at that now. 

-Neil

Re: Pterodactyl Gnus v0.91 is released

[Neil Crellin]

Neil Crellin <neilc@wallaby.cc> writes:
> Neil Crellin <neilc@wallaby.cc> writes:
> > Lars Magne Ingebrigtsen <larsi@ifi.uio.no> writes:
> > > MIME variable cleanup and agent active fixes.
> > 
> > Some change in 0.90 or 0.91 changed the invocation of xv on image
> > mime-subparts for me so it no longer works.  I used to enter
> > clari.living.comics.bizarro for example, press b and up would pop the
> > window in xv.  Now it offers to save the jpg to a file for me.
> 
> Reverting the s/string-match/string=/ patch to around line 490 of
> mailcap.el provides the expected old behaviour.  Looking more closely
> at that now. 

Using string= in place of string-match here is doing the totally wrong
thing with regard to wildcard matches in the mailcap parsing. I suggest
reverting that particular sub-patch.  I'm not seeing what exactly that
patch was needed for, so I don't know if this breaks anything else,
but it does fix wildcard mailcap matches.

-- 
Neil Crellin <neilc@wallaby.cc>

Re: Pterodactyl Gnus v0.91 is released

[Lars Magne Ingebrigtsen]

Neil Crellin <neilc@wallaby.cc> writes:

> Using string= in place of string-match here is doing the totally wrong
> thing with regard to wildcard matches in the mailcap parsing. I suggest
> reverting that particular sub-patch.

I've now reverted it.

-- 
(domestic pets only, the antidote for overdose, milk.)
  larsi@gnus.org * Lars Magne Ingebrigtsen

Re: Pterodactyl Gnus v0.91 is released

[Kai.Grossjohann]

Neil Crellin <neilc@wallaby.cc> writes:

> Using string= in place of string-match here is doing the totally wrong
> thing with regard to wildcard matches in the mailcap parsing. I suggest
> reverting that particular sub-patch.  I'm not seeing what exactly that
> patch was needed for, so I don't know if this breaks anything else,
> but it does fix wildcard mailcap matches.

Somebody had a string "c++" in there, and using string-match didn't
find it.  (I forgot the exact details.)

Can we make it grok "c++" and wildcards?

What is wildcard matching in mailcap parsing supposed to do?

kai
-- 
Life is hard and then you die.

Re: Pterodactyl Gnus v0.91 is released

[Neil Crellin]

Kai.Grossjohann@CS.Uni-Dortmund.DE writes:
> Neil Crellin <neilc@wallaby.cc> writes:
> 
> > Using string= in place of string-match here is doing the totally wrong
> > thing with regard to wildcard matches in the mailcap parsing. I suggest
> > reverting that particular sub-patch.  I'm not seeing what exactly that
> > patch was needed for, so I don't know if this breaks anything else,
> > but it does fix wildcard mailcap matches.
> 
> Somebody had a string "c++" in there, and using string-match didn't
> find it.  (I forgot the exact details.)
> 
> Can we make it grok "c++" and wildcards?
> 
> What is wildcard matching in mailcap parsing supposed to do?

A fairly common artifice in mailcaps are things like the following:

  audio/*; /usr/bin/play %s
  
  image/*; xv %s
  
  video/*; xanim %s

In the example I was trying, mailcap-possible-viewers on an image/gif
was found to match no viewer with the above mailcap definition when
string= replaced string-match.  With string-match, the viewer was
determined to be "xv %s".

-- 
Neil Crellin <neilc@wallaby.cc>

Re: Pterodactyl Gnus v0.91 is released

[Hrvoje Niksic]

Neil Crellin <neilc@wallaby.cc> writes:

> A fairly common artifice in mailcaps are things like the following:
> 
>   audio/*; /usr/bin/play %s
>   
>   image/*; xv %s
>   
>   video/*; xanim %s
> 
> In the example I was trying, mailcap-possible-viewers on an
> image/gif was found to match no viewer with the above mailcap
> definition when string= replaced string-match.  With string-match,
> the viewer was determined to be "xv %s".

How can that work?  string-match works with regexps, not with
wildcards.

Re: Pterodactyl Gnus v0.91 is released

[William M. Perry]

Hrvoje Niksic <hniksic@srce.hr> writes:

> Neil Crellin <neilc@wallaby.cc> writes:
> 
> > A fairly common artifice in mailcaps are things like the following:
> > 
> >   audio/*; /usr/bin/play %s
> >   
> >   image/*; xv %s
> >   
> >   video/*; xanim %s
> > 
> > In the example I was trying, mailcap-possible-viewers on an
> > image/gif was found to match no viewer with the above mailcap
> > definition when string= replaced string-match.  With string-match,
> > the viewer was determined to be "xv %s".
> 
> How can that work?  string-match works with regexps, not with
> wildcards.

I imagine lars kept the logic from the original 'mm.el' that converts
wildcard matches like that to regular expressions.

Something along the lines of:

	(setq minor
	      (cond
	       ((= ?* (or (char-after save-pos) 0)) ".*")
	       ((= (point) save-pos) ".*")
	       (t (buffer-substring save-pos (point)))))

I guess there should be a regexp-quote in there for the last case.  I have
no idea where this code lives in the currently redesigned 'mm-*'
libraries.  Lars?

-Bill P.

Re: Pterodactyl Gnus v0.91 is released

[Lars Magne Ingebrigtsen]

wmperry@aventail.com (William M. Perry) writes:

> I guess there should be a regexp-quote in there for the last case.  I have
> no idea where this code lives in the currently redesigned 'mm-*'
> libraries.  Lars?

It's in mailcap.el, and I've now made it regexp-quote the last case.

-- 
(domestic pets only, the antidote for overdose, milk.)
  larsi@gnus.org * Lars Magne Ingebrigtsen

Re: Pterodactyl Gnus v0.91 is released

[Neil Crellin]

Hrvoje Niksic <hniksic@srce.hr> writes:
> Neil Crellin <neilc@wallaby.cc> writes:
> 
> > A fairly common artifice in mailcaps are things like the following:
> > 
> >   audio/*; /usr/bin/play %s
> >   
> >   image/*; xv %s
> >   
> >   video/*; xanim %s
> > 
> > In the example I was trying, mailcap-possible-viewers on an
> > image/gif was found to match no viewer with the above mailcap
> > definition when string= replaced string-match.  With string-match,
> > the viewer was determined to be "xv %s".
> 
> How can that work?  string-match works with regexps, not with
> wildcards.

mailcap-possible-viewers is called with major set to 

((".*" (viewer . "xv %s") (type . "image/*")) ("x-xwd" (viewer . "xwud
-in %s") (type . "image/x-xwd") ("compose" . "xwd -frame > %s") (test
eq (mm-device-type) (quote x)) ("needsx11")) ("x11-dump" (viewer
. "xwud -in %s") (type . "image/x-xwd") ("compose" . "xwd -frame >
%s") (test eq (mm-device-type) (quote x)) ("needsx11")) ("windowdump"
(viewer . "xwud -in %s") (type . "image/x-xwd") ("compose" . "xwd
-frame > %s") (test eq (mm-device-type) (quote x)) ("needsx11")) (".*"
(viewer . "aopen %s") (type . "image/*") (test eq (mm-device-type)
(quote ns))) (".*" (viewer . "display %s") (type . "image/*") (test eq
(mm-device-type) (quote x)) ("needsx11")) (".*" (viewer . "ee %s")
(type . "image/*") (test eq (mm-device-type) (quote x)) ("needsx11")))

and minor set to "gif", if that helps.