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

p-chart is released under gplv3 #2175

Merged
merged 10 commits into from Nov 18, 2019
Merged

p-chart is released under gplv3 #2175

merged 10 commits into from Nov 18, 2019

Conversation

@lachlan-00
Copy link
Member

lachlan-00 commented Nov 18, 2019

restore it.

lachlan-00 added 10 commits Nov 18, 2019
This is gplv3 so i see no problem with it.
charts
it's gpl3
@lachlan-00 lachlan-00 merged commit d4a2a5c into develop Nov 18, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@wagnered

This comment has been minimized.

Copy link
Contributor

wagnered commented Nov 18, 2019

@lachlan-00

This comment has been minimized.

Copy link
Member Author

lachlan-00 commented Nov 18, 2019

It is definitely dual licensed still but has changed from MIT to gplv3 since that issue.

Graphs are disabled by default requiring config to enable so it could go in there.

I need to do more research in the reasoning behind the fedora packaging with dual licensing as I don't think a lot of the restrictions apply to Ampache itself.

@wagnered

This comment has been minimized.

Copy link
Contributor

wagnered commented Nov 18, 2019

The Fedora docs might shed some light on that.

I think someone at pChart finally decided to look at the wording of the of the MIT is excludes any attempt to impose copyright or proprietary clauses to the conditions of usage.

@lachlan-00

This comment has been minimized.

Copy link
Member Author

lachlan-00 commented Nov 18, 2019

I've reopend the issue #1515 and put it to debian maintainers of the old versions as well.

I think to avoid all this i'll separate c-pchart package from the base composer and detail the extra requirements to enable in the config and wiki. that way it's not included in the base install without multiple manual steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.