ruby-oembed 0.8.7 → 0.8.8

data/.gitignore

@@ -0,0 +1,7 @@
+/pkg
+/.bundle
+/rails_oembed
+/doc
+Gemfile.lock
+.yardoc
+.DS_Store

data/CHANGELOG.rdoc

@@ -1,5 +1,14 @@
 = CHANGELOG
 
+== 0.8.8 - 18 November 2012
+
+* OEmbed::Response::Photo#html now includes alt attribute; Pull #23 & #25 (François de Metz)
+* Always escape the url query param when requesting a resource from a provider; Pull #26 (Michael Cohen)
+* Allow query params in a provider endpoint URL; Pull #27 (Michael Andrews)
+* Added built-in provider for Skitch; Pull #24 (François de Metz)
+* Updated built-in Vimeo provider (Marcos Wright Kuhns)
+* For developers who work on this gem, removed the dependency on jeweler (Marcos Wright Kuhns)
+
 == 0.8.7 - 11 March 2012
 
 * Support for https provider endpoints; Issue #16 (Marcos Wright Kuhns)

data/Gemfile

@@ -1,12 +1,3 @@
-source :rubygems
+source "https://rubygems.org"
 
-group :development, :test do
-  gem 'jeweler'
-  gem 'rake'
-  gem 'json'
-  gem 'xml-simple'
-  gem 'nokogiri'
-  gem 'rspec', '>=2.0'
-  gem 'vcr', '~> 1.0'
-  gem 'fakeweb' # For use with vcr.
-end
+gemspec

data/Rakefile

@@ -1,29 +1,7 @@
-require File.expand_path(File.join(__FILE__, '../lib/oembed/version'))
-
 begin
-  require 'jeweler'
-  
-  Dir[File.join(File.dirname(__FILE__), "lib/tasks/*.rake")].sort.each { |ext| load ext }
-  
-  Jeweler::Tasks.new do |gemspec|
-    gemspec.name = "ruby-oembed"
-    gemspec.version = OEmbed::Version
-    gemspec.homepage = "http://github.com/judofyr/ruby-oembed"
-    gemspec.summary = "oEmbed for Ruby"
-    gemspec.description = "An oEmbed consumer library written in Ruby, letting you easily get embeddable HTML representations of supported web pages, based on their URLs. See http://oembed.com for more information about the protocol."
-    gemspec.license = "MIT"
-    gemspec.email = "arisbartee@gmail.com"
-    gemspec.authors = ["Magnus Holm","Alex Kessinger","Aris Bartee","Marcos Wright Kuhns"]
-    
-    gemspec.rdoc_options = %W(
-      --main README.rdoc
-      --title #{gemspec.full_name}
-      --inline-source
-      --exclude tasks
-      CHANGELOG.rdoc
-    )
-  end
-  Jeweler::RubygemsDotOrgTasks.new
+  require "bundler/gem_tasks"
 rescue LoadError
-  puts "Jeweler not available. Install it with: gem install jeweler"
+  puts "Bundler not available. Install it with: gem install bundler"
 end
+
+Dir[File.join(File.dirname(__FILE__), "lib/tasks/*.rake")].sort.each { |ext| load ext }

data/lib/oembed/provider.rb

@@ -1,3 +1,4 @@
+require 'cgi'
 module OEmbed
   # An OEmbed::Provider has information about an individual oEmbed enpoint.
   class Provider
@@ -94,7 +95,7 @@ module OEmbed
     def build(url, query = {})
       raise OEmbed::NotFound, url unless include?(url)
 
-      query = query.merge({:url=>url})
+      query = query.merge({:url => ::CGI.escape(url)})
       # TODO: move this code exclusively into the get method, once build is private.
       this_format = (query[:format] ||= @format.to_s).to_s
       
@@ -105,7 +106,8 @@ module OEmbed
         query.delete(:format)
       end
 
-      query = "?" + query.inject("") do |memo, (key, value)|
+      base = endpoint.include?('?') ? '&' : '?'
+      query = base + query.inject("") do |memo, (key, value)|
         "#{key}=#{value}&#{memo}"
       end.chop
 

data/lib/oembed/providers/embedly_urls.yml

@@ -19,6 +19,9 @@
 - http://*.dipdive.com/media/*
 - http://*.dipdive.com/v/*
 - http://*.kinomap.com/*
