Skip to content

fix: include OW2 attribution in ASM licenses - #1266

Open
dajiaohuang wants to merge 1 commit into
apache:mainfrom
dajiaohuang:fix-ow2-license-attribution-followup
Open

dajiaohuang wants to merge 1 commit into
apache:mainfrom
dajiaohuang:fix-ow2-license-attribution-followup

Conversation

@dajiaohuang

Copy link
Copy Markdown

Related: #1172

Purpose of the pull request

The merged #1176 added the Spring copyright lines to META-INF/NOTICE, but the OW2 Consortium copyright line identified in #1172 is still absent from the ASM license included in the shaded JAR and source distribution. Both current copies contain the ASM BSD-3-Clause text and INRIA/France Telecom notice but omit the OW2 notice.

What's changed?

Add Copyright (c) 1999-2009, OW2 Consortium <https://www.ow2.org/> to both:

  • fesod-shaded/src/main/resources/META-INF/LICENSE
  • dist/licenses/LICENSE-asm.txt

Validation

  • Confirmed the missing line in both current files and checked the merged Fix missing Spring and OW2 attributions in shaded JAR #1176 change set, which did not alter either ASM notice.
  • git diff --check passes; the line is present in both copies.
  • No runtime test applies to this license-text-only change.

Checklist

  • I have read the Contributor Guide.
  • I have updated the required LICENSE entries.
  • I have written the necessary doc or comment (not applicable to this metadata-only change).
  • I have added the necessary unit tests and all cases have passed (no executable behavior changed; tests are not applicable).

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant