Synthesis: AssetPackager released!

AssetPackager released!

sbecker — Mon, 19 Jun 2006 09:33:00 -0700

Asset Packager has been released! (Formerly known as MergeJS) New features include:

support for css files
versioning of individual packages
use of more meaningful subversion revision numbers (if available) (thanks Chris Van Pelt!)
namespaced rake tasks
no more revision numbers in the yaml file
lotsa refactoring
unit tests
more intuitive names for everything!

Go here to check it out: AssetPackager

"AssetPackager released!" by Scott Johnson

Wed, 28 Oct 2009 11:24:39 -0700

It seems the minify_cache (posted by Jason on 3/16/09) is now part of Rails core. So the merging aspect of this plugin is no longer necessary.

I set up a cap task to run yuicompressor on each *.js and *.css file, then use javascript_include_tag :cache => true to make Rails combine them into one.

"AssetPackager released!" by Dave Mauldin

Sat, 04 Apr 2009 17:31:17 -0700

So, I converted all of the regexp literals in jquery.tablesorter.js from “return /foo/.test(s)” to “return RegExp(/foo/).test(s)” and of course the script still works fine if loaded by itself and now it doesn’t raise an exception in jsmin.rb. Though I still can’t figure out why it’s not working properly. Every other script works fine, but tablesorter 2.0 just fails to work, with no errors (that I can see with firebug anyway). It’s one of those things that seems nearly impossible to troubleshoot. sigh

Anybody have any suggestions for how to troubleshoot such a problem? I can’t tell if this is a jsmin problem or poorly written js being passed in.

"AssetPackager released!" by Jason

Mon, 16 Mar 2009 08:55:55 -0700

Check out minify_cache to get a nice monkey patch to rails javascript_include_tag….:cache =>.. for automatically minifying the output.

"AssetPackager released!" by Hugo

Thu, 12 Feb 2009 11:55:32 -0800

I had to add require 'synthesis/asset_package' to asset_package_helper.rb as I run without automatic dependency loading.

Also jsmin.rb breaks on javascript like return /regexp/.test(string);, which I hit while processing jquery.tablesorter.js. It seems not to recognise the first / as the start of a regexp.

"AssetPackager released!" by tyan

Thu, 25 Dec 2008 05:37:00 -0800

Hi, is it possible to get this to work in development in would save me some serious time?

"AssetPackager released!" by gil

Fri, 05 Dec 2008 12:25:47 -0800

Also ran into the 2.2 problem with rails_asset_id, there’s also another problem with relative_url_root

"AssetPackager released!" by Dav

Thu, 04 Dec 2008 17:19:49 -0800

The definition of the javascript_include_merged uses a hard coded array as the list of default javascripts. This ignores any new defaults added by the application or plugins via ActionView::Helpers::AssetTagHelper#register_javascript_include_defaults@.I changed the hardcoded array to


ActionView::Helpers::AssetTagHelper.send(:class_variable_get,'@@javascript_expansions'.to_sym)[:defaults]

to get this working. Sorry no time for a git patch right now.

In the end this didn’t really matter though, as I quit using :defaults at all since using that rails shortcut means that the defaults will not be part of the asset package. If I understand it correctly, if you really want only one javascript file, you need to hand copy the javascripts :defaults into the asset_packages.yml anyhow so it will be up to you to resolve any non-standard defaults there.

"AssetPackager released!" by Rich Manalang

Mon, 24 Nov 2008 09:58:51 -0800

Hi Scott. Looks like asset_packager is busted under Rails 2.2:

