On Thu, 9 Mar 2023 17:12:53 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>>> I would argue that we should keep the replacement string matching the make 
>>> variable its getting the value from. It makes it easier to trace the origin 
>>> of the value.
>> 
>> Then we can define a new make variable that will also allow it be a 
>> different value than `OPENJDK_TARGET_OS` if desirable.
>
> There is no need to define a new variable, there are already too many in the 
> build and the build and the compiled files should be using the same symbols.

IMO the naming should make sense from the perspective of reading the 
post-processed source file - so the variable should refer to "current" not 
"target".

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

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

Reply via email to