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

Updated headers to GPLv3 (in case of approval) #617

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Knogle
Copy link
Contributor

@Knogle Knogle commented Oct 16, 2024

Ahoy. Just some work done, maybe it's useful when we have the approval to go for GPLv3.

@PartialVolume
Copy link
Collaborator

Thanks, that saves me some time having to go through all the files.

@PartialVolume
Copy link
Collaborator

PartialVolume commented Oct 17, 2024

I thought the hddtemp_scsi code was GPLv2 code?. You were correct it was taken from the external hdd_temp source code, I believe last updated in 2007?

@Knogle
Copy link
Contributor Author

Knogle commented Oct 17, 2024

I thought the hddtemp_scsi code was GPLv2 code?. You were correct it was taken from the external hdd_temp source code, I believe last updated in 2007?

Ahoy.
Yeah it says

GPLv2 [..] (at your option) any later version.

So there is no issue on moving it to GPLv3.

@PartialVolume
Copy link
Collaborator

I thought the hddtemp_scsi code was GPLv2 code?. You were correct it was taken from the external hdd_temp source code, I believe last updated in 2007?

Ahoy. Yeah it says

GPLv2 [..] (at your option) any later version.

So there is no issue on moving it to GPLv3.

That's great. Thanks. Just checking.

@PartialVolume
Copy link
Collaborator

@Knogle The reason I questioned it was the discrepancy between the header in the .c files that contain the or later clause and the file called COPYING which doesn't contain that clause and is just the original GPLv2. I guess the author forgot to update the COPYING file.

I'm looking at the last release https://download.savannah.nongnu.org/releases/hddtemp/hddtemp-0.3-beta9.tar.bz2

As the C files contain GPLv2 or later, I'm happy with that.

@Knogle
Copy link
Contributor Author

Knogle commented Oct 18, 2024

That's great! I hope i didn't miss anything.

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