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

Replace colorize gem with rainbow #152

Closed
wants to merge 3 commits into from
Closed

Conversation

listrophy
Copy link

@listrophy listrophy commented Nov 30, 2015

The colorize gem is licensed under GPL v2. While I sympathize with the
goal of the copyleft licenses, including library code that's licensed
under GPL can cause legal headaches. The rainbow gem seems to do
the same thing, except with an MIT license.

If replacing a GPL dependency with an MIT one isn't preferred, perhaps:

  1. Remove the 2 calls to .green and 2 calls to .red, or
  2. Use another gem's capability like bundler
  3. Implement the red and green in this codebase

Bradley Grzesiak added 2 commits September 15, 2016 13:56
The colorize gem is licensed under GPL v2. While I sympathize with the
goal of the copyleft licenses, including library code that's licensed
under GPL can cause legal headaches. The console-colors gem seems to do
the exact same thing.

If replacing a GPL dependency with an MIT one isn't preferred, perhaps:

  1. Remove the 2 calls to `.green` and 2 calls to `.red`, or
  2. Use another gem's capability like [bundler][bundler]
  3. Implement the red and green in this codebase

[bundler]:
https://github.com/bundler/bundler/blob/20f04aa8923c0d0f02e7a74d5e9609d772351668/lib/bundler/vendor/thor/lib/thor/shell/color.rb
@listrophy listrophy changed the title Replace colorize gem with console-colors. Replace colorize gem with rainbow Sep 15, 2016
@listrophy
Copy link
Author

Changed this to use rainbow instead of console-colors (since it's been deleted from GitHub).

@walterg2
Copy link

Any chance this is going to be implemented? It's pretty much a show stopper for my application to use this gem with the licensing issue contained within.

@vfonic
Copy link

vfonic commented Feb 27, 2017

👍
Are there any issues with merging this?

@damianlegawiec
Copy link
Member

Closed in favour of #168

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.

4 participants