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.

Sign up for GitHub

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

Jump to bottom

CDI bean classes from dependencies with pregenerated jandex index are ignored #915

Closed
mkouba opened this issue Feb 15, 2019 · 4 comments
Closed

CDI bean classes from dependencies with pregenerated jandex index are ignored #915

mkouba opened this issue Feb 15, 2019 · 4 comments
Labels
kind/bug Something isn't working
Milestone
0.10.0

Comments

@mkouba
Copy link
Contributor

mkouba commented Feb 15, 2019

This is caused by a regression in Jandex: https://issues.jboss.org/projects/JANDEX/issues/JANDEX-49.
@mkouba mkouba added kind/bug Something isn't working first public release labels Feb 15, 2019
@gastaldi
Copy link
Contributor

I have provided a fix in smallrye/jandex#52

@gastaldi gastaldi mentioned this issue Feb 19, 2019
Closed
@gastaldi
Copy link
Contributor

I've tested with the Jandex 2.1.1.Beta1-SNAPSHOT build from the master branch and I can confirm that it works for me.

@n1hility is it possible ot release Jandex and update protean with this new version?

@n1hility
Copy link
Member

yes I plan to cut it today I just wanted to make sure there was consensus on the other DotName inner class issue before releasing

@n1hility n1hility added this to the 0.10.0 milestone Feb 22, 2019
@n1hility
Copy link
Member

2.1.1.Final is now released

@n1hility n1hility reopened this Feb 22, 2019
@mkouba mkouba closed this as completed Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working
Projects
None yet
Milestone
0.10.0
Development

No branches or pull requests
4 participants
@gastaldi @n1hility @emmanuelbernard @mkouba