[kepler-dev] [Kepler - Bug #5669] replace ProjectLocator.findKeplerModulesDir() with something more robust.

[noreply at nceas.ucsb.edu]

Issue #5669 has been updated by Daniel Crawl.

Priority changed from Normal to High
Target version changed from Unspecified to 2.5.0

The kepler-tasks module cannot be patched: once a patch is installed for kepler-tasks, Kepler will not start. The problem is that findKeplerModulesDir() uses the location of the kepler-tasks jar as the location where Kepler is installed. Since the patch is in KeplerData/kepler.modules, findKeplerModulesDir() assumes that all modules are in this directory and will not find the modules in the Kepler installation directory.

----------------------------------------
Bug #5669: replace ProjectLocator.findKeplerModulesDir() with something more robust.
https://projects.ecoinformatics.org/ecoinfo/issues/5669#change-20710

* Author: Derik Barseghian
* Status: New
* Priority: High
* Assignee: jianwu jianwu
* Category: build system
* Target version: 2.5.0
* Bugzilla-Id: 5669
----------------------------------------
This method is problematic. Bug#5458 also mentions it.

    /**
     * 
     * FIXME replace this with something(s) more robust. Be careful when doing so, some things probably always want 
     * KeplerData/kepler.modules, others always the other "application modules dir", others one or the other depending on 
     * what exists.
     * 
     * On linux(at least) during ant run, MemoryProperties invokes this, and the search begins based on the location of ant.jar
     * i.e. /usr/share/ant/lib/ant.jar/build-area/modules.txt can be the first attempt.
     * 
     * find the project dir
     * WARNING this can return either the application modules dir, 
     * or KeplerData/kepler.modules, or a false positive (e.g. some arbitrary parent dir that happens to contain build-area/modules.txt), 
     * or the system-dependent default directory (in the catch).
     * @return
     */
    private static KeplerModulesDir findKeplerModulesDir()


-- 
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: https://projects.ecoinformatics.org/ecoinfo/my/account

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.nceas.ucsb.edu/kepler/pipermail/kepler-dev/attachments/20130827/0c2044e7/attachment.html>