Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
if you call get() can be yes. Happy to help if you can sort out a test like the one i sent. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | Linke

Re: Meecrowave java 9 support

2017-12-18 Thread Romain Manni-Bucau
2017-12-18 15:19 GMT+01:00 John D. Ament : > If you explicitly add the dependencies that Ravi's mentioning, you'll get > the correct output without --add-modules. However, it is a hard dependency > for CXF to include these dependencies. Some of their core feature work is > based on JAX-WS featur

Re: Meecrowave java 9 support

2017-12-18 Thread John D. Ament
If you explicitly add the dependencies that Ravi's mentioning, you'll get the correct output without --add-modules. However, it is a hard dependency for CXF to include these dependencies. Some of their core feature work is based on JAX-WS features, so there's no way to avoid the dependency. On M

Re: Injecting Array types as fields

2017-12-18 Thread John D. Ament
We register a single bean per injection point. That bean handles the look up but only returns the type of the injection point. But its still a bit odd. I added CDI.current().select(int[].class, new ConfigProperty...) to my test to see what happens. In Weld, I see it call the producer but fail (

Re: Meecrowave java 9 support

2017-12-18 Thread Romain Manni-Bucau
Hi I hope to get rid of the dependency completely since we shouldn't need it at all but for now the way to have it working is to use --add-modules. Feel free to do a PR to remove it completely if you want, it would be very welcomed. side note: we also need to upgrade OWB when released to support

Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
Isn't it cause we reuse a generic producer? Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn 2

Re: Injecting Array types as fields

2017-12-18 Thread John D. Ament
That's good that this works. However, I wonder if my issue is specific to 3rd Party Bean creation? Same problem seems to be on both OWB and Weld, which makes me think it's an impl issue, but I can't think of anything else required on the impl side. Bean.getTypes() correctly returns String[].class

Meecrowave java 9 support

2017-12-18 Thread Ravisankar Challa
Meecrowave not working with java 9 (using latest snapshot of openwebbeans) Caused by: org.apache.webbeans.exception.WebBeansException: java.lang.NoClassDefFoundError: javax/xml/bind/JAXBException at org.apache.webbeans.event.ObserverMethodImpl.notify(ObserverMethodImpl.java:371) ~[openwebbeans

Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
Works for me: public class ArrayProducerTest extends AbstractUnitTest { @Test public void test() { startContainer(StringProducer.class); final String[] produced1 = getInstance(String[].class); final StringProducer[] produced2 = getInstance(StringProducer[].cla

Re: Injecting Array types as fields

2017-12-18 Thread Mark Struberg
I think it should work. Could you please create a ticket? txs and LieGrue, strub > Am 18.12.2017 um 03:47 schrieb John D. Ament : > > MP Config is adding arrays as a valid type for config properties. In doing > so, without making any changes to Geronimo Config I get this failure (not > what I