undefined method `rails_asset_id’ for #

In Rails 2.2, it looks like rails_asset_id became a private method.

"AssetPackager released!" by suhrawardi

Thu, 20 Nov 2008 06:45:18 -0800

I had a little problem with the fact that the rewrite of the compute_public_path function does not take into account the “has_request = @controller.respond_to?(:request)” part from the original function. This made the rails image_path function fail in an email template with inline attachments.

A simple rename of this rewritten compute_public_path function did solve this:


diff --git a/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb b/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb
index 31d0958..df2d3b7 100644
--- a/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb
+++ b/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb
@@ -41,7 +41,7 @@ module Synthesis
     private
       # rewrite compute_public_path to allow us to not include the query string timestamp
       # used by ActionView::Helpers::AssetTagHelper
-      def compute_public_path(source, dir, ext=nil, add_asset_id=true)
+      def asset_packager_compute_public_path(source, dir, ext=nil, add_asset_id=true)
         source = source.dup
         source << ".#{ext}" if File.extname(source).blank? && ext
         unless source =~ %r{^[-a-z]+://}
@@ -55,13 +55,13 @@ module Synthesis

       # rewrite javascript path function to not include query string timestamp
       def javascript_path(source)
-        compute_public_path(source, 'javascripts', 'js', false)       
+        asset_packager_compute_public_path(source, 'javascripts', 'js', false) 
       end

       # rewrite stylesheet path function to not include query string timestamp
       def stylesheet_path(source)
-        compute_public_path(source, 'stylesheets', 'css', false)
+        asset_packager_compute_public_path(source, 'stylesheets', 'css', false)
       end

   end
-end
\ No newline at end of file
+end

"AssetPackager released!" by Neil

Thu, 02 Oct 2008 01:18:58 -0700

roobnoob, are you using this with a staging environment? And is anyone using Git instead of SVN for the versions the rake task creates?

"AssetPackager released!" by Neil

Thu, 02 Oct 2008 01:18:48 -0700

roobnoob, are you using this with a staging environment? And is anyone using Git instead of SVN for the versions the rake task creates?

"AssetPackager released!" by roobnoob

Thu, 28 Aug 2008 09:27:28 -0700

Does this work in other environments besides “production”? I’d like to enable this in another env, is there an option to do so?

"AssetPackager released!" by Paddy

Fri, 08 Aug 2008 05:17:48 -0700

Is this compatible with rails 1.1.6? Please do let me know.

When Installed it doesn’t boot my application and i don’t see any error in the log file.

Please do let me know.

Thank you! Paddy

"AssetPackager released!" by Paddy

Fri, 08 Aug 2008 05:17:38 -0700

Is this compatible with rails 1.1.6? Please do let me know.

When Installed it doesn’t boot my application and i don’t see any error in the log file.

Please do let me know.

Thank you! Paddy

"AssetPackager released!" by icebofh@gmail.com

Sat, 19 Jul 2008 22:38:00 -0700

Not sure if asset_packager does this already, but it would be nice to be able to specify a cache subdirectory (a la rails2) so that it is easier to clean up the directory and you lessen the change of accidentally deleting the source file.

"AssetPackager released!" by Alex Le

Thu, 05 Jun 2008 11:40:20 -0700

I hacked the merged_file method to support files with with general filename

With this hack, I can put in the full filename in the yml file and the builder doesn’t complain about the file not exists

def merged_file
  merged_file = "" 
  @sources.each {|s| 
    # hack by alex
    fn = "#{@asset_path}/#{s}" 
    fn << ".#{@extension}" unless fn.downcase.split( '.' ).last == @extension.downcase
    File.open( fn, "r") { |f| 
      merged_file += f.read + "\n" 
    }
  }
  merged_file
end

"AssetPackager released!" by roger

Mon, 02 Jun 2008 09:16:36 -0700

Is this plugin useful in rails 2? (see http://maintainable.com/articles/rails_asset_cache)? Thanks!

"AssetPackager released!" by Klaus Hartl

Wed, 14 May 2008 06:53:36 -0700

@Joel that worked better for me (based on r57):


source << ".#{ext}" if File.extname(source).blank? && ext || ext =~ /css|js/ && File.extname(source).delete('.') != ext

"AssetPackager released!" by Klaus Hartl

Wed, 14 May 2008 02:58:25 -0700

I am getting a “Service Temporarily Unavailable” error for the tracker, thus I’m posting it here:

I recently ran into a subtle bug with compressed style sheets. If you have a comment in a property or value without a following white space like this:

property: /**/value;

the parser will not see the end of this comment and instead strip away everything up until it encounters the next closing comment.

"AssetPackager released!" by Joel

Sat, 29 Mar 2008 06:20:51 -0700

Seems like the compute_public_path has some trouble with file names containing periods (common in jquery libs among others).

The .js either does not get appended (in development) or if u explicit set it in the asset_packages.yml it gets appended even if it should not.

Hackfix:


  line 41: source << ".#{ext}" if File.extname(source).blank?
  === changed to:
  source << ".#{ext}" if File.extname(source).blank? || (!ext.nil? && File.extname(source) != ext)

Seems to fix it.

"AssetPackager released!" by Jim

Tue, 18 Mar 2008 18:44:22 -0700

I see that Synthesis::AssetPackage.build_all has a dependency on the “get_file_revision” method which depends on SVN.

Has an update been released to make this Git compatible?

"AssetPackager released!" by David Vanderson

Fri, 14 Mar 2008 15:10:19 -0700

Thanks for the great plugin!

One issue that I ran into was the svn command dying when running under mongrel versions > 1.0.2 that are launched as root but drop their privileges. You get this error message in the mongrel log:

svn: Can't check path '/root/.subversion': Permission denied

$HOME is still /root, so svn tries to look for /root/.subversion, which throws an exception (due to permissions), which eventually returns a default revision of 1. In my case I worked around it by adding “—config-dir /does-not-exist” to the svn command.

Thanks again,
Dave

"AssetPackager released!" by Jyrki J. Lilja

Wed, 27 Feb 2008 12:51:00 -0800

This seems to fix the problem:


--- a/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb
+++ b/vendor/plugins/asset_packager/lib/synthesis/asset_package_helper.rb
@@ -21,7 +21,10 @@ module Synthesis
         AssetPackage.targets_from_sources("javascripts", sources) : 
         AssetPackage.sources_from_targets("javascripts", sources))

-      sources.collect {|source| javascript_include_tag(source, options) }.join("\n")
+      sources.collect { |source|
+        source = javascript_path(source)
+        content_tag("script", nil, { "type" => "text/javascript", "src" => source }.merge(options))
+      }.join("\n")
     end

     def stylesheet_link_merged(*sources)

"AssetPackager released!" by Jyrki J. Lilja

Tue, 26 Feb 2008 05:00:32 -0800

Hello! Thanks for great plugin =) I ran into a problem with javascript_include_merged, which, for some odd reason adds the querystring timestamp to src. Is there any solution, because my JS is over 100 kB and is hard for bandwitch because it’s transferred every time I load the page.

Thanks in advance, Jyrki J.

"AssetPackager released!" by Alf Mikula

Wed, 13 Feb 2008 18:01:31 -0800

I ran into a problem with adding a single javascript file to the asset_packages.yml file. Since the svn version number on the new file was older than the previous version, asset_packager didn’t build a new file, and likewise, browsers wouldn’t know that the file had changed because the version number didn’t change. I think since any change to asset_packages.yml could affect any of the packages, the version number of the asset_packages.yml needs to be included in the calculation. Here’s my patch:

http://pastie.caboo.se/151920