On Tue, 21 Jan 2025 16:34:56 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

>> This is so that when `idea.sh` finally calls make again, we will get back to 
>> our configuration. Otherwise, if we have multiple configurations, `idea.sh` 
>> will fail since it can't single out a specific one. This will not show up on 
>> CI runs where we only have a single configuration, but can arise when 
>> running the test locally with several configurations.
>
> I thought that was about what I said in the comment, and I feel like I mostly 
> repeat myself. Can you suggest a better way to phrase it, if it is still not 
> clear enough?

Suggestion:

# Unset these variables to avoid having the current environment interfere with
# the idea generation script. SPEC needs to remain as idea.sh will call back to
# make and that call needs to have SPEC defined.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23200#discussion_r1924170617

Reply via email to