On Fri, 10 Mar 2023 08:06:58 GMT, Eirik Bjorsnos <d...@openjdk.org> wrote:

>> Please review this PR which brings  the DataDescriptorSignatureMissing test 
>> back to life.
>> 
>> This test currently calls out to Python to create a test vector ZIP with a 
>> Data Descriptor without the recommended but optional signature. The Python 
>> dependency has turned out to be very brittle, so the test is currently 
>> marked with `@ignore` 
>> 
>> The PR replaces Python callouts with directly creating the test vector ZIP 
>> in the test itself. We can then remove the `@ignore`tag and run this useful 
>> test automatically.
>
> Eirik Bjorsnos has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Use "Signatureless" consistently

test/jdk/java/util/zip/DataDescriptorSignatureMissing.java line 2:

> 1: /*
> 2:  * Copyright 2012, 2023 Google, Inc.  All Rights Reserved.

It's my understanding that unless you are doing this change on behalf of 
Google, you shouldn't be changing that line. Instead, one has to add another 
line below that, of the form:

Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.

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

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

Reply via email to