RFR: 8255352: Archive important test outputs in submit workflow [v2]
    Aleksey Shipilev 
    shade at openjdk.java.net
       
    Mon Oct 26 08:22:48 UTC 2020
    
    
  
On Mon, 26 Oct 2020 08:11:47 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Remove duplicate PATH item
>>    
>>    Co-authored-by: Robin Westberg <robin at westberg.se>
>>  - Remove duplicate PATH item
>>    
>>    Co-authored-by: Robin Westberg <robin at westberg.se>
>
> Looks good! I also tried including the entire test-support folder at some point, but as you noticed it takes quite a bit of time and space. So this approach seems much better.
> 
> Minor comment: The cygwin\bin folder is included twice in PATH on Windows to work around a bug in JTReg (the regex used to detect cygwin does not match the very first entry in the PATH list. Perhaps I should file a bug for that..) but it's not really needed if you are executing something else.
Thanks! I have not even noticed there is double item in `PATH`.
-------------
PR: https://git.openjdk.java.net/jdk/pull/842
    
    
More information about the build-dev
mailing list