RFR: 8336695: Update Commons BCEL to Version 6.10.0
Chen Liang
liach at openjdk.org
Wed Sep 24 20:08:44 UTC 2025
On Wed, 24 Sep 2025 18:03:08 GMT, Joe Wang <joehw at openjdk.org> wrote:
>> src/java.xml/share/classes/com/sun/org/apache/bcel/internal/classfile/AnnotationEntry.java line 2:
>>
>>> 1: /*
>>> 2: * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
>>
>> IANAL, but I think the original year needs to be here, as apparently it has been in the repository.
>>
>> Also, I see other files, e.g., `ArrayElementValue.java` has the same header ("reserved comment block"). Should they also have Oracle copyright?
>
> Oracle copyright is added when we make changes to the upstream source. In this case, I replaced the newly introduced reference to Apache Commons Lang with a Utils method, thus the copyright year 2025 since we haven't made changes to the upstream source until now. Also added is the LastModified tag as required by the license. Note the tag was not there until now.
If Apache's bcel constantly introduce new dependencies, monkey patches like these are no better than migrating the whole xsltc to use ClassFile API for bytecode generation, in my opinion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27459#discussion_r2376935621
More information about the core-libs-dev
mailing list