Skip to content

Fix license text for several files that should be LGPLv3+#39

Merged
alexey-tikhonov merged 1 commit intoSSSD:masterfrom
alexey-tikhonov:license-fixes
Apr 24, 2026
Merged

Fix license text for several files that should be LGPLv3+#39
alexey-tikhonov merged 1 commit intoSSSD:masterfrom
alexey-tikhonov:license-fixes

Conversation

@alexey-tikhonov
Copy link
Copy Markdown
Member

A follow up to d7999c3

Resolves: #36

@alexey-tikhonov
Copy link
Copy Markdown
Member Author

@jas4711, could you please check if I missed anything?

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the license headers across multiple files, transitioning the project from the GNU General Public License (GPL) to the GNU Lesser General Public License (LGPL). I have no feedback to provide.

A follow up to d7999c3

Resolves: SSSD#36

Assisted-By: Claude Code (Opus 4.6)
@jas4711
Copy link
Copy Markdown
Contributor

jas4711 commented Apr 23, 2026

LGTM -- eventually the Debian QA linters will pick this up and complain further, but I think it is fine to merge meanwhile.

@alexey-tikhonov
Copy link
Copy Markdown
Member Author

Note: build will be fixed by 5fc3394

Copy link
Copy Markdown

@ikerexxe ikerexxe left a comment

Choose a reason for hiding this comment

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

I see one additional file that can be modified, although maybe that was skipped intentionally. It's COPYING, can you confirm we don't need this change there?

@alexey-tikhonov
Copy link
Copy Markdown
Member Author

alexey-tikhonov commented Apr 24, 2026

I see one additional file that can be modified, although maybe that was skipped intentionally. It's COPYING, can you confirm we don't need this change there?

COPYING is a text of GPL and should be included because it is referred from COPYING.LESSER (LGPL):

  This version of the GNU Lesser General Public License incorporates
the terms and conditions of version 3 of the GNU General Public
License, supplemented by the additional permissions listed below.

Copy link
Copy Markdown

@ikerexxe ikerexxe left a comment

Choose a reason for hiding this comment

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

LGTM

@alexey-tikhonov alexey-tikhonov merged commit 0bcd7c8 into SSSD:master Apr 24, 2026
1 check failed
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.

GPL license?

3 participants