On Thu, Jan 21, 2010 at 10:19:30AM +0100, Christoph Berg wrote:
> > We're trying to move elinks out of the main repository and would like to
> > use w3m instead. You can a find both manual pages for comparison at:
> >  * http://people.canonical.com/~mathiaz/mutt-manual/manual.elinks.html
> >  * http://people.canonical.com/~mathiaz/mutt-manual/manual.w3m.html
> >
> > They look the same altough the html diff is quite huge. I haven't
> > investigate the actual differences. Another advantage is to drop one
> > part of the debian specific patch, since upstream already supports
> > building with w3m.
> 
> >  --- a/doc/Makefile.am
> >  +++ b/doc/Makefile.am
> > -@@ -100,9 +100,7 @@ uninstall-local:
> > - 
> > - check:
> > - manual.txt: manual.html
> > --  -LC_ALL=C lynx -dump -nolist -with_backspaces manual.html > $@ || \
> > --  LC_ALL=C w3m -dump manual.html > $@ || \
> > --  LC_ALL=C elinks -dump -no-numbering -no-references manual.html | sed -e 
> > 's,\\001, ,g' > $@
> > -+  LC_ALL=C elinks -dump -dump-charset utf8 -no-numbering -no-references 
> > $< > $@
> 
> That would need a Build-Conflicts against lynx so the build result is
> predictable. Also, I dislike the "-" before that line as it make
> failures non-fatal. I don't think we can drop that hunk completely.

One thing I noticed when evaluating similar suggestion for
dictionaries-common (#566078). You may want to modify that patch to make
sure that '-O UTF-8' is used in the w3m call (mutt upstream does not set
it) 

LC_ALL=C w3m -dump -O UTF-8 manual.html > $@ || \

to force the output charset to UTF-8. Otherwise output may be incorrectly 
rendered by w3m if 8bit chars are present (renders as '?' for 'C' locale). 

Cheers,

-- 
Agustin



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to