+- http://*.linkedin.com/in/*
+- http://*.linkedin.com/pub/*
+- http://*.looplogic.com/*
 - http://*.posterous.com/*
 - http://*.smugmug.com/*
 - http://*.smugmug.com/*#*
@@ -28,6 +31,10 @@
 - http://*.tumblr.com/post/*
 - http://*.twitrpix.com/*
 - http://*.uservoice.com/*/suggestions/*
+- http://*.vzaar.me/*
+- http://*.wi.st/*
+- http://*.wikipedia.org/wiki/*
+- http://*.wistia.com/*
 - http://*.youtube.com/*#*/*
 - http://*.youtube.com/playlist*
 - http://*.youtube.com/profile*
@@ -55,10 +62,11 @@
 - http://*twitch.tv/*
 - http://*twitch.tv/*/b/*
 - http://*twitvid.com/*
-- http://*viddler.com/explore/*/videos/*
+- http://*viddler.com/v/*
 - http://*yfrog.*/*
 - http://*youtube.com/watch*
 - http://23hq.com/*/photo/*
+- http://360.io/*
 - http://4sq.com/*
 - http://99dollarmusicvideos.com/*/episode/*
 - http://99dollarmusicvideos.com/episode/*
@@ -72,7 +80,9 @@
 - http://aniboom.com/animation-video/*
 - http://animal.discovery.com/videos/*
 - http://animoto.com/play/*
+- http://answers.polldaddy.com/poll/*
 - http://api.shopstyle.com/action/apiVisitRetailer*
+- http://app.sliderocket.com/*
 - http://app.wistia.com/embed/medias/*
 - http://asofterworld.com/*.jpg
 - http://audioboo.fm/boos/*
@@ -83,6 +93,8 @@
 - http://barelydigital.com/episode/*
 - http://barelypolitical.com/*/episode/*
 - http://barelypolitical.com/episode/*
+- http://behance.net/gallery/*
+- http://beta-sliderocket.com/*
 - http://bigthink.com/ideas/*
 - http://bigthink.com/series/*
 - http://blip.tv/*/*
@@ -107,13 +119,18 @@
 - http://collegehumor.com/video/*
 - http://collegehumor.com/video:*
 - http://color.com/s/*
+- http://confreaks.com/videos/*
 - http://confreaks.net/videos/*
+- http://coub.com/embed/*
+- http://coub.com/view/*
 - http://crocodoc.com/*
 - http://crunchbase.com/*/*
+- http://d.pr/i/*
 - http://dailybooth.com/*/*
 - http://dipdive.com/media/*
 - http://dipdive.com/member/*/media/*
 - http://dipdive.com/v/*
+- http://distrify.com/film/*
 - http://dotsub.com/view/*
 - http://drbl.in/*
 - http://dsc.discovery.com/videos/*
@@ -136,13 +153,12 @@
 - http://freemusicarchive.org/music/*
 - http://funnyordie.com/m/*
 - http://funnyordie.com/videos/*
-- http://gametrailers.com/video/*
+- http://gametrailers.com/video*
 - http://gist.github.com/*
 - http://godtube.com/featured/video/*
 - http://godtube.com/watch/*
-- http://google.com/buzz/*
-- http://google.com/buzz/*/*/*
 - http://google.com/profiles/*
+- http://gopollgo.com/*
 - http://graphicly.com/*/*/*
 - http://grindtv.com/*/video/*
 - http://grooveshark.com/*
@@ -168,8 +184,13 @@
 - http://investigation.discovery.com/videos/*
 - http://issuu.com/*/docs/*
 - http://itunes.apple.com/*
+- http://jdsupra.com/legalnews/*
+- http://jibjab.com/view/*
+- http://khanacademy.org/*
 - http://lightbox.com/*
 - http://link.brightcove.com/services/player/bcpid*
+- http://linkedin.com/in/*
+- http://linkedin.com/pub/*
 - http://liveleak.com/view?*
 - http://lockerz.com/s/*
 - http://logotv.com/video/*
@@ -196,10 +217,14 @@
 - http://movies.yahoo.com/movie/*/trailer
 - http://movies.yahoo.com/movie/*/video
 - http://movies.yahoo.com/movie/*/video/*
+- http://muvi.es/*
 - http://my.opera.com/*/albums/show.dml?id=*
 - http://my.opera.com/*/albums/showpic.dml?album=*&picture=*
 - http://myloc.me/*
 - http://nzonscreen.com/title/*
