Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Update LICENSE #229

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

Update LICENSE #229

wants to merge 2 commits into from

Conversation

paladox
Copy link
Contributor

@paladox paladox commented Jun 13, 2015

Please see #148

@PhrozenByte
Copy link
Contributor

It's not that simple! Before doing any license changes, you must ask every single contributor for his/her permission.

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

Oh ok.

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

How can that be done.

@PhrozenByte
Copy link
Contributor

By asking them 😄 There are some tools out there making this process easier, e.g. https://github.com/beberlei/license-manager. I'm not sure (@oyejorge), but does less.php use or adapt code of less.js? If this is the case, less.php even can't change its license until less.js does so...

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

I doint think so because if it did it would use a js file. Which it doesent.

@oyejorge
Copy link
Owner

does less.php use or adapt code of less.js

The code of less.php is based entirely off of less.js. The code was translated file by file into php from js.

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

Oh so would the license affect us changing it here.

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

I carnt do https://github.com/beberlei/license-manager since I run windows 10.

@paladox
Copy link
Contributor Author

paladox commented Jun 13, 2015

Hi isent this file https://github.com/oyejorge/less.php/blob/master/lessc.inc.php based on https://github.com/leafo/lessphp/blob/master/lessc.inc.php for compatibility reasons woulden then that require the dual licence since its based on it and that file is dual lincensed for GPL3 and MIT.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants