shairontoledo / rghost

RGhost is a document creation and conversion API. It uses the Ghostscript framework for the format conversion, utilizes EPS templates and is optimized to work with larger documents. Support(PDF,PS,GIF,TIF,PNG,JPG,etc)
http://rghost.rubyforge.org
MIT License
186 stars 46 forks source link

Iconv deprecated #23

Closed stclara closed 12 years ago

stclara commented 12 years ago

Salve, mestre. O que deve ser feito para corrigir esta mensagem que aparece quando inicia o servidor:

/home/stclara/.rvm/gems/ruby-1.9.3-p0/gems/rghost-0.8.7.6/lib/rghost/ruby_ghost_config.rb:2:in `<top (required)>': iconv will be d eprecated in the future, use String#encode instead.

[]'s

Stclara.

shairontoledo commented 12 years ago

Opa, tenta nukar o iconv

RGhost::Config::GS[:charset_convert]=nil

stclara commented 12 years ago

Salve, mestre. Sem sucesso. Continua a mensagem.

[]'s

Stclara.

mhfs commented 12 years ago

Quem emite o warning é o próprio require 'iconv'. Esse warning foi adicionado no 1.9.3. Até o 1.9.2 não acontecia.

1.9.3 (main)> require 'iconv'
/Users/mhfs/.rbenv/versions/1.9.3-p125-perf/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require': iconv will be deprecated in the future, use String#encode instead.
=> true

Fiz esse fix do commit acima que parece resolver a questão. Desculpe se ficou meio feioso. :D

mhfs commented 12 years ago

@shairontoledo @stclara tiveram a chance de olhar o patch? :)

shairontoledo commented 12 years ago

Estou sem tempo de dar uma olhada nisso. Tem como fazer um pull request? Eu aprovo aqui.

mhfs commented 12 years ago

@shairontoledo criei a pull request como você pediu. Será que rola um release da gem? =D

Obrigado e abraços.

shairontoledo commented 12 years ago

Fiz o merge e gerei a versão 0.8.7.7, obrigado o/