geode icon indicating copy to clipboard operation
geode copied to clipboard

Fixing bug with geode home determination

Open thesteve0 opened this issue 3 years ago • 6 comments

If you place the product_dir in a path with "bin" in, the old method would ruin the path. For example C:\Users\steve\bin\apache-geode-1.14.4\bin\

would become C:\Users\steveapache-geode-1.14.4

This implementation cuts off the last 4 letters of the bin directory to get to the installation location.

For all changes:

  • [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

  • [ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

  • [ ] Is your initial contribution a single, squashed commit?

  • [ ] Does gradlew build run cleanly?

  • [ ] Have you written or updated unit tests to verify your changes?

  • [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

thesteve0 avatar Apr 04 '22 18:04 thesteve0

Changes made

thesteve0 avatar Apr 06 '22 16:04 thesteve0

That concourse build failure does not look related to my change - how do I clear it?

thesteve0 avatar Apr 10 '22 23:04 thesteve0

That concourse build failure does not look related to my change - how do I clear it?

@thesteve0 You can either use:

git commit --allow-empty-message -m "Trigger precheckin"

or if you have access to tanzu projects, you can trigger just precheckin for unit-test using:

https://cio.hdb.gemfire-ci.info/pr-tool

kirklund avatar Apr 12 '22 23:04 kirklund

Have you investigated why the unit tests are failing?

There are 37 failures - all of them Java exceptions some related to Mockito.

I changed 1 line in a windows bat file.

This seems to be something wrong with the test harness since I did not touch Java code

thesteve0 avatar Apr 19 '22 15:04 thesteve0