+- http://on.aol.com/playlist/*
+- http://on.aol.com/video/*
+- http://open.spotify.com/*
 - http://ow.ly/i/*
 - http://pastebin.com/*
 - http://pastie.org/*
@@ -210,13 +235,17 @@
 - http://picasaweb.google.com*/*/*
 - http://picasaweb.google.com*/*/*#*
 - http://picasaweb.google.com*/lh/photo/*
-- http://picplz.com/*
 - http://pics.brizzly.com/*.jpg
+- http://pics.lockerz.com/s/*
 - http://pikchur.com/*
 - http://ping.fm/p/*
 - http://pinterest.com/pin/*
 - http://pixorial.com/watch/*
 - http://planetgreen.discovery.com/videos/*
+- http://plus.google.com/*
+- http://polldaddy.com/community/poll/*
+- http://polldaddy.com/poll/*
+- http://portal.sliderocket.com/*
 - http://post.ly/*
 - http://prezi.com/*/*
 - http://qik.com/*
@@ -243,9 +272,10 @@
 - http://sendables.jibjab.com/originals/*
 - http://sendables.jibjab.com/view/*
 - http://share.ovi.com/media/*/*
-- http://shitmydadsays.com/notice/*
+- http://showme.com/sh/*
 - http://siteanalytics.compete.com/*
 - http://skitch.com/*/*/*
+- http://sliderocket.com/*
 - http://slidesha.re/*
 - http://snd.sc/*
 - http://snotr.com/video/*
@@ -258,6 +288,7 @@
 - http://soundcloud.com/groups/*
 - http://speakerdeck.com/u/*/p/*
 - http://sports.yahoo.com/video/*
+- http://spreecast.com/events/*
 - http://statsheet.com/*/teams/*/*
 - http://statsheet.com/statplot/charts/*/*/*/*
 - http://statsheet.com/statplot/charts/e/*
@@ -265,9 +296,7 @@
 - http://storify.com/*/*
 - http://streetfire.net/video/*.htm*
 - http://teachertube.com/viewVideo.php*
-- http://techcrunch.tv/*/watch*
-- http://techcrunch.tv/show/*/*
-- http://techcrunch.tv/watch*
+- http://telly.com/*
 - http://theonion.com/video/*
 - http://threadbanger.com/*/episode/*
 - http://threadbanger.com/episode/*
@@ -302,10 +331,11 @@
 - http://video.forbes.com/fvn/*
 - http://video.google.com/videoplay?*
 - http://video.jardenberg.se/*
-- http://video.nationalgeographic.com/*/*/*.html
+- http://video.nationalgeographic.com/video/*/*/*/*
 - http://video.pbs.org/video/*
 - http://video.yahoo.com/network/*
 - http://video.yahoo.com/watch/*/*
+- http://videodetective.com/*/*
 - http://videos.nymag.com/*
 - http://vids.myspace.com/index.cfm?fuseaction=vids.individual&videoid*
 - http://vimeo.com/*
@@ -313,10 +343,15 @@
 - http://vimeo.com/m/#/*
 - http://vodcars.com/*/episode/*
 - http://vodcars.com/episode/*
+- http://vzaar.com/videos/*
+- http://vzaar.me/*
+- http://vzaar.tv/*
 - http://washingtonpost.com/wp-dyn/*/video/*/*/*/*
 - http://wh.gov/photos-and-video/video/*
 - http://wh.gov/video/*
+- http://wi.st/*
 - http://wirewax.com/*
+- http://wistia.com/*
 - http://wordpress.tv/*/*/*/*/
 - http://worldstarhiphop.com/videos/video*.php?v=*
 - http://www.23hq.com/*/photo/*
@@ -335,16 +370,17 @@
 - http://www.barelydigital.com/episode/*
 - http://www.barelypolitical.com/*/episode/*
 - http://www.barelypolitical.com/episode/*
+- http://www.behance.net/gallery/*
 - http://www.boston.com/*video*
 - http://www.boston.com/video*
 - http://www.bravotv.com/*/*/videos/*
-- http://www.break.com/*/*
 - http://www.canalplus.fr/*
 - http://www.channelfrederator.com/*/episode/*
 - http://www.channelfrederator.com/episode/*
 - http://www.clikthrough.com/theater/video/*
 - http://www.clipfish.de/*/*/video/*
 - http://www.clipshack.com/Clip.aspx?*
