License Violation! about roo HOT 14 CLOSED

I don't see why it would be appropriate to license roo under more restrictive terms just because one of our dependencies is restrictive. Particularly given that those who do not use XLS aren't affected.

I'm happy to mention spreadsheet's GPL licensing in the readme, feel free to submit a pull request to update it. Beyond that you're better off trying to convince spreadsheet to adopt a more liberal license.

from roo.

GLPv2 and GLP3 are compatible with the MIT/Expat License (http://gplv3.fsf.org/wiki/index.php/Compatible_licenses), I think this issue can be safely closed.

from roo.

Hey @pabloh I don't see MIT/BSD in the GPL3 list. Can you point it out for me?

from roo.

@Empact the MIT license is identical to the Expat license (http://directory.fsf.org/wiki/License:Expat , http://www.debian.org/legal/licenses/mit), which's inside the first list.

from roo.

Good to know, thanks.

Even so, I see that Expat is compatible with GPL2 but I only see the following for GPL3:

GPLv3-compatible licenses

These licenses are compatible with the GNU GPL version 3.
Apache Software License, version 2.0

from roo.

As far as I knew every license compatible with GPLv2 was also compatible with GPLv3.

Nevertheless, I just checked and it seems to be the case: http://www.gnu.org/licenses/quick-guide-gplv3.html

Bellow 'New Compatible Licenses' the following sentence can be found within that section's 2nd paragraph:
"...GPLv3 is compatible with just about everything GPLv2 is, and then some."

from roo.

@Empact I'm there with you. I would love spreadsheetto change to MIT, but they won't. They sell licences for companies who use spreedsheet but don't want to open source their code, which is good in a way, although not very cheap. So when discovering roo and it's free licensing we automatically switched to it, not seeing that it uses spreedsheet.

I will send a pull request with the respective mention of roo's underlying gems licences.

from roo.

My pull request #43 with respecitve code sleeptillseven@2a8b916

from roo.

Thanks!

from roo.

Hi @Empact,

Thanks for the great gem :)

I had the same problem as @sleeptillseven (even after reading the new text :S). My understanding of that sentence was that "...this gem is better than spreadsheet because it uses MIT license..."! English is not my native language :)

One important thing that should be mentioned in the README is that Roo depends on spreadsheet gem. It's not explicitly mentioned right now.

I've created a fork of this repo here that doesn't depend on spreadsheet gem (and doesn't support *.xls files) for people, like me, that don't need the support for *.xls.

from roo.

@rui-castro That sounds like a great solution. Splitting out a roo-xls gem would make a lot of sense particularly considering its legacy nature. I'll look at making this change and publishing the gem later this week. Thanks!

from roo.

@Empact great ;)

from roo.

I've done this now, but haven't released the gems yet - see roo-rb/roo-xls.

from roo.

@Empact nice. Thanks.

from roo.