DaGeRe / peass

Tool for Performance analysis of software system
GNU Affero General Public License v3.0
10 stars 9 forks source link

Fixed detection of gradle module-names #108

Closed mawHBT closed 1 year ago

mawHBT commented 1 year ago

If you modify gradle module-names in settings.gradle like this:

rootProject.children.each { subproject ->
     subproject.projectDir = file("module-" + subproject.name)
 } 

peass added a "/" between prefix and suffix, resulting in "module-/subproject.name". I removed adding the "/".

DaGeRe commented 1 year ago

Thanks for the PR, this looks fine to me. Could you also test whether this works with https://github.com/DaGeRe/peass/tree/jadexDebug for you? In general, the modules are more easily detectable when the tasks are derived (since the tasks are assigned to modules). In the future, only the results from the ./gradlew tasks should be used (since parsing the files is cumbersome and might not always work).