Opened 3 years ago

Closed 3 years ago

#52268 closed defect (fixed)

neomutt fails to build documentation

Reported by: neverpanic (Clemens Lang) Owned by: lbschenkel (Leonardo Brondani Schenkel)
Priority: Normal Milestone:
Component: ports Version: 2.3.99
Keywords: Cc:
Port: neomutt

Description

The current neomutt port ships with an empty manual.txt because documentation generation fails due to a number of missing dependencies. The attached patch fixes this.

I've chosen w3m over lynx or elinks, because it seems to be the one that has the fewest (or most commonly used) dependencies and is available as a binary archives from our buildbot.

  • Portfile

     
    55
    66name                neomutt
    77version             20160910
     8revision            1
    89categories          mail
    910platforms           darwin
    1011license             GPL-2
     
    2122homepage            https://www.neomutt.org
    2223
    2324conflicts           mutt
     25depends_build       port:docbook-xml-4.2 \
     26                    port:docbook-xsl \
     27                    port:libxslt \
     28                    port:w3m
     29
    2430depends_lib         port:gettext \
    2531                    port:libiconv \
    2632                    port:ncurses

OK to commit?

Attachments (1)

neomutt.patch (1.4 KB) - added by lbschenkel (Leonardo Brondani Schenkel) 3 years ago.
update to 20160916

Download all attachments as: .zip

Change History (6)

comment:1 Changed 3 years ago by lbschenkel (Leonardo Brondani Schenkel)

Yes.

I haven't noticed that the manual.txt was empty. Interesting that they changed the manual generation in NeoMutt because this doesn't happen with vanilla mutt.

comment:2 Changed 3 years ago by neverpanic (Clemens Lang)

I don't think they changed the generation, I think it may be related to the fact that we're running autoreconf or somehow changing timestamps on some files. I think we should look into that rather than adding the dependencies then, maybe we can get away without the additional deps.

comment:3 Changed 3 years ago by lbschenkel (Leonardo Brondani Schenkel)

I use autoreconf because they made a mistake in NeoMutt when building the .tar.gz file and omitted some files which makes the install phase fail. This was reported on https://github.com/neomutt/neomutt/issues/140 and fixed, but I was hoping that by using the sources and autoreconf I could avoid having to patch it locally and be more resilient against packaging mistakes by upstream, but this may have backfired regarding the documentation.

I will try to use the .tar.gz with a local patch and see if that works.

comment:4 Changed 3 years ago by lbschenkel (Leonardo Brondani Schenkel)

There is a new version of neomutt recently released, they fixed the packaging problem so I can build from the tarball now and the documentation is correctly built. I have attached the patch.

Changed 3 years ago by lbschenkel (Leonardo Brondani Schenkel)

Attachment: neomutt.patch added

update to 20160916

comment:5 Changed 3 years ago by neverpanic (Clemens Lang)

Resolution: fixed
Status: newclosed

Committed in r152777.

Note: See TracTickets for help on using tickets.