[Bug 269616] [NEW PORT] astro/osmium-tool: Command line tool for working with OpenStreetMap data based on the Osmium library

From: <bugzilla-noreply_at_freebsd.org>
Date: Thu, 10 Aug 2023 18:06:56 UTC

Ian Wagner <ianthetechie@gmail.com> changed:

           What    |Removed                     |Added
 Attachment #240228|0                           |1
        is obsolete|                            |
 Attachment #244012|                            |maintainer-approval+
              Flags|                            |

--- Comment #4 from Ian Wagner <ianthetechie@gmail.com> ---
Created attachment 244012
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=244012&action=edit
Suggested patch (update 2)

Good call-outs, Robert. I looked into these and it looks like this is a fun one
given how the build scripts are set up. I'm not a CMake expert, so I may have
missed something.

Manpages appear to be build solely based on Pandoc's availability. See the
unconditional include here:
In the CMakeLists for that directory, it looks like manpages are built if
Pandoc is available and there is no option to change that behavior:

I think the simplest option would be to just add Pandoc to BUILD_DEPENDS
unconditionally and scrap the MANPAGES option. Any objections to that approach?

LZ4 does have an option:
I've amended the Makefile to pass this through. Point taken on it not costing
much to enable; I assume most systems would already have it anyways, so this is
enabled by default in the new patch.

You are receiving this mail because:
You are the assignee for the bug.