[
https://issues.apache.org/jira/browse/OFBIZ-1712?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12586750#action_12586750
]
Ray Barlow commented on OFBIZ-1712:
-----------------------------------
I did think about the providers deployment choice as to whether they would run
with svn folders embeded or removed and decided it was really a preference that
the project doesn't need to worry about controlling. Some will want to and some
wont, those that don't can easily use the svn export method that Adrian
mentioned.
The patch itself is small in terms of code change and works for everybody
during development and those who choose to deploy with svn folders.
> Skip ".svn" subdirectories when searching for JAR files during ofbiz startup
> ----------------------------------------------------------------------------
>
> Key: OFBIZ-1712
> URL: https://issues.apache.org/jira/browse/OFBIZ-1712
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Environment: n/a
> Reporter: Yin T
> Priority: Trivial
> Attachments: start_filter_svn.patch
>
>
> During ofbiz startup, it looks for jar files in the /ofb/framework/base/lib.
> I noticed that when it is iterating through this subdirectory, it looks into
> the ".svn" subdirectory as well. This sub directory should be skipped.
> The files affected are shown below. I don't know why "CVS" string exists in
> the code. Perhaps its legacy code from way back.
> /ofbiz/framework/base/src/base/org/ofbiz/base/container/ComponentContainer.java,
> line 182
> change...
> if (componentPath.isDirectory() && !sub.equals("CVS")) {
> to...
> if (componentPath.isDirectory() && !sub.equals("CVS") &&
> !sub.equals(".svn")) {
> /ofbiz/framework/base/src/start/org/base/start/Start.java, line 174:
> change...
> if (file.isDirectory() && !"CVS".equals(fileName) && recurse) {
> to...
> if (file.isDirectory() && !"CVS".equals(fileName) &&
> !".svn".equals(fileName) && recurse) {
> ofbiz-trunk/framework/shark/webapp/shark/repository - showRepository.ftl
> There's an occurrence of "CVS" string as well. But I do not know
> Freemarker macros.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
