Debian Bug report logs - #566076
mutt: Use w3m to build the manual instead of elinks

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Mathias Gug <mathiaz@ubuntu.com>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: mutt: Use w3m to build the manual instead of elinks

Date: Wed, 20 Jan 2010 18:27:16 -0500

[Message part 1 (text/plain, inline)]

Package: mutt
Version: 1.5.20-5
Severity: normal
Tags: patch
User: ubuntu-devel@lists.ubuntu.com
Usertags: origin-ubuntu lucid ubuntu-patch



*** /tmp/tmpt0BBDj
In Ubuntu, we've applied the attached patch to achieve the following:

  * debian/control, debian/patches/debian-specific/build_doc_adjustments.diff:
    use w3m instead of elinks for generating documentation.

We thought you might be interested in doing the same. 

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.

Thanks,

-- System Information:
Debian Release: squeeze/sid
  APT prefers lucid-updates
  APT policy: (500, 'lucid-updates'), (500, 'lucid-security'), (500, 'lucid')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.24-25-server (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash

[tmpd3C8yE (text/plain, attachment)]

Message #8 received at 566076@bugs.debian.org (full text, mbox, reply):

From: Christoph Berg <myon@debian.org>

To: Mathias Gug <mathiaz@ubuntu.com>, 566076@bugs.debian.org

Subject: Re: Bug#566076: mutt: Use w3m to build the manual instead of elinks

Date: Thu, 21 Jan 2010 10:19:30 +0100

[Message part 1 (text/plain, inline)]

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

Besides that, I wouldn't oppose using w3m. Last time I checked, elinks
looked nicer, but the difference wasn't huge.

Btw, how do the the html links you posted prove anything? That patch
is about the .txt version of the manual.

Christoph
-- 
cb@df7cb.de | http://www.df7cb.de/

[signature.asc (application/pgp-signature, inline)]

Message #13 received at 566076@bugs.debian.org (full text, mbox, reply):

From: Agustin Martin <agmartin@debian.org>

To: Mathias Gug <mathiaz@ubuntu.com>, 566076@bugs.debian.org

Subject: Re: Bug#566076: mutt: Use w3m to build the manual instead of elinks

Date: Fri, 22 Jan 2010 12:43:49 +0100

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

Message #20 received at 566076@bugs.debian.org (full text, mbox, reply):

From: Antonio Radici <antonio@dyne.org>

To: Agustin Martin <agmartin@debian.org>, 566076@bugs.debian.org

Cc: Mathias Gug <mathiaz@ubuntu.com>, control@bugs.debian.org

Subject: Re: Bug#566076: mutt: Use w3m to build the manual instead of elinks

Date: Thu, 30 Dec 2010 17:14:23 +0000

tag 566076 +confirmed pending
thanks

On Fri, Jan 22, 2010 at 12:43:49PM +0100, Agustin Martin wrote:
> 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). 

Since it seems that other debian packages are moving to w3m we could try
this for 1.5.21-2 in experiemental and see how it behaves (adding -O
UTF-8 as suggested).

Cheers
Antonio

Message #25 received at 566076@bugs.debian.org (full text, mbox, reply):

From: Antonio Radici <antonio@dyne.org>

To: Mathias Gug <mathiaz@ubuntu.com>

Cc: Agustin Martin <agmartin@debian.org>, 566076@bugs.debian.org

Subject: Re: Bug#566076: mutt: Use w3m to build the manual instead of elinks

Date: Sat, 1 Jan 2011 19:08:55 +0000

[Message part 1 (text/plain, inline)]

This has just been committed in git, the patch is now called
debian-specific/566076-build_doc_adjustments.patch (attached here for your
convenience); the patch has been renamed to be consistent with our naming scheme
for patches.

Cheers
Antonio

[566076-build_doc_adjustments.patch (text/x-diff, attachment)]

Message #30 received at 566076-close@bugs.debian.org (full text, mbox, reply):

From: Antonio Radici <antonio@dyne.org>

To: 566076-close@bugs.debian.org

Subject: Bug#566076: fixed in mutt 1.5.21-2

Date: Sun, 02 Jan 2011 23:47:22 +0000

Source: mutt
Source-Version: 1.5.21-2

We believe that the bug you reported is fixed in the latest version of
mutt, which is due to be installed in the Debian FTP archive:

mutt-dbg_1.5.21-2_i386.deb
  to main/m/mutt/mutt-dbg_1.5.21-2_i386.deb
mutt-patched_1.5.21-2_i386.deb
  to main/m/mutt/mutt-patched_1.5.21-2_i386.deb
