On Tue, 1 Aug 2023 12:50:47 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. > > Andreas Steiner has updated the pull request incrementally with one > additional commit since the last revision: > > set default cups include location if with_cups is not set only, update > copyright year Moving the check down a bit into a `if test "x$CUPS_FOUND" = xno; then` section might have a little benefit , it would detect cups headers on AIX at the standard location in case a bad non working path has been set with '-with-cups' or 'with-cups-include' . But I am okay with both. ------------- PR Comment: https://git.openjdk.org/jdk/pull/15100#issuecomment-1662124782
