Skip to content

Update Note in LGPL-x.y-only #2796

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update Note in LGPL-x.y-only #2796

wants to merge 2 commits into from

Conversation

jlovejoy
Copy link
Member

fixes #2705

Signed-off-by: Jilayne Lovejoy

@jlovejoy jlovejoy added this to the 3.28.0 milestone Jul 25, 2025
@jlovejoy jlovejoy self-assigned this Jul 25, 2025
@jlovejoy jlovejoy added the minor updates to file URL update, notes update, etc. label Jul 25, 2025
xsuchy
xsuchy approved these changes Jul 25, 2025
View reviewed changes
Copy link
Collaborator

@xsuchy xsuchy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@xsuchy
Copy link
Collaborator

xsuchy commented Jul 25, 2025

I would merge this, but CI is failing with:

Ignoring warning 'Duplicates licenses: LGPL-2.1-only, LGPL-2.1-or-later'
Ignoring warning 'Duplicates licenses: LGPL-2.1-only, LGPL-2.1'
make: *** [Makefile:30: validate-canonical-match] Error 64

@goneall
Copy link
Member

goneall commented Jul 25, 2025

Just FYI - I improved the error messages in the publisher to point out exactly which warnings are not being ignored and @swinslow has improved the input to the expected warnings. Once I publish the next release and update the CI - these will be easier to diagnose.
\

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

@xsuchy xsuchy xsuchy approved these changes

Assignees

@jlovejoy jlovejoy

Labels
minor updates to file URL update, notes update, etc.
Projects
None yet
Milestone
3.28.0
Development

Successfully merging this pull request may close these issues.

Discrepancy in LGPL-2.1-only & LGPL-2.0-only 'Notes' re file header example.
3 participants
@jlovejoy @xsuchy @goneall