I'll take a look. Thanks
Jon On 19 Jan 2018 10:47, "Romain Manni-Bucau" <[email protected]> wrote: > Hi guys, > > Recent merges broke the build / tests it seems. > I also saw some weird code like > https://github.com/apache/tomee/blob/master/container/ > openejb-core/src/main/java/org/apache/openejb/assembler/ > classic/Assembler.java#L2562 > which is doing twice the "same thing" so sounds like we have 2 ways to > deploy an app container now > (org.apache.openejb.config.AppInfoBuilder#buildAppResources was probably > intended to be dropped). > > Can someone with the knowledge at least check the build and if the issue is > too nasty revert the related code to ensure we are back to green on master > at least - if ambiguous, it is fine for me to be red if somebody works on > fixing it, it is not if we stay like that ;)? > > Thanks, > Romain Manni-Bucau > @rmannibucau <https://twitter.com/rmannibucau> | Blog > <https://rmannibucau.metawerx.net/> | Old Blog > <http://rmannibucau.wordpress.com> | Github <https://github.com/ > rmannibucau> | > LinkedIn <https://www.linkedin.com/in/rmannibucau> >
