[Bug 225672] graphics/poppler-glib: Build fails with OPENJPEG disabled

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=225672

            Bug ID: 225672
           Summary: graphics/poppler-glib: Build fails with OPENJPEG
                    disabled
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Keywords: patch
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: gnome at FreeBSD.org
          Reporter: vlad-fbsd at acheronmedia.com
          Assignee: gnome at FreeBSD.org
             Flags: maintainer-feedback?(gnome at FreeBSD.org)

Created attachment 190326
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=190326&action=edit
Correct OPENJPEG option's function both for MASTER and SLAVES

With OPENJPEG option disabled, build of poppler-glib fails.

The problem seems to be in the graphics/poppler (master) conditional that
checks whether it's a SLAVE or MASTER build. The conditional is setting
OPTIONS-defined CONFIGURE_ARGS only for MASTER (though the comment for the
.else section seems to think it's for SLAVES), so for slaves the missing
OPENJPEG option does not set --enable-libopenjpeg to none.

As the default for that is "auto", the build fails at configure stage, looking
for libopenjpeg:

> configure: error: Install libopenjpeg2 or libopenjpeg1 before trying to build poppler. You can also decide to use the internal unmaintained JPX decoder or none at all. See --help.

This patch extracts the OPENJPEG conditional outside of master-slave check
(thus it applies for both), and now poppler, poppler-utils, poppler-data and
poppler-glib build fine, with OPENJPEG disabled.

Please note I don't know if this breaks anything functionally, or even if it's
the correct thing to do, or it is but CAIRO and CURL options need to be
extracted outside of the master-slave conditional as well.

Build tested with Poudriere 11.1 amd64.

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