[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-12 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r83104105 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-12 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r83104635 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-12 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r83083196 --- Diff: INSTALL.md --- @@ -26,33 +25,41 @@ source code. Unpack as follows. tar xzf /accumulo-X.Y.Z-bin.tar.gz cd accumulo-X.Y.Z

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-12 Thread billierinaldi
Github user billierinaldi commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r83066536 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-11 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82926529 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-11 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82909346 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-11 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82907896 --- Diff: INSTALL.md --- @@ -26,33 +25,41 @@ source code. Unpack as follows. tar xzf /accumulo-X.Y.Z-bin.tar.gz cd accumulo-X.Y.Z

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82539162 --- Diff: INSTALL.md --- @@ -26,33 +25,41 @@ source code. Unpack as follows. tar xzf /accumulo-X.Y.Z-bin.tar.gz cd accumulo-X.Y.Z

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82538794 --- Diff: INSTALL.md --- @@ -26,33 +25,41 @@ source code. Unpack as follows. tar xzf /accumulo-X.Y.Z-bin.tar.gz cd accumulo-X.Y.Z

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82539018 --- Diff: assemble/bin/accumulo-single --- @@ -0,0 +1,80 @@ +#! /usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82538889 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82539085 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-09 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/160#discussion_r82539061 --- Diff: assemble/bin/accumulo --- @@ -51,137 +75,179 @@ locationByProgram() eval "${2}=${RESULT}" } -test -z "${JAVA_HOME}"

[GitHub] accumulo pull request #160: Simplify Accumulo scripts and config files

2016-10-06 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/160 Simplify Accumulo scripts and config files See [JIRA issue](https://issues.apache.org/jira/browse/ACCUMULO-4490) for background. This pull request is not ready to be merged. I am