Return-Path: X-Original-To: apmail-accumulo-dev-archive@www.apache.org Delivered-To: apmail-accumulo-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id CEA661175B for ; Wed, 16 Jul 2014 13:23:42 +0000 (UTC) Received: (qmail 34293 invoked by uid 500); 16 Jul 2014 13:23:42 -0000 Delivered-To: apmail-accumulo-dev-archive@accumulo.apache.org Received: (qmail 34251 invoked by uid 500); 16 Jul 2014 13:23:42 -0000 Mailing-List: contact dev-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list dev@accumulo.apache.org Received: (qmail 34238 invoked by uid 99); 16 Jul 2014 13:23:42 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 16 Jul 2014 13:23:42 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 2F0721DB6BF; Wed, 16 Jul 2014 13:23:30 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4449575183485098187==" MIME-Version: 1.0 Subject: Re: Review Request 23393: ACCUMULO-2423 Clean up shell scripts From: "Alex Moundalexis" To: "Alex Moundalexis" Cc: "Mike Drob" , "accumulo" Date: Wed, 16 Jul 2014 13:23:30 -0000 Message-ID: <20140716132330.24004.4490@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Alex Moundalexis" X-ReviewGroup: accumulo X-ReviewRequest-URL: https://reviews.apache.org/r/23393/ X-Sender: "Alex Moundalexis" References: <20140716131147.24004.69835@reviews.apache.org> In-Reply-To: <20140716131147.24004.69835@reviews.apache.org> Reply-To: "Alex Moundalexis" X-ReviewRequest-Repository: accumulo --===============4449575183485098187== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit > On July 16, 2014, 1:11 p.m., Alex Moundalexis wrote: > > assemble/bin/stop-here.sh, line 42 > > > > > > prefer egrep -q '(localhost|127.0.0.1)' rather than escaping the pipe > > Mike Drob wrote: > parenthesis are necessary? On OS X they appear to be, plus it's more clear what's being done. > On July 16, 2014, 1:11 p.m., Alex Moundalexis wrote: > > assemble/bin/config.sh, lines 82-85 > > > > > > Presume the goal here is to echo literal $ACCUMULO_HOME... > > Mike Drob wrote: > Yes. Good to go, then. - Alex ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23393/#review47875 ----------------------------------------------------------- On July 10, 2014, 8:04 p.m., Mike Drob wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23393/ > ----------------------------------------------------------- > > (Updated July 10, 2014, 8:04 p.m.) > > > Review request for accumulo and Alex Moundalexis. > > > Bugs: ACCUMULO-2423 > https://issues.apache.org/jira/browse/ACCUMULO-2423 > > > Repository: accumulo > > > Description > ------- > > ACCUMULO-2423 Clean up shell scripts > > Standardize interpreter line on '#! /usr/bin/env foo' > Replace `..` with $(..) > Replace [..] with [[..]] > Replace = with == in [[..]] > Replace arithmetic [[..]] with ((..)) > Remove useless invocations of echo > Add double quotes to prevent word splitting > > > Diffs > ----- > > assemble/bin/LogForwarder.sh 5fe7871fc4b8b6aab9dbfc0227aa2e36f4a43aba > assemble/bin/accumulo bdd742d7aee81dc82872acee1b4333ee6e8348c7 > assemble/bin/bootstrap_config.sh fb643daf2f98f325bf041a3f75e179df74dfef59 > assemble/bin/bootstrap_hdfs.sh 6f38f633f4789ac7bc09639eeee58495ee4c3ea2 > assemble/bin/build_native_library.sh 907ce2c946ba1b6c4ddcc889011f899fc1d96575 > assemble/bin/config.sh 76634731abd2e51dc924804f76c1b73a9f5086d6 > assemble/bin/generate_monitor_certificate.sh edd415912bb0c67e203236122f5117fad88bf34e > assemble/bin/start-all.sh b71ba705889f6c157194ae4a816f8e21e9377b8b > assemble/bin/start-here.sh 9abdb387273c7e7fed24928da7ba11183e4f5986 > assemble/bin/start-server.sh 62ad91d57731d7e32b3ce6e51a961b11c64bf163 > assemble/bin/stop-all.sh 3bd30bd758a9226d7e1e8e0b66cd36704e052d0e > assemble/bin/stop-here.sh 4d5533acc6b4b4fc4aadde97ae95906ece347492 > assemble/bin/stop-server.sh 6fbe0afde2b881cb7ded6b66d6fcaad35102d7cd > assemble/bin/tool.sh 8126b3b1a5753a395a21f47fe7849de79af05135 > assemble/bin/tup.sh b26def5790cf322cbd203fd53faa337d3cec617d > assemble/build.sh 5f65419dcff6120dc6e7ca46905420936b439953 > assemble/conf/templates/accumulo-env.sh e136a3f41a86ad710e1f93f15dc1dfadf3a4de03 > core/src/main/scripts/generate-protobuf.sh 9a3b08a34b4eed50b74bf502de1bc64daa80418a > core/src/main/scripts/generate-thrift.sh ee51b11a921205e6164bf6c5303d874fd4278ee4 > proxy/examples/python/TestClient.py 5509ded0946a16455eea8b570a96a71ba15d4ebf > start/src/test/shell/makeTestJars.sh 13018b1973fbddf1b8885f156d0c6d7196574d14 > test/src/test/scripts/run-test.sh 02a5090e622fdd22b0239cd4a48117067878f94c > test/system/continuous/analyze-missing.sh 62ad15b6e7a6a1fe5b707e9f3eced64763b002c1 > test/system/continuous/continuous-env.sh.example d3fdd62768afafc496d4e94ecadfe26c0dc84454 > test/system/continuous/mapred-setup.sh e6bd2caa8cb2557c23b17740f89102482f4300b0 > test/system/continuous/run-moru.sh 3133b0518c9153c14d8cb1a8e78adc7ab1dc0333 > test/system/continuous/run-verify.sh 72ecc14fecf18ab8e8f14c8e2420e8f0102701f3 > test/system/continuous/start-agitator.sh e82e5ab66866122161a0b6e1be132114b42e94da > test/system/continuous/start-batchwalkers.sh b3dd7becf677b28d81349f3622cb18d612d786d0 > test/system/continuous/start-ingest.sh 468482935dcddf63a87b23f2695c0b6c057f3ffb > test/system/continuous/start-scanners.sh 63b7fb318ddb3ae9d1c396bee493083f6403d1b1 > test/system/continuous/start-stats.sh a9013c44f942eaf3bc54096059b02ae9dfe6e503 > test/system/continuous/start-walkers.sh 32fbd515b71a75f6ebb36e437be8babe1c3a1504 > test/system/continuous/stop-agitator.sh 136b451f2303b5900cdfd746956752ff4d96fa54 > test/system/continuous/stop-batchwalkers.sh 0250656291572bbb9c10faa15bcee6872aada587 > test/system/continuous/stop-ingest.sh 15f1715ebfc128a825ae65abf8262f5af6abd442 > test/system/continuous/stop-scanners.sh 8b1852b39df4dad803ee9c04ac8a08699e06d83c > test/system/continuous/stop-stats.sh a806f2d30e13bae929678318b075a1899f8cef26 > test/system/continuous/stop-walkers.sh 7e759698833e04b54592ad201c702cdfe3df10a8 > test/system/randomwalk/bin/apocalypse.sh 899256c8521bd7c319533ed8fd24676a367b65d6 > test/system/randomwalk/bin/copy-config.sh 01a90e0579c31200fca8b91673cf865e36d6daf0 > test/system/randomwalk/bin/kill-all.sh 75e73af8d5b5294a98e01ab96bb246bb8f95f89c > test/system/randomwalk/bin/kill-local.sh 1d5af9b0545c640cad2e258e549099b4ec5ea199 > test/system/randomwalk/bin/reset-cluster.sh d3942220181a9e150177818c2dad447f620acaad > test/system/randomwalk/bin/start-all.sh 48219c6ea72cff08811ef5fadbc074c51c99f8e2 > test/system/randomwalk/bin/start-local.sh 524a80b1ec0e4696332dd607be2f57354e5887dc > test/system/stress/reader.sh 11386850e45073b2620ae19d11d1930b529f7fff > test/system/stress/start-readers.sh 709c46e27adbee35c137da837f48ea5de2508545 > test/system/stress/start-writers.sh b9464ccf8a39945cc1fe09cc5246df3f5bc4af0b > test/system/stress/stop-readers.sh 1c5997185a66b952bc3eb2c95e2a96bbb31b151e > test/system/stress/stop-writers.sh d925cb331118d0446728f849a5fab5e6e1fd6389 > test/system/stress/writer.sh ef26fed1c0761aadd2d69b9cd14970c3298f2969 > test/system/upgrade_test.sh 6259e1c3f267947facd7c712c5fa7ae3b114c242 > > Diff: https://reviews.apache.org/r/23393/diff/ > > > Testing > ------- > > > Thanks, > > Mike Drob > > --===============4449575183485098187==--