+- http://www.clipsyndicate.com/video/playlist/*/*
 - http://www.cnbc.com/id/*/play/1/video/*
 - http://www.cnbc.com/id/*?*video*
 - http://www.cnn.com/video/*
@@ -354,10 +390,14 @@
 - http://www.collegehumor.com/video/*
 - http://www.collegehumor.com/video:*
 - http://www.comedycentral.com/videos/index.jhtml?*
+- http://www.confreaks.com/videos/*
 - http://www.confreaks.net/videos/*
 - http://www.crunchbase.com/*/*
 - http://www.dailymile.com/people/*/entries/*
 - http://www.escapistmagazine.com/videos/*
+- http://www.eyeem.com/a/*
+- http://www.eyeem.com/p/*
+- http://www.eyeem.com/u/*
 - http://www.facebook.com/photo.php*
 - http://www.facebook.com/v/*
 - http://www.facebook.com/video/video.php*
@@ -371,15 +411,14 @@
 - http://www.freemusicarchive.org/music/*
 - http://www.funnyordie.com/m/*
 - http://www.funnyordie.com/videos/*
-- http://www.gametrailers.com/video/*
+- http://www.gametrailers.com/video*
 - http://www.globalpost.com/dispatch/*
 - http://www.globalpost.com/video/*
 - http://www.godtube.com/featured/video/*
 - http://www.godtube.com/watch/*
 - http://www.gogoyoko.com/song/*
-- http://www.google.com/buzz/*
-- http://www.google.com/buzz/*/*/*
 - http://www.google.com/profiles/*
+- http://www.gopollgo.com/*
 - http://www.grindtv.com/*/video/*
 - http://www.guardian.co.uk/*/video/*/*/*/*
 - http://www.hark.com/clips/*
@@ -394,6 +433,8 @@
 - http://www.ifood.tv/video/*
 - http://www.indymogul.com/*/episode/*
 - http://www.indymogul.com/episode/*
+- http://www.jdsupra.com/legalnews/*
+- http://www.khanacademy.org/*
 - http://www.kickstarter.com/projects/*/*
 - http://www.kiva.org/lend/*
 - http://www.koldcast.tv/#video:*
@@ -468,6 +509,8 @@
 - http://www.shopstyle.com/action/apiVisitRetailer*
 - http://www.shopstyle.com/action/viewLook*
 - http://www.shopstyle.com/browse*
+- http://www.showme.com/sh/*
+- http://www.sliderocket.com/*
 - http://www.slideshare.net/*/*
 - http://www.slideshare.net/mobile/*/*
 - http://www.snotr.com/video/*
@@ -475,6 +518,7 @@
 - http://www.some.ly/*
 - http://www.someecards.com/*/*
 - http://www.spike.com/video/*
+- http://www.spreecast.com/events/*
 - http://www.streetfire.net/video/*.htm*
 - http://www.studivz.net/*
 - http://www.studivz.net/Gadgets/Info/*
@@ -488,6 +532,7 @@
 - http://www.ted.com/index.php/talks/lang/*/*.html*
 - http://www.ted.com/talks/*.html*
 - http://www.ted.com/talks/lang/*/*.html*
+- http://www.telly.com/*
 - http://www.thedailyshow.com/collection/*/*/*
 - http://www.thedailyshow.com/full-episodes/*
 - http://www.thedailyshow.com/watch/*
@@ -521,16 +566,21 @@
 - http://www.ustream.tv/recorded/*
 - http://www.vevo.com/video/*
 - http://www.vevo.com/watch/*
+- http://www.videodetective.com/*/*
+- http://www.viewrz.com/*/event/*/*
+- http://www.viewrz.com/event/*/*
 - http://www.vimeo.com/*
 - http://www.vimeo.com/groups/*/videos/*
 - http://www.vodcars.com/*/episode/*
 - http://www.vodcars.com/episode/*
+- http://www.vzaar.com/videos/*
+- http://www.vzaar.tv/*
 - http://www.washingtonpost.com/wp-dyn/*/video/*/*/*/*
 - http://www.whitehouse.gov/photos-and-video/video/*
 - http://www.whitehouse.gov/video/*
