you are welcome and all setup. So feel free to continue on any areas you want to.
-- Jean-Louis Monteiro http://twitter.com/jlouismonteiro http://www.tomitribe.com On Sat, May 24, 2014 at 5:32 AM, Chamil Jeewantha <[email protected]>wrote: > Thanks for your help and attention! > > > On Fri, May 23, 2014 at 9:03 PM, Jean-Louis Monteiro < > [email protected]> wrote: > > > All done and committed. > > Thanks Chamil. > > > > JLouis > > > > -- > > Jean-Louis Monteiro > > http://twitter.com/jlouismonteiro > > http://www.tomitribe.com > > > > > > On Fri, May 23, 2014 at 5:24 PM, Chamil Jeewantha <[email protected] > > >wrote: > > > > > https://issues.apache.org/jira/browse/TOMEE-1221 > > > > > > above issue is created on JIRA and attached the patch. Please Review. > > > > > > My JIRA ID is kdchamil. > > > > > > > > > On Fri, May 23, 2014 at 5:47 PM, Chamil Jeewantha <[email protected] > > > >wrote: > > > > > > > Thanks, > > > > > > > > I will go ahead. > > > > > > > > > > > > On Fri, May 23, 2014 at 5:35 PM, Romain Manni-Bucau < > > > [email protected] > > > > > wrote: > > > > > > > >> Well basically if you want to tackle it do the following steps: > > > >> > > > >> 1) checkout tomee (svn.apache.org/repos/asf/tomee/tomee/trunk/) > > > >> 2) remove this block > > > >> 3) svn diff > mypatch.patch > > > >> 4) attach mypatch.patch to a jira on > > > >> issues.apache.org/jira/browse/TOMEEgiving rights to Apache > > > >> > > > >> We'll apply it > > > >> > > > >> > > > >> > > > >> > > > >> Romain Manni-Bucau > > > >> Twitter: @rmannibucau > > > >> Blog: http://rmannibucau.wordpress.com/ > > > >> LinkedIn: http://fr.linkedin.com/in/rmannibucau > > > >> Github: https://github.com/rmannibucau > > > >> > > > >> > > > >> 2014-05-23 14:01 GMT+02:00 Chamil Jeewantha <[email protected]>: > > > >> > > > >> > Thanks for your quick reply, > > > >> > > > > >> > I think it would be a good point for a newcomer to start > > contribution. > > > >> If > > > >> > someone can guide, I can fix that. > > > >> > > > > >> > Regards, > > > >> > Chamil > > > >> > > > > >> > > > > >> > On Fri, May 23, 2014 at 5:26 PM, Romain Manni-Bucau > > > >> > <[email protected]>wrote: > > > >> > > > > >> > > Hi > > > >> > > > > > >> > > good catch. You are right...but this block should actually be > > > removed > > > >> ;) > > > >> > > > > > >> > > > > > >> > > > > > >> > > Romain Manni-Bucau > > > >> > > Twitter: @rmannibucau > > > >> > > Blog: http://rmannibucau.wordpress.com/ > > > >> > > LinkedIn: http://fr.linkedin.com/in/rmannibucau > > > >> > > Github: https://github.com/rmannibucau > > > >> > > > > > >> > > > > > >> > > 2014-05-23 13:52 GMT+02:00 Chamil Jeewantha <[email protected] > >: > > > >> > > > > > >> > > > I found the following code segment on init method of > > > >> > > > TomcatWebAppBuilder.java > > > >> > > > > > > >> > > > if (standardContext.getConfigFile() == null) { > > > >> > > > final String s = File.pathSeparator; > > > >> > > > final File contextXmlFile = new > > > >> > > > File(standardContext.getDocBase() + s + "META-INF" + s + > > > >> > "context.xml"); > > > >> > > > if (contextXmlFile.exists()) { > > > >> > > > > > > BackportUtil.getAPI().setConfigFile(standardContext, > > > >> > > > contextXmlFile); > > > >> > > > standardContext.setOverride(true); > > > >> > > > } > > > >> > > > } > > > >> > > > > > > >> > > > I wonder why the File.pathSeparator is used instead of > > > >> File.separator. > > > >> > Is > > > >> > > > this purposefully done? If so what is the reason? > > > >> > > > > > > >> > > > > > > >> > > > Thanks in Advance! Sorry if this is a mad question. > > > >> > > > -- > > > >> > > > [email protected] > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > >> > > > > >> > -- > > > >> > [email protected] > > > >> > > > > >> > > > > > > > > > > > > > > > > -- > > > > [email protected] > > > > > > > > > > > > > > > > -- > > > [email protected] > > > > > > > > > -- > [email protected] >
