I was about to commit a fix, but let's proceed that way. First open a JIRA in https://issues.apache.org/jira/browse/TOMEE Attach the patch file to it.
Then, ping here again so we can commit it for you. Don't forget to give us your ID the first time at least so that we can affect it to you and mark it resolve by you. I'd be more than happy to commit it for you. Thanks JLouis -- Jean-Louis Monteiro http://twitter.com/jlouismonteiro http://www.tomitribe.com On Fri, May 23, 2014 at 5:01 AM, Chamil Jeewantha <[email protected]>wrote: > 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] >
