-
Notifications
You must be signed in to change notification settings - Fork 91
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
Preserve sub-directory structure when only minifying #29
Comments
This reverts commit f74b642.
I seriously need this for my Stripes project in the Java 1.6 compatible builds. Vote +1. Find me on the Stripes IRC channel. |
@AFaust Can you please test the branch |
@timothystone I've merge this fix into the branch v1.6.X. Please update the plugin version in your POM file to 1.6.1. |
@samaxes thanks! I'll test it today! |
Merged into master. |
Hi, I want to use this preserve sub directory feature, but could not find the example how to enable it. Can you please let me know how to use this feature in Release 1.6.1? What attribute do I need to include in my pom.xml inside this plugin tag? |
I downloaded the demo example from the Release 1.6.1 https://github.com/samaxes/minify-maven-plugin/releases/download/minify-maven-plugin-1.6.1/minify-maven-plugin-demo-1.6.1-src.zip When I ran the command mvn clean install, it merges all js (from subfolders also) into single one. I don't want to merge and maintain the original folder structure. |
<nomunge>true</nomunge>
<skipMerge>true</skipMerge> This is what I'm using and it works. Admittedly I'm using a custom spin of this plugin, but it's based on 1.6.1 and only adds a feature from 1.7 (nosuffix). |
Thanks Timothystone...it works !! :) |
BTW, if you want "my custom spin" which ports the "nosuffix" feature from 1.7 to 1.6, pick up my pull request: #39 |
Hi guys,
Cheers |
As stated in an after-fix comment on #17 a minification without merging (skipMerge = true) currently removes any subdirectory structure captured by jsSourceIncludes / cssSourceIncludes which can be very problematic for projects relying on that sub-directory structure.
From the after-fix comment:
The text was updated successfully, but these errors were encountered: