RFR: 8296478: Rework 8282948 and 8282700 to use the new autoconf UTIL_ARG_WITH [v7]

[Julian Waters]

On Tue, 6 Dec 2022 10:16:03 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>> 
>>  - Change
>>  - Merge remote-tracking branch 'upstream/master' into util
>>  - Include the broken options for now
>>  - Uh oh
>>  - branding.conf should use the DEFAULT_ prefix
>>  - Use DEFAULT_DESC
>>  - Merge remote-tracking branch 'upstream/master' into util
>>  - Squash
>
> make/conf/branding.conf line 27:
> 
>> 25: 
>> 26: LAUNCHER_NAME=openjdk
>> 27: PRODUCT_NAME=OpenJDK
> 
> This mix of `NNNN` and `DEFAULT_NNNN` looks a bit strange. Is it so that the one's that are not prefixed with `DEFAULT_` cannot be overridden by configure flags?

It's more just me trying to preserve the original names, I don't really want to change too many things here. I'll change them to DEFAULT_ instead if you prefer that though. I don't feel like this is my decision to make

Sorry for taking so long to reply, half the keys don't work on this disintegrating piece of junk (such as the letter "t"), I've had to Ctrl+C Ctrl+V all the letters that are broken on this keyboard just to get coherent messages out

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

PR: https://git.openjdk.org/jdk/pull/11020