RFR: 8303920: Avoid calling out to python in DataDescriptorSignatureMissing test [v3]
Jaikiran Pai
jpai at openjdk.org
Fri Mar 10 08:18:16 UTC 2023
On Fri, 10 Mar 2023 08:06:58 GMT, Eirik Bjorsnos <duke at 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
More information about the core-libs-dev
mailing list