Skip to content

Update license information on READMEs#192

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
bkmgit:bkmgit-patch-1
Jan 8, 2024
Merged

Update license information on READMEs#192
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
bkmgit:bkmgit-patch-1

Conversation

@bkmgit
Copy link
Contributor

@bkmgit bkmgit commented Aug 8, 2023

No description provided.

@deepin-ci-robot
Copy link

Hi @bkmgit. Thanks for your PR.:smiley:

@github-actions
Copy link

github-actions bot commented Aug 8, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@bkmgit
Copy link
Contributor Author

bkmgit commented Aug 8, 2023

I have read the CLA Document and I hereby sign the CLA.

@bkmgit
Copy link
Contributor Author

bkmgit commented Aug 8, 2023

recheck

deepin-bot bot added a commit to linuxdeepin/cla that referenced this pull request Aug 8, 2023
@bkmgit bkmgit force-pushed the bkmgit-patch-1 branch 2 times, most recently from 5245319 to bee48d8 Compare August 8, 2023 11:41
@BLumia BLumia requested a review from dengbo11 August 9, 2023 05:10
@BLumia
Copy link
Member

BLumia commented Aug 10, 2023

Thanks for the patch! I'm not sure if this is the correct fix tho. It seems most of the source files declared them as GPL instead if LGPL, maybe the LICENSE file in the root folder is the one that needs to be updated instead.

We'll check which is the actual case and let you know when we got an answer.

@18202781743
Copy link
Contributor

We'll check which is the actual case and let you know when we got an answer.

这个有结论了么?

@BLumia
Copy link
Member

BLumia commented Sep 1, 2023

这个有结论了么?

事太杂跟进漏了 = =

我下周一问下法务那边,如果周一我还没更新这个进度的话可以再 ping 我一下 😂

@BLumia
Copy link
Member

BLumia commented Sep 5, 2023

Sorry I once again forgot to confirm if this fix is okay.

After reaching out with the legal team, there are two options:

  1. Since there are some source files using GPL instead of LGPL, this project also needs to use GPL as its main license.
  2. Or we can relicense all these GPL source files to LGPL, and thus we can use LGPL as its main license.

This project mainly provides libraries and plugins for other projects to use, thus actually LGPL is preferred, but correcting the LICENSE file in the project root folder can be an easy fix. I personally suggest we update the LICENSE file to GPL, then check if we need to relicense those GPL source to LGPL, and finally change our main license to LGPL.

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Jan 5, 2024

TAG Bot

New tag: 2.0.17
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #212

Update README to indicate LGPL rather than GPL
license is used.
@Decodetalkers
Copy link
Contributor

@BLumia 这个pr那么这样就可以合并了吗

@Decodetalkers
Copy link
Contributor

/check obs

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Jan 5, 2024

Deepin Obs Bot
Deepin redeliveried failed, need update or recreate pull request!!!

@BLumia
Copy link
Member

BLumia commented Jan 6, 2024

#216 should addressed this issue :)

@BLumia BLumia closed this Jan 6, 2024
@Decodetalkers
Copy link
Contributor

Decodetalkers commented Jan 8, 2024

@BLumia 我是希望我把代码的处理了,然后可以合并这个的。。emm,所以readme还没有变

@BLumia
Copy link
Member

BLumia commented Jan 8, 2024

oh sorry

@BLumia BLumia reopened this Jan 8, 2024
@Decodetalkers
Copy link
Contributor

/merge

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bkmgit, Decodetalkers

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Decodetalkers
Copy link
Contributor

Thanks very much

@deepin-bot deepin-bot bot merged commit 7a3f3e9 into linuxdeepin:master Jan 8, 2024
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