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

zookeeper protocol is not included at owner-extras jar #114

Closed
ksaritek opened this issue Dec 26, 2014 · 7 comments
Closed

zookeeper protocol is not included at owner-extras jar #114

ksaritek opened this issue Dec 26, 2014 · 7 comments

Comments

@ksaritek
Copy link
Contributor

@ksaritek ksaritek commented Dec 26, 2014

protocol classes are not included at assembled jar. Can it be because of
org.apache.felix-maven-bundle-plugin-1.4.0
org.aeonbits.owner.*
configuration at root pom file

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Jan 6, 2015

👍

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Feb 2, 2015

@ksaritek can you please verify that last commit on master branch fixes this? I had to change the OSGi plugin to have the manifest compatible with OSGi without changing the way jars are built.
In case the problem is still there, please reopen this bug.

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Feb 2, 2015

Sorry for the long delay, very little time to work on owner lately.

I will publish a new release asap, in order to have some good jars on maven central.

@ksaritek
Copy link
Contributor Author

@ksaritek ksaritek commented Mar 30, 2015

when will new release published?

@lviggiano
Copy link
Owner

@lviggiano lviggiano commented Mar 30, 2015

Hi. Sorry for the long wait.

I released this evening. There was an issue with sonatype jar validation that prevented the release to succeed. I had to create a dummy class just to have a jar generated by maven to be deployed in the repository.

See commit 0d7d3fd

Releasing on sonatype is always fun.

Sorry for the delay. And thanks for the help.

Please confirm that everything is ok, if so ;)

Luigi

@ksaritek
Copy link
Contributor Author

@ksaritek ksaritek commented Apr 2, 2015

Yep, I will check :)

@ksaritek
Copy link
Contributor Author

@ksaritek ksaritek commented Apr 9, 2015

sorry for delay, seems ok 👍

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.