Red Hat Bugzilla – Bug 163760
mhshow fails to read mhshow-show-text/html from mhn.defaults
Last modified: 2007-11-30 17:11:10 EST
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.9) Gecko/20050711 Firefox/1.0.5
Description of problem:
If I put the following line in /etc/nmh/nmh.defaults:
mhshow-show-text/html: %plynx -dump -localhost -force_html -noredir '%F' | less
it is ignored and html messages are shown as raw html. But if I put that same line in my ~/.mh_profile, lynx is called and renders the html. I know that the settings in mhn.defaults aren't all being ignored by mhshow, since whether I have
there makes a difference when reading messages with that charset.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Get a message which has Content-type: text/html.
Actual Results: mhshow displays a bunch of raw html.
Expected Results: mhshow should call lynx to render the html.
*** Bug 163763 has been marked as a duplicate of this bug. ***
OK, I've verified this bug. It's fixed in the upstream CVS. I'll try to find
the fix in the very near future.
I've not had time to extract a proper patch, and I no longer plan on doing so.
There is a new release of nmh on the way, and I'd rather just wait to package
the new versus fixing the old:
I have a source patch for this very issue, which causes mhshow to effectively
ignore mhn.defaults when at the point of deciding what program to use.
The problem is that the settings read in from mhn.defaults are overwritten
by another call to context_read().
Thius patch short-circuits the secondary call to that function.
Then the mhshow tool of choice is properly obeyed from mhn.defaults.
Created attachment 121260 [details]
Patch for nmh-1.1
This patch is relatively straightforward fix for this regression (it used to
work way back in RH9!!).
Thanks for the patch. I'm sorry this bug has been open for so long, the nmh
release is taking much longer than expected. I'm checking in this into the nmh
cvs repository tonight. If there is no 1.2 before the new year, or I need to
roll a new nmh for some reason, I will include this patch.
I found some time this evening to patch this. Look for nmh-1.1-14.fc to
show up soon in extras.