Skip to content
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.

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

Change license to BSD-2-Clause #121

Merged
merged 12 commits into from
Nov 12, 2024
Merged

Change license to BSD-2-Clause #121

merged 12 commits into from
Nov 12, 2024

Conversation

dehanj
Copy link
Member

@dehanj dehanj commented Oct 23, 2024

Description

  • License update to BSD-2-Clause
  • Follow reuse spec
  • Bump dependencies, especially tkey-device-signer, tkeyutil and tkeyclient that are changed from GPLv2 only to BSD-2-Clause.
  • Update golanglint-ci
  • Fix issues newer golanglint-ci warns about
  • Remove dco

Type of change

Please tick any that are relevant to this PR and remove any that aren't.

  • Feature (non breaking change which adds functionality)
  • Documentation (a change to documentation)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)

@dehanj dehanj requested a review from mchack-work October 23, 2024 07:59
@dehanj dehanj marked this pull request as ready for review October 23, 2024 08:00
@dehanj dehanj marked this pull request as draft October 23, 2024 08:01
@dehanj dehanj force-pushed the change-license branch 4 times, most recently from dfcdd86 to b6af9ba Compare October 23, 2024 09:32
@dehanj dehanj changed the title Change license Change license to BSD-2-Clause Oct 23, 2024
@mchack-work
Copy link
Member

Removed some build artefacts from the REUSE spec, changed missing license expression in Debian control file and goreleaser. Goreleaser builds packages fine.

Want to see if dpkg complains when installing package on Debian before approving.

@dehanj
Copy link
Member Author

dehanj commented Nov 12, 2024

tkeyclient bump commit disappeared, so added it back and also bumped tkeysign.
Now there should not be any GPL dependencies left.

@dehanj dehanj marked this pull request as ready for review November 12, 2024 08:54
Copy link
Member

@mchack-work mchack-work left a comment

Choose a reason for hiding this comment

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

  • Changed typos in tkeysign bump.
  • Tested to install the .deb from goreleaser
  • Remove toolchain from go.mod

Merging.

dehanj and others added 12 commits November 12, 2024 17:05
- tkeyutil is now released under BSD-2-Clause
- tkeyclient is now released under BSD-2-Clause
- tkeysign is now released under BSD-2-Clause
v1.0.2 of the signer is licensed under BSD-2-Clause
The DCO is deemd unnecessary since GitHub's Terms of Service covers the
same topics. See GitHub's Terms of Service D3 and D6.
This was announced on the Tillitis Blog the 2nd of October 2024, for
more info visit https://tillitis.se

Co-authored-by: Michael Cardell Widerkrantz <mc@tillitis.se>
- Add REUSE configuration
- Add compliant license in LICENSES/
- Add copyright SPDX tags
- Run REUSE Compliance check in CI
In CI we like to control what exact version of the Go compiler is used
in our own OCI image.
@mchack-work mchack-work merged commit 14bd4b7 into main Nov 12, 2024
6 checks passed
@mchack-work mchack-work deleted the change-license branch November 12, 2024 16:09
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.

2 participants