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

License is MIT #8

Merged
merged 2 commits into from
Oct 13, 2021
Merged

License is MIT #8

merged 2 commits into from
Oct 13, 2021

Conversation

MekDrop
Copy link
Member

@MekDrop MekDrop commented Nov 1, 2020

License file has license for GPLv2 but composer.json said that we use MIT.

Because MIT for using free components in commercial products, I decided that MIT would be better for us for that.

Do, you agree? :)

Copy link
Member

@fiammybe fiammybe left a comment

Choose a reason for hiding this comment

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

No issue with that from me.

@MekDrop MekDrop changed the title License is MIT WIP: License is MIT Nov 6, 2020
@MekDrop MekDrop changed the title WIP: License is MIT [WIP] License is MIT Nov 6, 2020
@MekDrop
Copy link
Member Author

MekDrop commented Nov 6, 2020

It seems that this change is possible after we change license to Impresscms otherwise we will have some legal issues with this.

@skenow
Copy link

skenow commented Nov 14, 2020

There was a LOT of pushback when we first tried even just altering the headers in files without changing the license.

@fiammybe
Copy link
Member

@skenow that was because we restructured files that were originally coming from Xoops. This is stuff we started by ourselves, so I don't see why we cannot license the composer installer as MIT.

@MekDrop
Copy link
Member Author

MekDrop commented Jan 10, 2021

@fiammybe problem right now that we somehow need to access impresscms constants from this library - how to do that not including all ImpressCMS and also same license I don't know :(

@fiammybe fiammybe marked this pull request as draft September 30, 2021 10:21
@MekDrop MekDrop changed the title [WIP] License is MIT License is MIT Oct 13, 2021
@MekDrop MekDrop marked this pull request as ready for review October 13, 2021 10:43
@MekDrop MekDrop added this to the 2.0 milestone Oct 13, 2021
@MekDrop MekDrop merged commit fa05fc3 into ImpressCMS:master Oct 13, 2021
@MekDrop MekDrop deleted the change/use-mit-license branch October 13, 2021 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants