On Tue, 1 Aug 2023 07:28:02 GMT, Andreas Steiner <d...@openjdk.org> wrote:

> Add the default include location(/opt/freeware/include/) for cups on AIX. 
> With this set the additional configure parameter --with-cups-include can be 
> removed, which was needed on AIX.

I think the check for the cups default location on AIX fits better in the next 
section, after line 70/79 `# Are the cups headers installed in the default 
/usr/include location?`

But other than that, looks good to me.

-------------

Marked as reviewed by clanger (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15100#pullrequestreview-1556370701

Reply via email to