-- http://www.whosay.com/content/*
-- http://www.whosay.com/photos/*
-- http://www.whosay.com/videos/*
+- http://www.whosay.com/*/content/*
+- http://www.whosay.com/*/photos/*
+- http://www.whosay.com/*/videos/*
 - http://www.wikimedia.org/wiki/File*
 - http://www.wikipedia.org/wiki/*
 - http://www.wirewax.com/*
@@ -546,18 +596,23 @@
 - http://xkcd.com/*
 - http://youtu.be/*
 - https://*.crocodoc.com/*
+- https://*.looplogic.com/*
+- https://*.wi.st/*
+- https://*.wistia.com/*
 - https://*.youtube.com/v/*
 - https://*youtube.com/watch*
 - https://app.wistia.com/embed/medias/*
 - https://crocodoc.com/*
 - https://foursquare.com/*
 - https://ganxy.com/*
+- https://gist.github.com/*
 - https://img.skitch.com/*
 - https://itunes.apple.com/*
 - https://mobile.twitter.com/*/status/*
 - https://mobile.twitter.com/*/status/*/photo/*
 - https://mobile.twitter.com/*/statuses/*
 - https://mobile.twitter.com/*/statuses/*/photo/*
+- https://plus.google.com/*
 - https://skitch.com/*/*/*
 - https://twitter.com/*/status/*
 - https://twitter.com/*/status/*/photo/*
@@ -565,6 +620,9 @@
 - https://twitter.com/*/statuses/*/photo/*
 - https://urtak.com/clr/*
 - https://urtak.com/u/*
+- https://vimeo.com/*
+- https://wi.st/*
+- https://wistia.com/*
 - https://www.facebook.com/photo.php*
 - https://www.facebook.com/v/*
 - https://www.facebook.com/video/video.php*
@@ -574,3 +632,4 @@
 - https://www.twitter.com/*/status/*/photo/*
 - https://www.twitter.com/*/statuses/*
 - https://www.twitter.com/*/statuses/*/photo/*
+- https://www.vimeo.com/*

data/lib/oembed/providers.rb

@@ -43,7 +43,7 @@ module OEmbed
 
       # Register all Providers built into this gem.
       # The including_sub_type parameter should be one of the following values:
-      # * :aggregators: also register provider agregator endpoints, like Embedly
+      # * :aggregators: also register provider aggregator endpoints, like Embedly
       def register_all(*including_sub_type)
         register(*@@to_register[""])
         including_sub_type.each do |sub_type|
@@ -128,8 +128,14 @@ module OEmbed
     
     # Provider for youtube.com
     # http://apiblog.youtube.com/2009/10/oembed-support.html
-    # To get the iframe embed code, instead of flash-based, pass
-    # :iframe=>1 as a query in your get request.
+    #
+    # Options:
+    # * To get the iframe embed code
+    #     OEmbed::Providers::Youtube.endpoint += "?iframe=1"
+    # * To get the flash/object embed code
+    #     OEmbed::Providers::Youtube.endpoint += "?iframe=0"
+    # * To require https embed code
+    #     OEmbed::Providers::Youtube.endpoint += "?scheme=https"
     Youtube = OEmbed::Provider.new("http://www.youtube.com/oembed")
     Youtube << "http://*.youtube.com/*"
     Youtube << "https://*.youtube.com/*"
@@ -167,8 +173,8 @@ module OEmbed
     add_official_provider(Hulu)
 
     # Provider for vimeo.com
-    # http://vimeo.com/api/docs/oEmbed
-    Vimeo = OEmbed::Provider.new("http://www.vimeo.com/api/oembed.{format}")
+    # http://developer.vimeo.com/apis/oembed
+    Vimeo = OEmbed::Provider.new("http://vimeo.com/api/oembed.{format}")
     Vimeo << "http://*.vimeo.com/*"
     Vimeo << "https://*.vimeo.com/*"
     add_official_provider(Vimeo)
@@ -249,6 +255,13 @@ module OEmbed
     SoundCloud << "http://*.soundcloud.com/*"
     add_official_provider(SoundCloud)
 
+    # Provider for skitch.com
+    # http://skitch.com/oembed/%3C/endpoint
+    Skitch = OEmbed::Provider.new("http://skitch.com/oembed")
+    Skitch << "http://*.skitch.com/*"
+    Skitch << "https://*.skitch.com/*"
+    add_official_provider(Skitch)
+
     ## Provider for clikthrough.com
     # http://corporate.clikthrough.com/wp/?p=275
     #Clickthrough = OEmbed::Provider.new("http://www.clikthrough.com/services/oembed/")
