cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Romain Manni-Bucau <rmannibu...@gmail.com>
Subject Re: [cdi] regression for case without jaxrs resources?
Date Sun, 23 Oct 2016 14:34:16 GMT
2016-10-23 16:31 GMT+02:00 John D. Ament <johndament@apache.org>:

> On Sat, Oct 22, 2016 at 3:42 PM Romain Manni-Bucau <rmannibucau@gmail.com>
> wrote:
>
> > 2016-10-22 21:31 GMT+02:00 John D. Ament <johndament@apache.org>:
> >
> > > On Sat, Oct 22, 2016 at 3:10 PM Romain Manni-Bucau <
> > rmannibucau@gmail.com>
> > > wrote:
> > >
> > > > Hi guys,
> > > >
> > > > think now we have a default Application but without checking we have
> > > > anything to deploy. Means that the extension will fail CDI deployment
> > if
> > > > the module doesn't need JAX-RS which concretely makes the extension
> > > > unusable outside a end user application.
> > > >
> > > > Can it be fixed and deployment skipped if nothing should be deployed?
> > > >
> > >
> > > Yep, I'll raise a PR shortly (eg next day or so).  I still want to add
> > some
> > > systests for this case.  Do you have a JIRA for this or no?
> > >
> > >
> > No, wanted to check it was unintended before.
> >
>
> Well, to put it a different way.  The idea that you would include CXF but
> not deploy any JAX-RS resources hadn't come into my mind when working on
> it.  If its a real use case, can definitely patch it.
>
>
Anywhere where JAXRS is considered as part of the framework/runtime and not
the application it is so yes I'd do it. In particular cause it is not hard
to see we only have the default application bean and no resources in the
extension ;). Can help on that if needed or you have some time issues.


>
>
> >
> >
> > >
> > > >
> > > > Romain Manni-Bucau
> > > > @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> > > > <https://blog-rmannibucau.rhcloud.com> | Old Wordpress Blog
> > > > <http://rmannibucau.wordpress.com> | Github <
> > > > https://github.com/rmannibucau> |
> > > > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
> > > > <http://www.tomitribe.com> | JavaEE Factory
> > > > <https://javaeefactory-rmannibucau.rhcloud.com>
> > > >
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message