Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version of yuicompressor is hardcoded into Java source #24

Closed
GoogleCodeExporter opened this issue Aug 20, 2015 · 2 comments
Closed

Version of yuicompressor is hardcoded into Java source #24

GoogleCodeExporter opened this issue Aug 20, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

>> What steps will reproduce the problem?

1. Use htmlcompressor-0.9.7.jar with yuicompressor-2.4.4.jar
2. Set --compress-js or compress-css
3. Run

>> What is the expected output? What do you see instead?

No errors should be reported and Javascript and CSS within the HTML files 
should also be compressed.

Instead, I see an error relating to how the yuicompressor-2.4.2.jar must be in 
the same directory in order for htmlcompressor to find it, regardless of 
version. Also, no HTML was compressed at all, while it might be expected that 
html would be compressed while the --compress-js / css options would simply be 
ignored and HTML compression would continue.

>> Please provide any additional information below.

YUI Compressor 2.4.2 had a serious bug which munged CSS media queries 
(http://yuilibrary.com/projects/yuicompressor/ticket/2527991) and was fixed in 
2.4.4 which is still not up on the Public YUI download page. 

The overview of this project mentions that the filename of yuicompressor cannot 
be configured.

Perhaps scanning the filesystem only in the same directory as htmlcompressor 
and looking for any version of yuicompressor would work better. To workaround, 
I've simply renamed the newer yuicompressor (probably should patch my local src 
with the newer yuicompressor filename).

-----

Thanks, Serg; this compressor is really great!

Original issue reported on code.google.com by mga...@iplantcollaborative.org on 29 Dec 2010 at 1:28

@GoogleCodeExporter
Copy link
Author

Class path must be hardcoded and there is no easy workaround as far as I know. 
I added support for yuicompressor-2.4.4.jar in 0.9.8 release by hardcoding it 
as well (so now it searches for both versions).

Original comment by serg472@gmail.com on 30 Dec 2010 at 6:15

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Sweet; thanks much!

Original comment by mga...@iplantcollaborative.org on 3 Jan 2011 at 4:32

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

No branches or pull requests

1 participant