Oops sorry Basically doing 1 shouldnt change the behavior but would solve a bunch of common issues so id like to get it as default.
If im wrong please explain me, i maybe missed sthg Le 14 mai 2013 04:26, "David Blevins" <[email protected]> a écrit : > I don't think I understand this sentence. Can you rephrase? > > -David > > On May 13, 2013, at 2:19 PM, Romain Manni-Bucau <[email protected]> > wrote: > > > Basically proove me there is any regression introduce by it...noone so no > > change on the behavior...it just pass when it was not starting > > Le 13 mai 2013 23:14, "Jean-Louis MONTEIRO" <[email protected]> a > écrit : > > > >> Let's see what will be the result. It's interesting. > >> > >> But once again, -1 to change the default behavior. > >> > >> Jean-Louis > >> > >> > >> 2013/5/13 Romain Manni-Bucau <[email protected]> > >> > >>> well IMO we should go for 1 with a config to fallback to what we have > now > >>> to get a nice default > >>> > >>> i'll try to commit it to see how TCKs behave with such a config, then > >> we'll > >>> see i think > >>> > >>> *Romain Manni-Bucau* > >>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > >>> *Blog: **http://rmannibucau.wordpress.com/*< > >>> http://rmannibucau.wordpress.com/> > >>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > >>> *Github: https://github.com/rmannibucau* > >>> > >>> > >>> > >>> 2013/5/13 David Blevins <[email protected]> > >>> > >>>> In a similar boat. I think #1 is a feature we need regardless, though > >> I > >>>> agree with JL that the default should be as it is now. > >>>> > >>>> I have a small hope that we could find a way to make managing shaded > >>>> dependencies a bit easier. If we could find a cleaner way to do it > >> than > >>> we > >>>> do now, might be more realistic to try. It's a bit of a project > >> though. > >>>> > >>>> > >>>> -David > >>>> > >>>> On May 13, 2013, at 1:31 PM, Romain Manni-Bucau < > [email protected] > >>> > >>>> wrote: > >>>> > >>>>> i'm playing with 1 ATM, 2 would be the best but we'd need to maintain > >>>>> it...impossible today > >>>>> > >>>>> a shade will not work without custom clever transformer (never a good > >>>>> solution) > >>>>> > >>>>> so basically i'm for 1/ too if my tests show it works fine enough > >>>>> > >>>>> *Romain Manni-Bucau* > >>>>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > >>>>> *Blog: **http://rmannibucau.wordpress.com/*< > >>>> http://rmannibucau.wordpress.com/> > >>>>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > >>>>> *Github: https://github.com/rmannibucau* > >>>>> > >>>>> > >>>>> > >>>>> 2013/5/13 Jean-Louis MONTEIRO <[email protected]> > >>>>> > >>>>>> 1/ just adding a property for an app with values like parent-first > >>>> (default > >>>>>> - today) and parent-last (new value). > >>>>>> 2/ that usually the way other apps servers are used to do (IBM > >>>> Websphere AS > >>>>>> IIR) > >>>>>> 3/Don't know > >>>>>> 4/No > >>>>>> > >>>>>> The first one looks the more simple to implement (at first glance) > >> and > >>>> the > >>>>>> one which produce less impacts and possible side effects for users. > >>>>>> The second is something which can be studied furthermore and > >> implement > >>>> as a > >>>>>> cherry on the cake. > >>>>>> > >>>>>> Then, if I had to choose, I would choose 1/ > >>>>>> > >>>>>> Jean-Louis > >>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> 2013/5/13 Romain Manni-Bucau <[email protected]> > >>>>>> > >>>>>>> Hi > >>>>>>> > >>>>>>> as you probably know we changed quite often the classloading these > >>> last > >>>>>>> versions. > >>>>>>> > >>>>>>> We still have regularly issues with tomee provided libs (mainly > >>> because > >>>>>> of > >>>>>>> cxf/amq/... transitive deps) > >>>>>>> > >>>>>>> I would like to go further and use an URLClassLoaderFirst for lib > >>> part > >>>> of > >>>>>>> ear to allow apps to not rely on tomee libs first. > >>>>>>> > >>>>>>> The alternative is to shade/repackage all deps. > >>>>>>> > >>>>>>> PS: i want to avoid OSGi > >>>>>>> > >>>>>>> So basically here is the question: to solve conflict lib issue what > >>>> your > >>>>>>> preferred solution: > >>>>>>> 1) URLClassLoaderFirst even for ear lib part - if this one please > >> say > >>>> if > >>>>>>> you would like a property to switch back to old behavior > >>>>>>> (openejb.ear.lib.classloader.first=false?) > >>>>>>> 2) repackage all deps > >>>>>>> 3) other > >>>>>>> 4) OSGi/OSGi like > >>>>>>> ? > >>>>>>> > >>>>>>> If nothing moves in the following days i'll go for 1 (then we can > >>> still > >>>>>>> discuss ;). > >>>>>>> > >>>>>>> *Romain Manni-Bucau* > >>>>>>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > >>>>>>> *Blog: **http://rmannibucau.wordpress.com/*< > >>>>>>> http://rmannibucau.wordpress.com/> > >>>>>>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > >>>>>>> *Github: https://github.com/rmannibucau* > >>>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> -- > >>>>>> Jean-Louis > >>>>>> > >>>> > >>>> > >>> > >> > >> > >> > >> -- > >> Jean-Louis > >> > >
