You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sirona.apache.org by Romain Manni-Bucau <rm...@gmail.com> on 2015/06/24 17:31:53 UTC
Re: splitting the webapp?
Hi guys,
not sure what I missed but seems we still have a mismatch EE/spring.
Comes from the address specified in spring (/sironaservices). I dont care
much about the value but it should go in @ApplicationPath IMO and be
removed from spring only: This way the app would work in both context.
Did i miss anything?
Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> | Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
<http://www.tomitribe.com>
2015-05-26 8:39 GMT+02:00 Olivier Lamy <ol...@apache.org>:
> no worries
>
> On 26 May 2015 at 14:39, Romain Manni-Bucau <rm...@gmail.com> wrote:
>
> > Ok then if we can comment it i would prefer rather than proposing twice
> the
> > "same" thing.
> > Le 25 mai 2015 21:21, "Olivier Lamy" <ol...@apache.org> a écrit :
> >
> > > perso I don't mind :-)
> > >
> > > On 26 May 2015 at 12:54, Romain Manni-Bucau <rm...@gmail.com>
> > wrote:
> > >
> > > > +1, do we keep angular module/webapp for this one?
> > > > Le 25 mai 2015 17:50, "Olivier Lamy" <ol...@apache.org> a écrit :
> > > >
> > > > > release?
> > > > >
> > > > > On 19 May 2015 at 21:15, Olivier Lamy <ol...@apache.org> wrote:
> > > > >
> > > > > >
> > > > > >
> > > > > > On 19 May 2015 at 18:30, Romain Manni-Bucau <
> rmannibucau@gmail.com
> > >
> > > > > wrote:
> > > > > >
> > > > > >> well today angular one doesnt "exist" in my refactoring. When
> > fully
> > > > > ready
> > > > > >> I
> > > > > >> expect it to replace the existing one (that's what I tried to
> say
> > > ;)).
> > > > > ATM
> > > > > >> angular impl is too hardcoded to replace old impl (plugin are
> > broken
> > > > > >> AFAIK).
> > > > > >>
> > > > > >
> > > > > > haha no need to be to much diplomatic!! if it doesn't work as
> > > expected
> > > > it
> > > > > > doesn't work that's it :-)
> > > > > > So no worries move forward!
> > > > > >
> > > > > >
> > > > > >
> > > > > >>
> > > > > >>
> > > > > >> Romain Manni-Bucau
> > > > > >> @rmannibucau <https://twitter.com/rmannibucau> | Blog
> > > > > >> <http://rmannibucau.wordpress.com> | Github <
> > > > > >> https://github.com/rmannibucau> |
> > > > > >> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
> > > > > >> <http://www.tomitribe.com>
> > > > > >>
> > > > > >> 2015-05-19 4:37 GMT+02:00 Olivier Lamy <ol...@apache.org>:
> > > > > >>
> > > > > >> > A bit weird to have 2 webapp: one with angularjs and an other
> > one
> > > > with
> > > > > >> .vm
> > > > > >> > ?
> > > > > >> >
> > > > > >> > On 15 May 2015 at 18:56, Romain Manni-Bucau <
> > > rmannibucau@gmail.com>
> > > > > >> wrote:
> > > > > >> >
> > > > > >> > > Did the work for reporting-webapp (added
> > reporting-webapp-full).
> > > > > >> > >
> > > > > >> > > @Olivier: guess the target is reporting-ui but when ready
> I'd
> > > like
> > > > > to
> > > > > >> > merge
> > > > > >> > > it back in reporting-webapp (to ensure we have a "single"
> > > webapp).
> > > > > We
> > > > > >> can
> > > > > >> > > either use META-INF/resources/<all resources> to reuse them
> as
> > > > today
> > > > > >> > > through -classes.jar or do a shade/overlay of resources, I
> > don't
> > > > > >> really
> > > > > >> > > care while dependencies are still correct for EE/servlet
> > > > packaging.
> > > > > >> > >
> > > > > >> > > @All: a review would be welcomed
> > > > > >> > >
> > > > > >> > >
> > > > > >> > >
> > > > > >> > > Romain Manni-Bucau
> > > > > >> > > @rmannibucau <https://twitter.com/rmannibucau> | Blog
> > > > > >> > > <http://rmannibucau.wordpress.com> | Github <
> > > > > >> > > https://github.com/rmannibucau> |
> > > > > >> > > LinkedIn <https://www.linkedin.com/in/rmannibucau> |
> > Tomitriber
> > > > > >> > > <http://www.tomitribe.com>
> > > > > >> > >
> > > > > >> > > 2015-05-15 4:48 GMT+02:00 Jean-Louis MONTEIRO <
> > > jeanouii@gmail.com
> > > > >:
> > > > > >> > >
> > > > > >> > > > Same here.
> > > > > >> > > > Please no crazy maven configuration for the war plug-in.
> > > > > >> > > >
> > > > > >> > > > Le jeu 14 mai 2015 16:27, Olivier Lamy <ol...@apache.org>
> a
> > > > > écrit :
> > > > > >> > > >
> > > > > >> > > > > Hi,
> > > > > >> > > > > Apologise for so late answer :-(
> > > > > >> > > > >
> > > > > >> > > > > I like solution 1. (no ugly usage of the war plugin)
> > > > > >> > > > >
> > > > > >> > > > >
> > > > > >> > > > > On 21 April 2015 at 20:09, Romain Manni-Bucau <
> > > > > >> rmannibucau@gmail.com
> > > > > >> > >
> > > > > >> > > > > wrote:
> > > > > >> > > > >
> > > > > >> > > > > > Up ;)
> > > > > >> > > > > >
> > > > > >> > > > > > Would like to sort out the webapp EE/Servlet issue
> > before
> > > > > >> > releasing.
> > > > > >> > > > > Issues
> > > > > >> > > > > > are:
> > > > > >> > > > > >
> > > > > >> > > > > > - for EE JAXRS will get deployed automatically so no
> > need
> > > to
> > > > > >> bring
> > > > > >> > > > > > spring/cxf/...
> > > > > >> > > > > > - for EE JAXRS endpoint will be deployed respecting
> > > > > Application
> > > > > >> > > > > > @ApplicationPath (so we need to provide one instead of
> > the
> > > > > >> mapping
> > > > > >> > in
> > > > > >> > > > > > webx.ml). Cool side effect: we are sure it will work
> > with
> > > > our
> > > > > >> > front.
> > > > > >> > > > > >
> > > > > >> > > > > >
> > > > > >> > > > > > Solution I see:
> > > > > >> > > > > >
> > > > > >> > > > > > 1. 2 artifacts
> > > > > >> > > > > > - do a "clean/light" EE war we deploy classes jar as
> > well
> > > > > >> > > > > > - do a servlet war just adding dependencies, web.xml
> and
> > > the
> > > > > >> > previous
> > > > > >> > > > > > -classes jar.
> > > > > >> > > > > >
> > > > > >> > > > > > 2. 1 artifact
> > > > > >> > > > > > - we can play with maven war plugin to create 2 wars
> > (with
> > > > > >> > > > > exclude/include)
> > > > > >> > > > > > from the same artifact and deploy both but it sounds
> > like
> > > a
> > > > > >> pain to
> > > > > >> > > > > > maintain to me
> > > > > >> > > > > >
> > > > > >> > > > > > wdyt? Any other idea?
> > > > > >> > > > > >
> > > > > >> > > > > >
> > > > > >> > > > > >
> > > > > >> > > > > > Romain Manni-Bucau
> > > > > >> > > > > > @rmannibucau <https://twitter.com/rmannibucau> |
> Blog
> > > > > >> > > > > > <http://rmannibucau.wordpress.com> | Github <
> > > > > >> > > > > > https://github.com/rmannibucau> |
> > > > > >> > > > > > LinkedIn <https://www.linkedin.com/in/rmannibucau> |
> > > > > Tomitriber
> > > > > >> > > > > > <http://www.tomitribe.com>
> > > > > >> > > > > >
> > > > > >> > > > > > 2015-03-19 7:58 GMT+01:00 Romain Manni-Bucau <
> > > > > >> > rmannibucau@gmail.com
> > > > > >> > > >:
> > > > > >> > > > > >
> > > > > >> > > > > > > Well issue was more on web.xml, appctx and libs.
> > > > > >> > > > > > >
> > > > > >> > > > > > > I cant look further before devoxxfr is gone but I d
> > love
> > > > to
> > > > > >> fix
> > > > > >> > it
> > > > > >> > > > and
> > > > > >> > > > > > > release.
> > > > > >> > > > > > > Le 19 mars 2015 05:23, "Olivier Lamy" <
> > olamy@apache.org
> > > >
> > > > a
> > > > > >> > écrit :
> > > > > >> > > > > > >
> > > > > >> > > > > > >> Hey
> > > > > >> > > > > > >> Yup my really bad as I didn't work on that.
> > > > > >> > > > > > >> And not a lot of time ATM :-(
> > > > > >> > > > > > >> AFAIK The code is already splitted.
> > > > > >> > > > > > >>
> > > > > >> > > > > > >>
> > > > > >> > > > > > >> On 19 March 2015 at 09:28, Romain Manni-Bucau <
> > > > > >> > > > rmannibucau@gmail.com>
> > > > > >> > > > > > >> wrote:
> > > > > >> > > > > > >>
> > > > > >> > > > > > >> > Hi guys
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> > seems webapp is still not portable in EE
> > containers,
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> > do we plan anything on this topic? ie a webapp-ee
> > and
> > > > > >> webapp
> > > > > >> > > > maybe?
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> > Romain Manni-Bucau
> > > > > >> > > > > > >> > @rmannibucau <https://twitter.com/rmannibucau> |
> > > Blog
> > > > > >> > > > > > >> > <http://rmannibucau.wordpress.com> | Github <
> > > > > >> > > > > > >> > https://github.com/rmannibucau> |
> > > > > >> > > > > > >> > LinkedIn <
> https://www.linkedin.com/in/rmannibucau>
> > |
> > > > > >> > Tomitriber
> > > > > >> > > > > > >> > <http://www.tomitribe.com>
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> > 2014-10-29 2:36 GMT+01:00 Olivier Lamy <
> > > > olamy@apache.org
> > > > > >:
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >> > > Sure.
> > > > > >> > > > > > >> > > Yeah I hope to find time before xmas!
> > > > > >> > > > > > >> > >
> > > > > >> > > > > > >> > > On 28 October 2014 18:13, Romain Manni-Bucau <
> > > > > >> > > > > rmannibucau@gmail.com
> > > > > >> > > > > > >
> > > > > >> > > > > > >> > > wrote:
> > > > > >> > > > > > >> > > > there is no hurry, we could maybe try a
> release
> > > for
> > > > > >> Xmas
> > > > > >> > > with
> > > > > >> > > > > > "just"
> > > > > >> > > > > > >> > > > angularjs work? wdyt?
> > > > > >> > > > > > >> > > >
> > > > > >> > > > > > >> > > >
> > > > > >> > > > > > >> > > > Romain Manni-Bucau
> > > > > >> > > > > > >> > > > @rmannibucau
> > > > > >> > > > > > >> > > > http://www.tomitribe.com
> > > > > >> > > > > > >> > > > http://rmannibucau.wordpress.com
> > > > > >> > > > > > >> > > > https://github.com/rmannibucau
> > > > > >> > > > > > >> > > >
> > > > > >> > > > > > >> > > >
> > > > > >> > > > > > >> > > > 2014-10-28 3:40 GMT+01:00 Olivier Lamy <
> > > > > >> olamy@apache.org
> > > > > >> > >:
> > > > > >> > > > > > >> > > >> Hi,
> > > > > >> > > > > > >> > > >> Ok I reckon for some ( good? summer coming
> > here,
> > > > > >> beach,
> > > > > >> > bbq
> > > > > >> > > > > with
> > > > > >> > > > > > >> beers
> > > > > >> > > > > > >> > > >> etc... :-) ) reasons, I didn't have time yet
> > to
> > > > work
> > > > > >> on
> > > > > >> > > that
> > > > > >> > > > > :-(.
> > > > > >> > > > > > >> > > >> I will try soon
> > > > > >> > > > > > >> > > >>
> > > > > >> > > > > > >> > > >> On 18 October 2014 18:17, Romain
> Manni-Bucau <
> > > > > >> > > > > > >> rmannibucau@gmail.com>
> > > > > >> > > > > > >> > > wrote:
> > > > > >> > > > > > >> > > >>> Le 18 oct. 2014 00:06, "Olivier Lamy" <
> > > > > >> olamy@apache.org
> > > > > >> > >
> > > > > >> > > a
> > > > > >> > > > > > écrit
> > > > > >> > > > > > >> :
> > > > > >> > > > > > >> > > >>>>
> > > > > >> > > > > > >> > > >>>> Uhm looks wrong.
> > > > > >> > > > > > >> > > >>>> It looks I will have to use tomee plugin
> :-)
> > > > > >> > > > > > >> > > >>>
> > > > > >> > > > > > >> > > >>> Or enhance tomcat one ;)
> > > > > >> > > > > > >> > > >>>
> > > > > >> > > > > > >> > > >>>> Not sure to have time to fix it during the
> > > > > weekend.
> > > > > >> > > > > Otherwise I
> > > > > >> > > > > > >> will
> > > > > >> > > > > > >> > > have
> > > > > >> > > > > > >> > > >>> a
> > > > > >> > > > > > >> > > >>>> look on Monday.
> > > > > >> > > > > > >> > > >>>>
> > > > > >> > > > > > >> > > >>>> Cheers
> > > > > >> > > > > > >> > > >>>> --
> > > > > >> > > > > > >> > > >>>> Olivier
> > > > > >> > > > > > >> > > >>>> On 17 Oct 2014 17:28, "Romain
> Manni-Bucau" <
> > > > > >> > > > > > >> rmannibucau@gmail.com>
> > > > > >> > > > > > >> > > wrote:
> > > > > >> > > > > > >> > > >>>>
> > > > > >> > > > > > >> > > >>>> > but a
> > > > > >> src/main/webapp/WEB-INF/applicationContext.xml?
> > > > > >> > > > > > >> > > >>>> >
> > > > > >> > > > > > >> > > >>>> >
> > > > > >> > > > > > >> > > >>>> > Romain Manni-Bucau
> > > > > >> > > > > > >> > > >>>> > @rmannibucau
> > > > > >> > > > > > >> > > >>>> > http://www.tomitribe.com
> > > > > >> > > > > > >> > > >>>> > http://rmannibucau.wordpress.com
> > > > > >> > > > > > >> > > >>>> > https://github.com/rmannibucau
> > > > > >> > > > > > >> > > >>>> >
> > > > > >> > > > > > >> > > >>>> >
> > > > > >> > > > > > >> > > >>>> > 2014-10-17 1:33 GMT+02:00 Olivier Lamy <
> > > > > >> > > olamy@apache.org
> > > > > >> > > > >:
> > > > > >> > > > > > >> > > >>>> > > Hi,
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > > AFAIK sirona-reporting-ui.war do not
> > > include
> > > > > >> > > cfx/spring
> > > > > >> > > > > > (just
> > > > > >> > > > > > >> > for
> > > > > >> > > > > > >> > > >>>> > > testing purpose when using the great
> mvn
> > > > > >> > tomcat7:run
> > > > > >> > > > :P )
> > > > > >> > > > > > >> > > >>>> > > Yup I missed to change the webapp part
> > > which
> > > > > >> should
> > > > > >> > > > > contain
> > > > > >> > > > > > >> the
> > > > > >> > > > > > >> > > ui.war
> > > > > >> > > > > > >> > > >>>> > > + cxf/spring to run the jaxrs part
> (and
> > > > delete
> > > > > >> old
> > > > > >> > ui
> > > > > >> > > > > impl)
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > > The reporting api doesn't have any
> > > > > dependencies
> > > > > >> on
> > > > > >> > > > > > >> cxf/spring.
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > > On 17 October 2014 08:05, Romain
> > > > Manni-Bucau <
> > > > > >> > > > > > >> > > rmannibucau@gmail.com>
> > > > > >> > > > > > >> > > >>>> > wrote:
> > > > > >> > > > > > >> > > >>>> > >> Hi guys,
> > > > > >> > > > > > >> > > >>>> > >>
> > > > > >> > > > > > >> > > >>>> > >> missed it but now the sirona webapp
> > > > > (reporting)
> > > > > >> > > depend
> > > > > >> > > > > on
> > > > > >> > > > > > >> cxf
> > > > > >> > > > > > >> > and
> > > > > >> > > > > > >> > > >>>> > >> spring, I was used to use it in a EE
> > > > > container
> > > > > >> > > (tomee
> > > > > >> > > > > and
> > > > > >> > > > > > >> resin
> > > > > >> > > > > > >> > > until
> > > > > >> > > > > > >> > > >>>> > >> now) and I clearly don't want to
> depend
> > > in
> > > > > cxf
> > > > > >> in
> > > > > >> > > the
> > > > > >> > > > > app
> > > > > >> > > > > > >> and
> > > > > >> > > > > > >> > > worse
> > > > > >> > > > > > >> > > >>> on
> > > > > >> > > > > > >> > > >>>> > >> spring.
> > > > > >> > > > > > >> > > >>>> > >>
> > > > > >> > > > > > >> > > >>>> > >> Can we get right of it? About spring:
> > > > that's
> > > > > >> easy
> > > > > >> > we
> > > > > >> > > > > don't
> > > > > >> > > > > > >> need
> > > > > >> > > > > > >> > > it at
> > > > > >> > > > > > >> > > >>>> > >> all, all can be done in web.xml or
> in a
> > > > > >> > > > > > >> > > ServletContainerInitializer.
> > > > > >> > > > > > >> > > >>>> > >> About CXF: the point is how to make
> the
> > > > JAXRS
> > > > > >> lib
> > > > > >> > > > > > optional.
> > > > > >> > > > > > >> > Here
> > > > > >> > > > > > >> > > we
> > > > > >> > > > > > >> > > >>>> > >> can maybe do 2 webapps, one with just
> > the
> > > > > >> business
> > > > > >> > > > code
> > > > > >> > > > > > for
> > > > > >> > > > > > >> EE
> > > > > >> > > > > > >> > > >>>> > >> containers and one with CXF for
> > > > > tomcat/servlet
> > > > > >> > > > > containers.
> > > > > >> > > > > > >> > > >>>> > >>
> > > > > >> > > > > > >> > > >>>> > >> wdyt?
> > > > > >> > > > > > >> > > >>>> > >>
> > > > > >> > > > > > >> > > >>>> > >>
> > > > > >> > > > > > >> > > >>>> > >> Romain Manni-Bucau
> > > > > >> > > > > > >> > > >>>> > >> @rmannibucau
> > > > > >> > > > > > >> > > >>>> > >> http://www.tomitribe.com
> > > > > >> > > > > > >> > > >>>> > >> http://rmannibucau.wordpress.com
> > > > > >> > > > > > >> > > >>>> > >> https://github.com/rmannibucau
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > >
> > > > > >> > > > > > >> > > >>>> > > --
> > > > > >> > > > > > >> > > >>>> > > Olivier Lamy
> > > > > >> > > > > > >> > > >>>> > > http://twitter.com/olamy |
> > > > > >> > > > http://linkedin.com/in/olamy
> > > > > >> > > > > > >> > > >>>> >
> > > > > >> > > > > > >> > > >>
> > > > > >> > > > > > >> > > >>
> > > > > >> > > > > > >> > > >>
> > > > > >> > > > > > >> > > >> --
> > > > > >> > > > > > >> > > >> Olivier Lamy
> > > > > >> > > > > > >> > > >> http://twitter.com/olamy |
> > > > > >> http://linkedin.com/in/olamy
> > > > > >> > > > > > >> > >
> > > > > >> > > > > > >> > >
> > > > > >> > > > > > >> > >
> > > > > >> > > > > > >> > > --
> > > > > >> > > > > > >> > > Olivier Lamy
> > > > > >> > > > > > >> > > http://twitter.com/olamy |
> > > > > http://linkedin.com/in/olamy
> > > > > >> > > > > > >> > >
> > > > > >> > > > > > >> >
> > > > > >> > > > > > >>
> > > > > >> > > > > > >>
> > > > > >> > > > > > >>
> > > > > >> > > > > > >> --
> > > > > >> > > > > > >> Olivier Lamy
> > > > > >> > > > > > >> http://twitter.com/olamy |
> > > http://linkedin.com/in/olamy
> > > > > >> > > > > > >>
> > > > > >> > > > > > >
> > > > > >> > > > > >
> > > > > >> > > > >
> > > > > >> > > > >
> > > > > >> > > > >
> > > > > >> > > > > --
> > > > > >> > > > > Olivier Lamy
> > > > > >> > > > > http://twitter.com/olamy | http://linkedin.com/in/olamy
> > > > > >> > > > >
> > > > > >> > > >
> > > > > >> > >
> > > > > >> >
> > > > > >> >
> > > > > >> >
> > > > > >> > --
> > > > > >> > Olivier Lamy
> > > > > >> > http://twitter.com/olamy | http://linkedin.com/in/olamy
> > > > > >> >
> > > > > >>
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Olivier Lamy
> > > > > > http://twitter.com/olamy | http://linkedin.com/in/olamy
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Olivier Lamy
> > > > > http://twitter.com/olamy | http://linkedin.com/in/olamy
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Olivier Lamy
> > > http://twitter.com/olamy | http://linkedin.com/in/olamy
> > >
> >
>
>
>
> --
> Olivier Lamy
> http://twitter.com/olamy | http://linkedin.com/in/olamy
>