eandrejko / asset_packager Goto Github PK

Line

system("java -jar #{yui_path}/yuicompressor-2.4.2.jar --type js -v #{full_asset_path(s)} >/dev/null")

can be changed

system("java -jar #{yui_path}/yuicompressor-2.4.2.jar --type js -v #{full_asset_path(s)}")

to avoid problem on Windows OS.

Hi,

I was just trying out this plug-in as a replacement for the built in logic in v2.3.5 of Rails but I had some issues getting it running.

asset_packager.rb, Line 184: uninitialized class variable @@asset_host in Synthesis::AssetPackage

After doing a search I noticed that this variable was only populated in the build_all method, which is part of a rake task.

As my site was trying to create the initial files on the fly all I did was change the code to the following to get it running:

# class variables
@@asset_host = Rails.configuration.action_controller.asset_host

....

def build_all
   @@asset_packages_yml.keys.each do |asset_type|
      @@asset_packages_yml[asset_type].each { |p| self.new(asset_type, p).build }
    end
end

I didn't have to make a change to the replace_asset_urls method.

EDIT: I just noticed this breaks the rake task, so I have now removed the line I added above and left it in build_all and added it to the replace_asset_urls method.

Regards,
Nick