Skip to content

Conversation

@amykyta3
Copy link
Member

@amykyta3 amykyta3 commented Jun 19, 2025

This PR changes the license of this project from GPLv3 to LGPLv3.

Explanation

The intent of using the copyleft GPLv3 license was to encourage PeakRDL community growth, and protect this library from being copied and re-distributed privately for profit. One unintended side-effect of GPLv3 is that it technically mandates that any program linking to this work (referencing this as a Python library/dependency) shall also be GPL-compatible. For Python libraries, this ends up being a pretty intrusive restriction, and was never my original intent.

Changing the license to LGPLv3 specifically removes this restriction. Wikipedia has a good summary on the difference.

Approvals Required

Despite this being a very minor change, it is considered good legal practice to get the approval of all contributors whose code would be impacted by the license change.

If your name is listed below, please comment in this PR explicitly approving the change from GPLv3 to LGPLv3.

Honorable Mentions

The following users are exempt from approval since their
contributions did not impact code that would be affected by this change.
Nonetheless, we appreciate their support!

  • krcb197

@coveralls
Copy link

Pull Request Test Coverage Report for Build 15747630680

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.402%

Totals Coverage Status
Change from base Build 14720272370: 0.0%
Covered Lines: 494
Relevant Lines: 508

💛 - Coveralls

@Smattr
Copy link
Contributor

Smattr commented Jun 19, 2025

Sure, I don’t have strong feelings about this. I would gladly license anything I contributed as Public Domain.

@amykyta3 amykyta3 merged commit 87cceab into main Jun 19, 2025
24 checks passed
@krcb197
Copy link
Collaborator

krcb197 commented Jun 19, 2025

@amykyta3

I think this a a sensible change.

I had recently been contacted by someone who wanted to use the peakrdl-python by didn't want the licenses extending into their code. At the time I wanted to remain aligned with the rest of the peakrdl community. However, if everyone else is making this change I will mirror it.

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.

5 participants