[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-02-01 Thread pdxrunner
Github user pdxrunner commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-178041809 I'm closing this PR in favor of looking at the longer term solution of gradlew vs. gradle in the release distros. The secondary problem I found (path error in

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-02-01 Thread pdxrunner
Github user pdxrunner closed the pull request at: https://github.com/apache/incubator-geode/pull/82 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-01-29 Thread pdxrunner
Github user pdxrunner commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-176913037 You're correct. I should have changed "./gradlew" to just "gradle" for the .txt and .md files I updated. I also agree that gradlew, not gradle, is the correct

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-01-29 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/82#issuecomment-176894749 You changed these to be ./gradle. I don't think that makes sense. Either someone is using the gradle wrapper - './gradlew' - or they are using in install

[GitHub] incubator-geode pull request: Fixes GEODE-874

2016-01-29 Thread pdxrunner
GitHub user pdxrunner opened a pull request: https://github.com/apache/incubator-geode/pull/82 Fixes GEODE-874 Replaced gradlew commands with gradle in text files. Corrected classpath shown for compiling and running app in Geode in 5 Minutes section of README.md You can merg