Emile CARCAMO | 22 Apr 2012 22:07
X-Face
Picon
Favicon

Feedback about exmh new version 2.8.0


Dead All,

	I've given it a try on a "non production" machine and
	I've installed the stuff as I did so many times. Nothing
	particular to notice, except some details I've listed for
	you below :

- to avoid troubles with msgShow.tcl, I had to replace all occurences
of exmh_lassign by lassign like it was the case wirh version 2.7.2 :

   	21:34	sed -i 's/exmh_lassign/lassign/g' html.tcl
   	21:34	sed -i 's/exmh_lassign/lassign/g' html_formtag.tcl
   	21:34	sed -i 's/exmh_lassign/lassign/g' html_get_http.tcl
   	21:34	sed -i 's/exmh_lassign/lassign/g' html_links.tcl
   	21:34	sed -i 's/exmh_lassign/lassign/g' msgShow.tcl
   	21:35	sed -i 's/exmh_lassign/lassign/g' utils.tcl

Below a list of files impacted with this change I did ... Note that I'm
using tcl/tk version 8.5 on a Fedora 16 distro.

- I also noticed that saving address database was taking forever ...
and was always run with a short time interval between 2 consecutive
runs. Since I'm not a tcl-tk guru at all, I've simply taken back the good
old /usr/local/exmh-2.7.2-CVS/lib/addr.tcl to replace the 2.8.0 version.

	Apart of these 2 troubles, I didn't notice anything bad so far.
	Thanks again and again to all those who pushed hard to get
	this new version in the field so quickly. With best regards,

(Continue reading)

Alexander Zangerl | 22 Apr 2012 23:58
Picon

Re: Feedback about exmh new version 2.8.0

On Sun, 22 Apr 2012 22:07:12 +0200, Emile CARCAMO writes:
>- to avoid troubles with msgShow.tcl, I had to replace all occurences
>of exmh_lassign by lassign like it was the case wirh version 2.7.2 :

this makes little sense, as the change *TO* exmh_lassign was 
made in 2010 precisely to avoid conflicts with tcl8.5's lassign, which 
has a different argument order and therefore doesn't work. 

(exmh's exmh_lassign wants (varnames,values), tcl8.5's lassign wants
(values,varnames).)

regards
az

--

-- 
Alexander Zangerl + GnuPG Keys 0x42BD645D or 0x5B586291 + http://snafu.priv.at/
Don't drop dead. Do, please, drop seriously, painfully, disablingly,
repulsively, incurably, but _not_ fatally ill. -- Mike Andrews
_______________________________________________
Exmh-users mailing list
Exmh-users <at> redhat.com
https://www.redhat.com/mailman/listinfo/exmh-users
Welch, Brent | 23 Apr 2012 02:20
Favicon

RE: Feedback about exmh new version 2.8.0

Yeah, it would be good to see the exact Tcl stack trace from the troubles you had with msgShow.tcl.  I'd like to
get to the root cause.
	Brent

-----Original Message-----
From: exmh-users-bounces <at> redhat.com [mailto:exmh-users-bounces <at> redhat.com] On Behalf Of
Alexander Zangerl
Sent: Sunday, April 22, 2012 2:59 PM
To: Discussion list for EXMH users
Subject: Re: Feedback about exmh new version 2.8.0

On Sun, 22 Apr 2012 22:07:12 +0200, Emile CARCAMO writes:
>- to avoid troubles with msgShow.tcl, I had to replace all occurences 
>of exmh_lassign by lassign like it was the case wirh version 2.7.2 :

this makes little sense, as the change *TO* exmh_lassign was made in 2010 precisely to avoid conflicts with
tcl8.5's lassign, which has a different argument order and therefore doesn't work. 

(exmh's exmh_lassign wants (varnames,values), tcl8.5's lassign wants
(values,varnames).)

regards
az

--
Alexander Zangerl + GnuPG Keys 0x42BD645D or 0x5B586291 + http://snafu.priv.at/ Don't drop dead. Do,
please, drop seriously, painfully, disablingly, repulsively, incurably, but _not_ fatally ill. --
Mike Andrews
Valdis.Kletnieks | 23 Apr 2012 02:07
Picon
Favicon

Re: Feedback about exmh new version 2.8.0

On Sun, 22 Apr 2012 22:07:12 +0200, Emile CARCAMO said:

> - to avoid troubles with msgShow.tcl, I had to replace all occurences
> of exmh_lassign by lassign like it was the case wirh version 2.7.2 :
>
>    	21:34	sed -i 's/exmh_lassign/lassign/g' html.tcl

wait, what?  Pretty much all releases of exmh were buggy under tcl8.5:

cvs repository 08/26/2010 valdis.kletnieks <at> vt.edu
  Finally found the cause of the exmh Log window breaking - we
  carry our own lassign that had the parameters in opposite order
  to what base 8.5 code expects, specifically breaking 'clock format'
  and who knows what else. So rename our lassign to exmh_lassign.
  lib/html.tcl, lib/html_formtag.tcl, lib/html_get_http.tcl,
  lib/html_links.tcl lib/msgShow.tcl, lib/utils.tcl

(Yes, it's an ugly fix, but the only realistic choice unless we insist on tcl
8.5 or later.  You want tcl 8.4 to work too, have to do it this way).

What "troubles" were you seeing? This sounds suspiciously like
untarring a 2.8 tarball over a 2.7.2 tree and failing to regenerate
the tclIndex file, which will result in it failing to find exmh_lassign and
throwing an error....

_______________________________________________
Exmh-users mailing list
Exmh-users <at> redhat.com
(Continue reading)

Emile CARCAMO | 24 Apr 2012 07:12
Picon
Favicon

Re: Feedback about exmh new version 2.8.0


Hello,

	I'll try to provide Brent and other people with the "tcl
	stack errors" I've noticed with the exmh_lassign usage.

	I've replaced it with lassign but afaik I don't think it
	was necessary to rebuild any kind of index like this :
>
>Then, put the attached file in your ~/.tk/exmh/ directory and make the 
>index:
>
>wish
>auto_mkindex ~/.tk/exmh/
>^D
>
	I did this index thing when I wanted to add extra buttons
	( like FetchMail or FetchNotes, for example ).

	What I've also noticed is that the automatic spell checker
	wasn't working as it should : I had to troubles with it
	earlier ! What about other users out there ???

	Thanks for any help, and with best regards.

-Émile
Ken Hornstein | 24 Apr 2012 16:38
X-Face
Picon
Favicon

Re: Feedback about exmh new version 2.8.0

>	I'll try to provide Brent and other people with the "tcl
>	stack errors" I've noticed with the exmh_lassign usage.
>
>	I've replaced it with lassign but afaik I don't think it
>	was necessary to rebuild any kind of index like this :

There's a "TclIndex" button in the install program; did you run that?

--Ken

Gmane