mutt_1.5.21-2.diff.gz
  to main/m/mutt/mutt_1.5.21-2.diff.gz
mutt_1.5.21-2.dsc
  to main/m/mutt/mutt_1.5.21-2.dsc
mutt_1.5.21-2_i386.deb
  to main/m/mutt/mutt_1.5.21-2_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 566076@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Antonio Radici <antonio@dyne.org> (supplier of updated mutt package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Sun, 02 Jan 2011 21:05:25 +0000
Source: mutt
Binary: mutt mutt-patched mutt-dbg
Architecture: source i386
Version: 1.5.21-2
Distribution: experimental
Urgency: low
Maintainer: Antonio Radici <antonio@dyne.org>
Changed-By: Antonio Radici <antonio@dyne.org>
Description: 
 mutt       - text-based mailreader supporting MIME, GPG, PGP and threading
 mutt-dbg   - debugging symbols for mutt
 mutt-patched - the Mutt Mail User Agent with extra patches
Closes: 537061 566076 576313 578087 578099 578583 603288 608706
Changes: 
 mutt (1.5.21-2) experimental; urgency=low
 .
   [ Christoph Berg ]
   * debian/patches: features/imap_fast_trash: Support purging of messages.
 .
   [ Antonio Radici ]
   * debian/patches:
     + upstream/578087-header-strchr.patch: prevent from segfaulting on malformed
       messages (Closes: 578087, 578583)
     + upstream/603288-split-fetches.patch: split FETCH's into smaller chunks,
       workaround for Exchange 2010 (Closes: 603288)
     + upstream/537061-dont-recode-saved-attachments.patch: as the patch says,
       see the patch for more info (Closes: 537061)
     + upstream/608706-fix-spelling-errors.patch: to fix some spelling errors
       (Closes: 608706)
     + debian-specific/566076-build_doc_adjustments.patch: use w3m to build the
       manual (Closes: 566076)
   * debian/extra/lib/mailto-mutt: replaced by a wrapper, added the reason to
     NEWS.Debian (Closes: 576313)
   * debian/extra/rc/compressed-folders.rc: added support for xz-compressed
     folders (Closes: 578099)
   * debian/*.lintian-overrides: ignore 'binary without manpage' errors due to
     the diversions to mutt-org/mutt-patched
Checksums-Sha1: 
 4e0f5bb3d46b1ec5cb1c393603de6c0758eecaeb 1404 mutt_1.5.21-2.dsc
 d529692cd1f9961a0033eb8bd3862bbd1a0b706e 157684 mutt_1.5.21-2.diff.gz
 c173e7de626451f2ccb8f60aa4add29ed39f510f 2089678 mutt_1.5.21-2_i386.deb
 7e61ab651fd524a25dae6fc488c814739f4f3c78 393648 mutt-patched_1.5.21-2_i386.deb
 b943d2bb783cb17fc297af5113a0a19c62e8ad33 1387512 mutt-dbg_1.5.21-2_i386.deb
Checksums-Sha256: 
 c2b1b663e53492f0f2dc34ab9d32eb172884162442568bbb7b85049bae6f337f 1404 mutt_1.5.21-2.dsc
 683253aeb64ddb25421d29d7bcf07b9ae3e8e6298fe9b0846030fe17a75c1ab0 157684 mutt_1.5.21-2.diff.gz
 f61e55b69d3bca4f61cfb757b7f8b57584e6cce177ca4815c55f5b0985b99681 2089678 mutt_1.5.21-2_i386.deb
 c9ee0c772f8bf97edbd85e06f81864d6205478ac5c139aaf89ffa73c0f2d6eb5 393648 mutt-patched_1.5.21-2_i386.deb
 f434b8f5bdb84efe86fb3feaee9820679acad1795d366adedb39e3281267e652 1387512 mutt-dbg_1.5.21-2_i386.deb
Files: 
 18b755631a93134ff7419f71ca9bedcb 1404 mail standard mutt_1.5.21-2.dsc
 d288e8f86204b6da69bfbd40173c74e5 157684 mail standard mutt_1.5.21-2.diff.gz
 8bbc5c217b41b8f3717a80a266dfabd0 2089678 mail standard mutt_1.5.21-2_i386.deb
 1ae90daf2329bd5ebe381ba369b9bb76 393648 mail extra mutt-patched_1.5.21-2_i386.deb
 bea8e633301127dfde2f383c3fb87371 1387512 debug extra mutt-dbg_1.5.21-2_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAk0hDHIACgkQ6Tq8aae+WrBYhwCcCKZC+KXVyBctXHpyDboWjip6
gCgAn2bLgSKfDzd4ncUY17wCc7+0Lv06
=KxMj
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.