bagger 0.6.0 → 0.6.1

data/CHANGELOG.md

@@ -1,4 +1,11 @@
 # CHANGELOG
+# v 0.6.1
+
+  * improve CSS URL rewriting
+  * fix rewriting for certain multi-URL properties
+  * correctly rewrite IE behavior URLs unless the reference objects or default behaviors
+
+  All changes by Martin Rehfeld
 
 # v 0.6.0
   * Add option to add the manifest itself to the manifest (Christian

data/README.md

@@ -33,8 +33,7 @@ on the client side by using the following techniques:
 	source_dir = "/applications/my_app/public"
   
 	# customize paths for file and cache manifest
-	manifest_path = File.join(target_dir, 'file_manifest.json') #defaults
-	to manifest.json
+	manifest_path = File.join(target_dir, 'file_manifest.json') # defaults to manifest.json
 
 	cache_manifest_path = 'cache/cache.manifest' # defaults to cache.manifest
 	
@@ -44,8 +43,8 @@ on the client side by using the following techniques:
 	stylesheets = ["css/style.css", "css/reset.css"]
 	javascripts = ["js/app.js", "js/utils.js"]
 
-	# define cache manifest bundles for directorfferent devices
-	# for convencien, the manfiest defined with `cache_manifest_path`
+	# define cache manifest bundles for different devices
+	# for convenience, the manifest defined with `cache_manifest_path`
 	# will always contain all resources
 	ipad_resources = ["images/troll-big.png"]
 	iphone_retina_resources = ["images/troll-retina.png"]
@@ -130,4 +129,3 @@ check the build status on [travis.ci](http://travis-ci.org/wooga/bagger)
 
 * [jammit](https://github.com/documentcloud/jammit)
 * [Rails 3 asset pipeline](http://blog.nodeta.com/2011/06/14/rails-3-1-asset-pipeline-in-the-real-world/)
-* [assets.io](http://www.assets.io/)

data/lib/bagger/packager.rb

@@ -105,16 +105,15 @@ module Bagger
     end
 
     def rewrite_urls_in_css(stylesheet_path)
-      url_regex = /(^|[{;,])(.*?url\(\s*['"]?)(.*?)(['"]?\s*\).*?)([,;}]|$)/ui
-      behavior_regex = /behavior:\s*url/ui
+      url_regex = /(url\(\s*['"]?)([^#].*?)(['"]?\s*\).*?)/ui
       data_regex = /^\s*data:/ui
       input = File.open(File.join(@target_dir, stylesheet_path)){|f| f.read}
       output = input.gsub(url_regex) do |full_match|
-        pre, url_match, post = ($1 + $2), $3, ($4 + $5)
-        if behavior_regex.match(pre) || data_regex.match(url_match)
+        pre, url_match, post = $1, $2, $3
+        if data_regex.match(url_match)
           full_match
         else
-          path = Addressable::URI.parse("/") + url_match
+          path = Addressable::URI.parse('/') + url_match
           target_url = @manifest[path.to_s]
           if target_url
             pre + @css_path_prefix + target_url + post
@@ -154,7 +153,7 @@ module Bagger
     def generate_cache_manifests
       cache_manifests = @options[:cache_manifests] || []
       if @cache_manifest_path
-        cache_manifests << { 
+        cache_manifests << {
           :target_path => @cache_manifest_path,
           :files => @manifest.keys
         }

data/lib/bagger/version.rb

@@ -1,3 +1,3 @@
 module Bagger
-  VERSION = "0.6.0"
+  VERSION = "0.6.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bagger
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-06 00:00:00.000000000 Z
+date: 2013-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -165,7 +165,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: bagger
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Asset packaging for the rest of us