Skip to content

Update LICENSE - #113

Merged
Sheridan merged 1 commit into
prometheus-community:masterfrom
bitfehler:master
Oct 25, 2023
Merged

Update LICENSE#113
Sheridan merged 1 commit into
prometheus-community:masterfrom
bitfehler:master

Conversation

@bitfehler

Copy link
Copy Markdown
Contributor

All source files have headers designating them as Apache-2.0-licensed. The LICENSE file however contains the text of the LGPL 3. This causes e.g. Github to declare the project as being LGPL licensed and also causes problems for packagers who need to figure out the project's license.

Since the LICENSE file seems to be a remnant of a distant past, update it to contain the Apache license, which seems to be what the project is intended to be licensed under.

All source files have headers designating them as Apache-2.0-licensed.
The `LICENSE` file however contains the text of the LGPL 3. This causes
e.g. Github to declare the project as being LGPL licensed and also
causes problems for packagers who need to figure out the project's
license.

Since the `LICENSE` file seems to be a remnant of a distant past, update
it to contain the Apache license, which seems to be what the project is
intended to be licensed under.

Signed-off-by: Conrad Hoffmann <ch@bitfehler.net>
@SuperQ

SuperQ commented Mar 3, 2023

Copy link
Copy Markdown
Contributor

I think we're going to need @Sheridan to approve this.

@bitfehler

Copy link
Copy Markdown
Contributor Author

Thanks. In the meantime, could you clarify what your current understanding of the license situation is?

@SuperQ

SuperQ commented Mar 3, 2023

Copy link
Copy Markdown
Contributor

When we adopted this exporter, we didn't notice the existing LICENSE file. We added the Apache headers without considering what the original license was.

@bitfehler

Copy link
Copy Markdown
Contributor Author

Okay. While we are at it: it was pointed out to me that the ebuild contained in this repo declares the project to be GPL licensed (not LGPL).

@SuperQ

SuperQ commented Mar 3, 2023

Copy link
Copy Markdown
Contributor

Yup, unfortunately, I think we need the original author's permission to clean this up.

@frittentheke

Copy link
Copy Markdown

Yup, unfortunately, I think we need the original author's permission to clean this up.

@Sheridan could you kindly take a look into this? There are more and more contributions coming in, would be awesome to not run into any licensing issues.

@robbat2

robbat2 commented Oct 16, 2023

Copy link
Copy Markdown
Contributor

I put an official ebuild, written from scratch into the Gentoo repository.
gentoo/gentoo@c18ea60

So you can clean up the outdated ebuild bundled inside the smartctl_exporter repo.

At that point, there is no file that contains a license tag other than Apache-2.0, leaving only @Sheridan's permission to change the license.

@Sheridan
Sheridan merged commit be7e4fa into prometheus-community:master Oct 25, 2023
@SuperQ

SuperQ commented Oct 25, 2023

Copy link
Copy Markdown
Contributor

Thanks @Sheridan!

@NiceGuyIT

Copy link
Copy Markdown
Member

Thank you @Sheridan!

@bitfehler bitfehler mentioned this pull request Oct 25, 2023
algitbot pushed a commit to alpinelinux/aports that referenced this pull request Nov 13, 2024
There was some confusion amongst the maintainers about the license for
this project, but they have now agreed to Apache 2.0.

See prometheus-community/smartctl_exporter#113
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.

6 participants