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

Replace jarchivlib with commons compress #11

Merged
merged 4 commits into from
Nov 19, 2013
Merged

Replace jarchivlib with commons compress #11

merged 4 commits into from
Nov 19, 2013

Conversation

b-long
Copy link
Contributor

@b-long b-long commented Nov 17, 2013

Fixes issue #7 and passes tests. Looking for others to test / verify, if anyone is willing. This changeset does NOT includes the errors that existed in my earlier pull request (mentioned here: #9 (comment) ). Will add a comment with the result of of the Travis CI build.

@eirslett
Copy link
Owner

It's a green build on Windows and Linux! Thanks a lot! I'll merge this in now.
Just have to let Travis run it on openjdk6, if it's still green I'll release it.

eirslett added a commit that referenced this pull request Nov 19, 2013
…mpress

Replace jarchivlib with commons compress
@eirslett eirslett merged commit 0fc6d9f into eirslett:master Nov 19, 2013
@b-long
Copy link
Contributor Author

b-long commented Nov 19, 2013

Awesome, thanks Eirik! Really glad to be able to help get this change in :) Do you think you'll consider releasing a new version at some point, since now frontend-maven-plugin is compatible with Java 1.6 ?

@eirslett
Copy link
Owner

Version 0.0.7 is released now, at the Sonatype OSS repository.
They'll automatically sync it to Maven Central in a couple of hours.
Note that this release also includes #6 ;-)

@b-long
Copy link
Contributor Author

b-long commented Nov 19, 2013

Fantastic, I'm looking forward to using it!

@eirslett
Copy link
Owner

It's out on Maven Central now.

@b-long
Copy link
Contributor Author

b-long commented Nov 20, 2013

Great, I'll check it out soon :)

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

Successfully merging this pull request may close these issues.

None yet

2 participants