chore: change the license to GPLv3-or-later #773

Merged
earl-warren merged 1 commit from earl-warren/runner:wip-gpl into main 2025-09-04 09:26:12 +00:00
Contributor

The Forgejo runner is part of the Forgejo project. The licensing
agreement in the governance repository reads like this:

Forgejo accepts contributions compatible with the GPLv3-or-later license.
The license under which Forgejo is distributed will be changed upon the acceptance of such contributions.

The first step is to update the LICENSE file to reflect that
decision. The individual copyright notice of each source file will be
updated when and if relevant. If a change is made that is eligible for
copyright, the author may decide change the copyright notice from MIT
or Apache 2 to GPLv3-or-later.

Refs licensing agreement codeberg.org/forgejo/governance@6eb522282f/AGREEMENTS.md (licensing)
Refs blog post about the licensing agreement https://forgejo.org/2024-08-gpl/

  • other
    • PR: chore: change the license to GPLv3-or-later
The Forgejo runner is part of the Forgejo project. The licensing agreement in the governance repository reads like this: > Forgejo accepts contributions compatible with the GPLv3-or-later license. > The license under which Forgejo is distributed will be changed upon the acceptance of such contributions. The first step is to update the LICENSE file to reflect that decision. The individual copyright notice of each source file will be updated when and if relevant. If a change is made that is eligible for copyright, the author may decide change the copyright notice from MIT or Apache 2 to GPLv3-or-later. Refs licensing agreement https://codeberg.org/forgejo/governance/src/commit/6eb522282f606f65c77b83b662b008aaf444dcfb/AGREEMENTS.md#licensing Refs blog post about the licensing agreement https://forgejo.org/2024-08-gpl/ <!--start release-notes-assistant--> <!--URL:https://code.forgejo.org/forgejo/runner--> - other - [PR](https://code.forgejo.org/forgejo/runner/pulls/773): <!--number 773 --><!--line 0 --><!--description Y2hvcmU6IGNoYW5nZSB0aGUgbGljZW5zZSB0byBHUEx2My1vci1sYXRlcg==-->chore: change the license to GPLv3-or-later<!--description--> <!--end release-notes-assistant-->
earl-warren changed title from chore: change the license from MIT to GPLv3-or-later [skip ci] [skip cascade] to chore: change the license from MIT to GPLv3-or-later 2025-07-31 05:30:11 +00:00
Author
Contributor

Although I'm convinced this change is in line with the governance decision on licensing and with the spirit of the Forgejo project itself, I would like your advice on how to best communicate about this. My inclination would be to add a section to the next monthly update. I don't think it is worth a dedicated blog post. It is an important step but it will also not come as a surprise to anyone.

What do you think?

Although I'm convinced this change is in line with the governance decision on licensing and with the spirit of the Forgejo project itself, I would like your advice on how to best communicate about this. My inclination would be to add a section to the next monthly update. I don't think it is worth a dedicated blog post. It is an important step but it will also not come as a surprise to anyone. What do you think?
earl-warren changed title from chore: change the license from MIT to GPLv3-or-later to chore: change the license from MIT to GPLv3-or-later [skip ci] [skip cascade] 2025-07-31 05:38:42 +00:00
earl-warren force-pushed wip-gpl from 1572f0e98e
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 5s
to 334d62c590 2025-07-31 05:38:51 +00:00
Compare
viceice approved these changes 2025-07-31 07:14:57 +00:00
Dismissed
sclu1034 approved these changes 2025-07-31 07:17:32 +00:00
sclu1034 left a comment
Member

I also think a section in the monthly update, with a link to the existing blog post about the license change should be enough.

Maybe also a more prominent entry in the next major version's changelog (just a sentence at the top).

I also think a section in the monthly update, with a link to the existing blog post about the license change should be enough. Maybe also a more prominent entry in the next major version's changelog (just a sentence at the top).
Owner

There have been talks about picking from act recently. Has there been any (successful? attempts to upstream code from Forgejo to act? I wonder if it might make sense to revisit this now that there might be another fork of act?

I 100% agree to the GPL requirement to the runner part, but I wonder if it makes sense to hold back for the act part.

There have been talks about picking from act recently. Has there been any (successful? attempts to upstream code from Forgejo to act? I wonder if it might make sense to revisit this now that there might be another fork of act? I 100% agree to the GPL requirement to the runner part, but I wonder if it makes sense to hold back for the act part.
Author
Contributor

@fnetX wrote in #773 (comment):

Has there been any (successful? attempts to upstream code from Forgejo to act?

Not really. Ever since the Forgejo runner started there were no significant attempt.

... but I wonder if it makes sense to hold back for the act part.

Given the current state of the act project, contributing to it would be a waste of time. A year ago it was different. This is the primary reason why the act codebase was merged into the runner a few days ago.

@fnetX wrote in https://code.forgejo.org/forgejo/runner/pulls/773#issuecomment-50788: > Has there been any (successful? attempts to upstream code from Forgejo to act? Not really. Ever since the Forgejo runner started there were no significant attempt. > ... but I wonder if it makes sense to hold back for the act part. Given the current state of the act project, contributing to it would be a waste of time. A year ago it was different. This is the primary reason why the act codebase was merged into the runner a few days ago.
earl-warren changed title from chore: change the license from MIT to GPLv3-or-later [skip ci] [skip cascade] to chore: change the license to GPLv3-or-later [skip ci] [skip cascade] 2025-08-03 07:24:46 +00:00
earl-warren force-pushed wip-gpl from 334d62c590 to f14fd13cae
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 4s
2025-08-06 13:23:12 +00:00
Compare
earl-warren changed title from chore: change the license to GPLv3-or-later [skip ci] [skip cascade] to chore: change the license to GPLv3-or-later 2025-08-06 13:24:27 +00:00
earl-warren changed title from chore: change the license to GPLv3-or-later to chore: change the license to GPLv3-or-later [skip ci] [skip cascade] 2025-08-06 13:24:40 +00:00
Author
Contributor

@viceice @sclu1034 the update of this pull request is a rebase and minor rewording to take into account that there also are Apache 2 licensed files.

@viceice @sclu1034 the update of this pull request is a rebase and minor rewording to take into account that there also are Apache 2 licensed files.
Author
Contributor

I will wait on more Forgejo contributors to review before moving forward. It is a step that can be difficult to revert and it is best if this is approved by a large number of the most active contributors. It can wait a few weeks for that to happen, there is no immediate urgency.

I will wait on more Forgejo contributors to review before moving forward. It is a step that can be difficult to revert and it is best if this is approved by a large number of the most active contributors. It can wait a few weeks for that to happen, there is no immediate urgency.
viceice approved these changes 2025-08-06 15:22:46 +00:00
Member

I haven’t been contributing to the runner and I cannot judge the license change concerning the runner and act. Thus, I abstain from reviewing here. I like the GPL and think the change is well covered by the agreement within Forgejo.

I haven’t been contributing to the runner and I cannot judge the license change concerning the runner and act. Thus, I abstain from reviewing here. I like the GPL and think the change is well covered by the agreement within Forgejo.
Author
Contributor

@mahlzahn this is fair. My thinking was (probably misguided) that this is a change that has a global impact on the Forgejo project. I would want to know about it when / if that happens in a part of Forgejo I'm not involved with. If only to state "I saw it, but I'll abstain", as you just did.

@mahlzahn this is fair. My thinking was (probably misguided) that this is a change that has a global impact on the Forgejo project. I would want to know about it when / if that happens in a part of Forgejo I'm not involved with. If only to state "I saw it, but I'll abstain", as you just did.
mfenniak approved these changes 2025-08-06 17:40:57 +00:00
mfenniak left a comment
Owner

Conceptually 👍

I wonder if adding SPDX metadata to files at this point indicating their license would help clarify the situation for all future changes?

Conceptually 👍 I wonder if adding SPDX metadata to files at this point indicating their license would help clarify the situation for all future changes?
Author
Contributor

I think it would.

I think it would.
earl-warren changed title from chore: change the license to GPLv3-or-later [skip ci] [skip cascade] to chore: change the license to GPLv3-or-later 2025-08-06 18:18:37 +00:00
Gusted approved these changes 2025-08-07 19:25:55 +00:00
Beowulf approved these changes 2025-08-07 20:29:06 +00:00
Dismissed
earl-warren force-pushed wip-gpl from f14fd13cae
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 4s
to 7d6fde1025
Some checks failed
checks / build and test (pull_request) Successful in 1m52s
issue-labels / release-notes (pull_request_target) Successful in 7s
checks / runner exec tests (pull_request) Successful in 41s
act / unit (pull_request) Successful in 4m41s
Integration tests for the release process / release-simulation (pull_request) Failing after 6m4s
act / integration (pull_request) Successful in 10m53s
/ cascade (pull_request_target) Successful in 38m59s
2025-08-12 10:40:45 +00:00
Compare
Contributor

cascading-pr updated at actions/setup-forgejo#593

cascading-pr updated at https://code.forgejo.org/actions/setup-forgejo/pulls/593
earl-warren force-pushed wip-gpl from 7d6fde1025
Some checks failed
checks / build and test (pull_request) Successful in 1m52s
issue-labels / release-notes (pull_request_target) Successful in 7s
checks / runner exec tests (pull_request) Successful in 41s
act / unit (pull_request) Successful in 4m41s
Integration tests for the release process / release-simulation (pull_request) Failing after 6m4s
act / integration (pull_request) Successful in 10m53s
/ cascade (pull_request_target) Successful in 38m59s
to 60d976dae5
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 11s
checks / build and test (pull_request) Successful in 1m32s
checks / runner exec tests (pull_request) Successful in 40s
act / unit (pull_request) Successful in 2m58s
Integration tests for the release process / release-simulation (pull_request) Successful in 6m0s
act / integration (pull_request) Successful in 12m41s
/ cascade (pull_request_target) Successful in 38m56s
2025-08-12 11:06:28 +00:00
Compare
Contributor

cascading-pr updated at actions/setup-forgejo#593

cascading-pr updated at https://code.forgejo.org/actions/setup-forgejo/pulls/593
Author
Contributor

No change other than rebasing and fixing an integration tests that was verifying MIT instead of GPL.

No change other than rebasing and fixing an integration tests that was verifying MIT instead of GPL.
Beowulf approved these changes 2025-08-12 11:08:39 +00:00
earl-warren requested reviews from crystal, pat-s and removed review requests for wetneb 2025-08-20 06:34:30 +00:00
Author
Contributor

@earl-warren wrote in #773 (comment):

@fnetX wrote in #773 (comment):

Has there been any (successful? attempts to upstream code from Forgejo to act?

Not really. Ever since the Forgejo runner started there were no significant attempt.

... but I wonder if it makes sense to hold back for the act part.

Given the current state of the act project, contributing to it would be a waste of time. A year ago it was different. This is the primary reason why the act codebase was merged into the runner a few days ago.

Does this address your concern? I would appreciate if you could formally approve in case it does (with an "Approve" to the PR that is 😁).

@earl-warren wrote in https://code.forgejo.org/forgejo/runner/pulls/773#issuecomment-50793: > @fnetX wrote in #773 (comment): > > > Has there been any (successful? attempts to upstream code from Forgejo to act? > > Not really. Ever since the Forgejo runner started there were no significant attempt. > > > ... but I wonder if it makes sense to hold back for the act part. > > Given the current state of the act project, contributing to it would be a waste of time. A year ago it was different. This is the primary reason why the act codebase was merged into the runner a few days ago. Does this address your concern? I would appreciate if you could formally approve in case it does (with an "Approve" to the PR that is 😁).
pat-s approved these changes 2025-08-20 06:42:48 +00:00
Author
Contributor

@0ko @n0toose @jerger @pat-s @crystal @oliverpool I wanted to bring this change to your attention. It is an important decision for the Forgejo project and I'm making my due diligence in making sure Forgejo contributors who I perceive to have been active this year have a chance to voice any concern they may have.

If you are not opposed but do not feel informed enough to approve, it is fine if you just say so. What matters is that you are aware it is happening. I would not want that to be a surprise to anyone.

@0ko @n0toose @jerger @pat-s @crystal @oliverpool I wanted to bring this change to your attention. It is an important decision for the Forgejo project and I'm making my due diligence in making sure Forgejo contributors who I perceive to have been active this year have a chance to voice any concern they may have. If you are not opposed but do not feel informed enough to approve, it is fine if you just say so. What matters is that you are aware it is happening. I would not want that to be a surprise to anyone.
fnetX approved these changes 2025-08-25 18:54:16 +00:00
fnetX left a comment
Owner

I'm fine with this change. I'd still love to see upstreaming efforts of this code in the future and hope the license change does not block it. However, I also think that the Forgejo runner is even more likely to benefit from the license change.

I'm fine with this change. I'd still love to see upstreaming efforts of this code in the future and hope the license change does not block it. However, I also think that the Forgejo runner is even more likely to benefit from the license change.
Author
Contributor

I'd still love to see upstreaming efforts of this code in the future...

I feel I'm missing something since that's the third time you suggest this would be a good thing. How do you see that happening, in the best case possible? Let say someone contributing to the Forgejo runner spare no effort to upstream what has been done to Nektos ACT, who exactly would they talk to?

> I'd still love to see upstreaming efforts of this code in the future... I feel I'm missing something since that's the third time you suggest this would be a good thing. How do you see that happening, in the best case possible? Let say someone contributing to the Forgejo runner spare no effort to upstream what has been done to Nektos ACT, who exactly would they talk to?
0ko approved these changes 2025-08-27 09:42:07 +00:00
0ko left a comment
Owner

I am not a user or a contributor of runner, but I approve of this change.

Having uniform licensing across different first-party Forgejo software is good.

I can't find any evidence of new commits being exchanged between Act on GitHub and the fork on Forgejo.code, and with more time passing it seems less likely to happen.

So now is a good time for this change.

I am not a user or a contributor of runner, but I approve of this change. Having uniform licensing across different _first-party_ Forgejo software is good. I can't find any evidence of new commits being exchanged between Act on GitHub and the fork on Forgejo.code, and with more time passing it seems less likely to happen. So now is a good time for this change.
Author
Contributor

@fnetX wrote in #773 (comment):

I'm fine with this change. I'd still love to see upstreaming efforts of this code in the future and hope the license change does not block it. However, I also think that the Forgejo runner is even more likely to benefit from the license change.

I found this discussion where you were advocating for GPLv3+ and I was advocating for keeping the license for the sake of contributing back to ACT. So the roles were effectively inverted back then, which I forgot.

What changed my mind, 9 months later, is the low activity in ACT and the creation of a fork by the main contributor. It did not receive much attention in the past six months.

image

Moreover the pace of changes in 2024 was already low. Because the main repository has so many stars, I don't believe the owner will be willing to give it to someone else, even to keep it going.

@fnetX wrote in https://code.forgejo.org/forgejo/runner/pulls/773#issuecomment-56915: > I'm fine with this change. I'd still love to see upstreaming efforts of this code in the future and hope the license change does not block it. However, I also think that the Forgejo runner is even more likely to benefit from the license change. I found [this discussion](https://codeberg.org/forgejo/discussions/issues/242) where you were advocating for GPLv3+ and I was advocating for keeping the license for the sake of contributing back to ACT. So the roles were effectively inverted back then, which I forgot. What changed my mind, 9 months later, is the low activity in ACT and the [creation of a fork](https://github.com/actions-oss) by the main contributor. It did not receive much attention in the past six months. ![image](/attachments/4a0bf98e-87cb-4d84-ae16-7e138e31c958) Moreover the pace of changes in 2024 was already low. Because the main repository has so many stars, I don't believe the owner will be willing to give it to someone else, even to keep it going.
earl-warren force-pushed wip-gpl from 60d976dae5
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 11s
checks / build and test (pull_request) Successful in 1m32s
checks / runner exec tests (pull_request) Successful in 40s
act / unit (pull_request) Successful in 2m58s
Integration tests for the release process / release-simulation (pull_request) Successful in 6m0s
act / integration (pull_request) Successful in 12m41s
/ cascade (pull_request_target) Successful in 38m56s
to e93a14567a
All checks were successful
cascade / forgejo (pull_request_target) Has been skipped
cascade / end-to-end (pull_request_target) Has been skipped
cascade / debug (pull_request_target) Successful in 23s
issue-labels / release-notes (pull_request_target) Successful in 36s
checks / validate mocks (pull_request) Successful in 1m20s
checks / build and test (pull_request) Successful in 2m5s
checks / runner exec tests (pull_request) Successful in 52s
Integration tests for the release process / release-simulation (pull_request) Successful in 6m30s
checks / runner integration tests (pull_request) Successful in 4m42s
checks / integration tests (pull_request) Successful in 17m22s
2025-09-01 13:59:15 +00:00
Compare
earl-warren force-pushed wip-gpl from e93a14567a
All checks were successful
cascade / forgejo (pull_request_target) Has been skipped
cascade / end-to-end (pull_request_target) Has been skipped
cascade / debug (pull_request_target) Successful in 23s
issue-labels / release-notes (pull_request_target) Successful in 36s
checks / validate mocks (pull_request) Successful in 1m20s
checks / build and test (pull_request) Successful in 2m5s
checks / runner exec tests (pull_request) Successful in 52s
Integration tests for the release process / release-simulation (pull_request) Successful in 6m30s
checks / runner integration tests (pull_request) Successful in 4m42s
checks / integration tests (pull_request) Successful in 17m22s
to bf0ee8c229
All checks were successful
issue-labels / release-notes (pull_request_target) Successful in 10s
checks / validate mocks (pull_request) Successful in 31s
checks / build and test (pull_request) Successful in 1m0s
checks / runner exec tests (pull_request) Successful in 40s
Integration tests for the release process / release-simulation (pull_request) Successful in 4m36s
checks / runner integration tests (pull_request) Successful in 3m30s
checks / integration tests (pull_request) Successful in 12m26s
cascade / debug (pull_request_target) Successful in 1s
cascade / end-to-end (pull_request_target) Successful in 5s
cascade / forgejo (pull_request_target) Successful in 19s
2025-09-01 18:43:03 +00:00
Compare
Author
Contributor

I believe enough time has passed (a month) and it was advertised so everyone got a chance to voice their concerns. Let's merge this.

I believe enough time has passed (a month) and it was advertised so everyone got a chance to voice their concerns. Let's merge this.
earl-warren deleted branch wip-gpl 2025-09-04 09:26:13 +00:00
Owner

i think this should be released as another major version, but it can wait

i think this should be released as another major version, but it can wait
Author
Contributor

It is a legal breaking change indeed.

It is a legal breaking change indeed.
earl-warren referenced this pull request from a commit 2025-09-05 07:11:14 +00:00
earl-warren referenced this pull request from a commit 2025-09-05 07:29:39 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
11 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo/runner!773
No description provided.