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

Prism is broken with the latest nokogiri release #5

Open
taazza opened this issue Jun 12, 2010 · 10 comments
Open

Prism is broken with the latest nokogiri release #5

taazza opened this issue Jun 12, 2010 · 10 comments
Labels

Comments

@taazza
Copy link

taazza commented Jun 12, 2010

/home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/microformat/hcard.rb:60:in []': can't convert String into Integer (TypeError) from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/microformat/hcard.rb:60 from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism.rb:102:incall'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism.rb:102:in extract_from' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism.rb:109:inparse'
from /usr/local/lib/site_ruby/1.8/rubygems/custom_require.rb:31:in collect' from /home/test_prism/vendor/gems/ruby/1.8/gems/nokogiri-1.4.2/lib/nokogiri/xml/node.rb:402:incall'
from /home/test_prism/vendor/gems/ruby/1.8/gems/nokogiri-1.4.2/lib/nokogiri/xml/node.rb:402:in each' from /home/test_prism/vendor/gems/ruby/1.8/gems/nokogiri-1.4.2/lib/nokogiri/xml/node.rb:401:ineach'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism.rb:109:in collect' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism.rb:109:inparse'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:144:in get_properties' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:140:ineach_pair'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:140:in get_properties' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:175:into_h'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:195:in empty?' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:148:inget_properties'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:147:in reject' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:147:inget_properties'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:175:in to_h' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:165:in[]'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/posh/base.rb:92:in fn' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/lib/prism/microformat/hcard.rb:99:into_vcard'
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/bin/prism:95
from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/bin/prism:80:in parse_microformats' from /home/test_prism/vendor/gems/ruby/1.8/gems/prism-0.1.0/bin/prism:95 from bin/prism:3:inload'
from bin/prism:3

When we instruct Gem bundle to use 1.4.1 everything works fine ;) Might be better off declaring which version of the dependency you need.

@atd
Copy link

atd commented Jun 22, 2010

Same problem here, please fix it :-)

@otobrglez
Copy link

Same problem here. With Ruby 1.8.7-p330 and 1.9.2-p136.

@ghost
Copy link

ghost commented Apr 13, 2011

Aha. This explains a lot. Spent the last hour or so trying to figure out where I was going wrong. This explains it :-)

The fact that this is still broken after a year is somewhat disturbing. Has anyone made any headway with this?

@ghost
Copy link

ghost commented Apr 14, 2011

Fixed in my fork. Sent a pull request.

@josephholsten
Copy link

Could you please merge and release?

@mwunsch
Copy link
Owner

mwunsch commented Nov 8, 2012

Merged, but having trouble releasing b/c this gem is real old.

I do have plans for it though. @josephholsten: do you need this gem to be released or can you point to the git repo?

@josephholsten
Copy link

I'd like a release to complement my webfinger toolkit discodactyl.

Do you have the time to maintain this? I can certainly help if that's useful.

@mwunsch
Copy link
Owner

mwunsch commented Nov 8, 2012

Was hoping to rewrite it actually 😐

@josephholsten
Copy link

Awesome. If there's anything I can do to keep prism 0.1 on life support while you're rewriting, let me know. mofo is beyond help at this point, so prism really is the best thing around for microformats.

@mwunsch
Copy link
Owner

mwunsch commented Nov 8, 2012

Well, I merged @Nimlhug's pull request in, but am unable to run the tests and am seeing lots of bundler errors. If you feel like taking over the version 0.1.0 branch of prism and figure out how to lock down the dependencies to have it run easily (in Ruby 1.8.7), I can begin working on a 0.2.0 branch.

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

No branches or pull requests

5 participants