Skip to content

Commit

Permalink
Improved: Don't exclude properties and labels file from the Jar
Browse files Browse the repository at this point in the history
(OFBIZ-11161)

In order to have an independent deployable jar, we need to include the
properties and labels inside the jar.

The properties and labels file was previously excluded from the jar
because it was not possible to replace the compile time values by
invalidating OFBiz caches which is convenient when developing
OFBiz. It was then necessary to reconstruct the jar and restart
OFBiz (See OFBIZ-8321 for more details).

With the recent improvment from revision 1865719 allowing to run OFBiz
without building a jar, it is now possible to enable this cache
invalidation by running both ‘gradle run’ in one shell and ‘gradlew
--continuous classes’ in a separate shell.  Doing so make the
combination of editing the label files and clearing the caches use
the new value defined in the source file.


git-svn-id: https://svn.apache.org/repos/asf/ofbiz/ofbiz-framework/trunk@1866971 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
mthl committed Sep 15, 2019
1 parent cfff42c commit 3236e70
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -278,16 +278,6 @@ sourceSets {
srcDirs += getDirectoryInActiveComponentsIfExists('config')
srcDirs += getDirectoryInActiveComponentsIfExists('dtd')
exclude excludedConfigFiles
// Below are necessary for unit tests run by Gradle and integration tests
exclude { FileTreeElement elem -> elem.getName().contains('Labels.xml') }
exclude { FileTreeElement elem -> elem.getName().contains('.properties') &&
!elem.getName().contains('start.properties') &&
!elem.getName().contains('load-data.properties') &&
!elem.getName().contains('debug.properties') &&
!elem.getName().contains('cache.properties') &&
!elem.getName().contains('test.properties') &&
!elem.getName().contains('rmi.properties')
}
}
}

Expand Down

0 comments on commit 3236e70

Please sign in to comment.