neomutt fails to build documentation

[neverpanic (Clemens Lang)]

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

@@ -5 +5 @@
 
 name                neomutt
 version             20160910
+revision            1
 categories          mail
 platforms           darwin
 license             GPL-2
@@ -21 +22 @@
 homepage            https://www.neomutt.org
 
 conflicts           mutt
+depends_build       port:docbook-xml-4.2 \
+                    port:docbook-xsl \
+                    port:libxslt \
+                    port:w3m
+
 depends_lib         port:gettext \
                     port:libiconv \
                     port:ncurses

OK to commit?

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

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

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

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

[lbschenkel (Leonardo Brondani Schenkel)]

update to 20160916

[neverpanic (Clemens Lang)]

Committed in r152777.