Bug 237436

Summary: java/openjdk8: configure fails when FONTCONFIG=off
Product: Ports & Packages Reporter: Dominik Lisiak <dominik.lisiak>
Component: Individual Port(s)Assignee: freebsd-java (Nobody) <java>
Status: Closed FIXED    
Severity: Affects Some People CC: dmgk, freebsd, glewis
Priority: --- Keywords: needs-qa
Version: LatestFlags: bugzilla: maintainer-feedback? (java)
koobs: merge-quarterly?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
openjdk8-8.212.4.1.log
none
patch file dmgk: maintainer-approval?

Description Dominik Lisiak 2019-04-21 12:24:14 UTC
Created attachment 203860 [details]
openjdk8-8.212.4.1.log

The port fails to configure if FONTCONFIG option is disabled. If enabled it works fine. I have attached Poudriere log.
Comment 1 Dmitri Goutnik freebsd_committer freebsd_triage 2019-04-21 13:39:10 UTC
Created attachment 203862 [details]
patch file

Looks like x11-fonts/fontconfig is always required and cannot depend on X11 knob.
Comment 2 Dmitri Goutnik freebsd_committer freebsd_triage 2019-04-21 13:41:56 UTC
(In reply to Dominik Lisiak from comment #0)

FONTCONFIG implies X11 and x11-fonts/fontconfig currently depends on the X11, so configure fails without X11.
Comment 3 Fredrik Eriksson 2019-04-22 09:41:38 UTC
I also see this, and can also report that the port will build if either X11 or FONTCONFIG is set, so even if FONTCONFIG is off it can still build if X11 is on.
Comment 4 commit-hook freebsd_committer freebsd_triage 2019-04-22 21:12:19 UTC
A commit references this bug:

Author: glewis
Date: Mon Apr 22 21:11:55 UTC 2019
New revision: 499687
URL: https://svnweb.freebsd.org/changeset/ports/499687

Log:
  Minor to whitespace to correct the PR for the previous commit.

  Previous commit was:

  Always depend on fontconfig

  PR:		237436
  Submitted by:   Dominik Lisiak <dominik.lisiak@bemsoft.pl>

Changes:
  head/java/openjdk8/Makefile
Comment 5 Greg Lewis freebsd_committer freebsd_triage 2019-04-22 21:18:17 UTC
Note that attempting to disable fontconfig with configure will yield the message:

configure: error: It is not possible to disable the use of fontconfig.

I.e. it is now a requirement as of 8u212.

This makes the FONTCONFIG option confusing from a naming standpoint since fontconfig that will be a dependency even with the option off.

It might be worth removing the option and always making the additional source code changes in that option to avoid the confusion.