@@ -261,7 +274,7 @@ module OEmbed
     #Kinomap << "http://www.kinomap.com/*"
     #add_official_provider(Kinomap)
 
-    # Provider for oohembed.com, which is a provider agregator. See
+    # Provider for oohembed.com, which is a provider aggregator. See
     # OEmbed::Providers::OohEmbed.urls for a full list of supported url schemas.
     # Embed.ly has taken over the oohembed.com domain and as of July 20 all oohEmbed
     # request will require you use an API key. For details on the transition see
@@ -301,11 +314,14 @@ module OEmbed
     OohEmbed << %r{http://yfrog.(com|ru|com.tr|it|fr|co.il|co.uk|com.pl|pl|eu|us)/(.*?)} # image & video hosting
     OohEmbed << "http://*.youtube.com/watch*"
 
-    # Provider for Embedly.com, which is a provider agregator. See
+    # Provider for Embedly.com, which is a provider aggregator. See
     # OEmbed::Providers::Embedly.urls for a full list of supported url schemas.
     # http://embed.ly/docs/endpoints/1/oembed
-    # You'll need to add your Embed.ly API key to each request as the "key"
-    # parameter. To get an API key you'll need to sign up here: http://embed.ly/pricing
+    #
+    # You can append your Embed.ly API key to the provider so that all requests are signed
+    #     OEmbed::Providers::Embedly.endpoint += "?key=#{my_embedly_key}"
+    # 
+    # If you don't yet have an API key you'll need to sign up here: http://embed.ly/pricing
     Embedly = OEmbed::Provider.new("http://api.embed.ly/1/oembed")
     # Add all known URL regexps for Embedly. To update this list run `rake oembed:update_embedly`
     YAML.load_file(File.join(File.dirname(__FILE__), "/providers/embedly_urls.yml")).each do |url|

data/lib/oembed/response/photo.rb

@@ -4,7 +4,7 @@ module OEmbed
     class Photo < self
       # Returns an <img> tag pointing at the appropraite URL.
       def html
-        "<img src='" + self.url + "' />"
+        "<img src='#{self.url}' alt='#{(self.respond_to?(:title) ? self.title : nil)}' />"
       end
       
       private
@@ -18,4 +18,4 @@ module OEmbed
       
     end
   end
-end
+end

data/lib/oembed/version.rb

@@ -2,7 +2,7 @@ module OEmbed
   class Version
     MAJOR = 0
     MINOR = 8
-    PATCH = 7
+    PATCH = 8
     STRING = "#{MAJOR}.#{MINOR}.#{PATCH}"
     
     class << self

data/lib/tasks/oembed.rake

@@ -14,7 +14,7 @@ begin
     
       url_regexps = []
       services.each do |service|
-        url_regexps += service['regex']
+        url_regexps += service['regex'].map{|r| r.strip }
       end
       url_regexps.sort!
     

data/ruby-oembed.gemspec

@@ -1,88 +1,36 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
 
+lib = File.expand_path('../lib/', __FILE__)
+$:.unshift lib unless $:.include?(lib)
+
+require 'oembed/version'
+
 Gem::Specification.new do |s|
   s.name = "ruby-oembed"
-  s.version = "0.8.7"
+  s.version = OEmbed::Version.to_s
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Magnus Holm", "Alex Kessinger", "Aris Bartee", "Marcos Wright Kuhns"]
-  s.date = "2012-03-12"
+  s.date = "2012-11-19"
   s.description = "An oEmbed consumer library written in Ruby, letting you easily get embeddable HTML representations of supported web pages, based on their URLs. See http://oembed.com for more information about the protocol."
   s.email = "arisbartee@gmail.com"
-  s.extra_rdoc_files = [
-    "LICENSE",
-    "README.rdoc"
-  ]
-  s.files = [
-    ".rvmrc",
-    ".travis.yml",
-    ".yardopts",
-    "CHANGELOG.rdoc",
-    "Gemfile",
-    "Gemfile.lock",
-    "LICENSE",
-    "README.rdoc",
-    "Rakefile",
-    "integration_test/test.rb",
-    "integration_test/test_urls.csv",
-    "lib/oembed.rb",
-    "lib/oembed/errors.rb",
-    "lib/oembed/formatter.rb",
-    "lib/oembed/formatter/base.rb",
-    "lib/oembed/formatter/json.rb",
-    "lib/oembed/formatter/json/backends/activesupportjson.rb",
-    "lib/oembed/formatter/json/backends/jsongem.rb",
-    "lib/oembed/formatter/json/backends/yaml.rb",
-    "lib/oembed/formatter/xml.rb",
-    "lib/oembed/formatter/xml/backends/nokogiri.rb",
-    "lib/oembed/formatter/xml/backends/rexml.rb",
-    "lib/oembed/formatter/xml/backends/xmlsimple.rb",
-    "lib/oembed/provider.rb",
-    "lib/oembed/provider_discovery.rb",
-    "lib/oembed/providers.rb",
-    "lib/oembed/providers/embedly_urls.yml",
-    "lib/oembed/providers/oohembed_urls.yml",
-    "lib/oembed/response.rb",
-    "lib/oembed/response/link.rb",
-    "lib/oembed/response/photo.rb",
-    "lib/oembed/response/rich.rb",
-    "lib/oembed/response/video.rb",
-    "lib/oembed/version.rb",
-    "lib/tasks/oembed.rake",
-    "lib/tasks/rspec.rake",
-    "ruby-oembed.gemspec",
-    "spec/cassettes/OEmbed_Provider.yml",
-    "spec/cassettes/OEmbed_ProviderDiscovery.yml",
-    "spec/formatter/ducktype_backend_spec.rb",
-    "spec/formatter/json/.DS_Store",
-    "spec/formatter/json/jsongem_backend_spec.rb",
-    "spec/formatter/json/yaml_backend_spec.rb",
-    "spec/formatter/xml/nokogiri_backend_spec.rb",
-    "spec/formatter/xml/rexml_backend_spec.rb",
-    "spec/formatter/xml/xmlsimple_backend_spec.rb",
-    "spec/formatter_spec.rb",
-    "spec/provider_discovery_spec.rb",
-    "spec/provider_spec.rb",
-    "spec/providers_spec.rb",
-    "spec/response_spec.rb",
-    "spec/spec_helper.rb",
-    "spec/spec_helper_examples.yml"
-  ]
   s.homepage = "http://github.com/judofyr/ruby-oembed"
   s.licenses = ["MIT"]
-  s.rdoc_options = ["--main", "README.rdoc", "--title", "ruby-oembed-0.8.7", "--inline-source", "--exclude", "tasks", "CHANGELOG.rdoc"]
+  
+  s.files = `git ls-files`.split("\n")
+  s.test_files = s.files.grep(%r{^(test|spec|features,integration_test)/})
+  
+  s.rdoc_options = ["--main", "README.rdoc", "--title", "ruby-oembed-#{OEmbed::Version}", "--inline-source", "--exclude", "tasks", "CHANGELOG.rdoc"]
+  s.extra_rdoc_files = s.files.grep(%r{\.rdoc$}) + %w{LICENSE}
+  
   s.require_paths = ["lib"]
-  s.rubygems_version = "1.8.10"
+  s.rubygems_version = "1.8.19"
   s.summary = "oEmbed for Ruby"
 
   if s.respond_to? :specification_version then
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<jeweler>, [">= 0"])
       s.add_development_dependency(%q<rake>, [">= 0"])
       s.add_development_dependency(%q<json>, [">= 0"])
       s.add_development_dependency(%q<xml-simple>, [">= 0"])
@@ -91,7 +39,6 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<vcr>, ["~> 1.0"])
       s.add_development_dependency(%q<fakeweb>, [">= 0"])
     else
-      s.add_dependency(%q<jeweler>, [">= 0"])
       s.add_dependency(%q<rake>, [">= 0"])
       s.add_dependency(%q<json>, [">= 0"])
       s.add_dependency(%q<xml-simple>, [">= 0"])
@@ -101,7 +48,6 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<fakeweb>, [">= 0"])
     end
   else
-    s.add_dependency(%q<jeweler>, [">= 0"])
     s.add_dependency(%q<rake>, [">= 0"])
     s.add_dependency(%q<json>, [">= 0"])
     s.add_dependency(%q<xml-simple>, [">= 0"])