Integrate the repl completely into scald.rb. Fixup scald-rb for better hdfs-local mode now with our provides

[ianoc]

Tested with ruby 1.9.3 , may not support 1.8.7, though not really too worried about that.

[sriramkrishnan]

Don't know enough Ruby to review this properly, but couple of comments:

• You can remove the common.sh, which should no longer be needed. You can also get rid of that from the test_tutorial.sh if it is no longer needed.
• Update needed to https://github.com/twitter/scalding/blob/develop/scalding-repl/README.md.

[sriramkrishnan]

I would keep the repl test.

[bholt]

Cool, thanks @ianoc! Can't test it at the moment, but looks like you've gotten it to find the paths for Hadoop, excellent.

[bholt]

I did have a couple problems running from 1.8.7, all it took to make it work for both (for me), was the following:

diff --git a/scripts/scald.rb b/scripts/scald.rb
index b6bcde6..b064668 100755
--- a/scripts/scald.rb
+++ b/scripts/scald.rb
@@ -165,7 +165,7 @@ def find_dependencies(org, dep, version)
   mapVer
 end

-def find_dependency(org="org.scala-lang", dep, version)
+def find_dependency(dep, version, org="org.scala-lang")
   dep = find_dependencies(org, dep, version)["#{org}:#{dep}:#{version}"]
   raise "Dependency #{dep}:#{version} not found" unless dep
   dep
@@ -574,10 +574,7 @@ SHELL_COMMAND =
   end

 def getStty()
-  r, w = IO.pipe
-  system("stty -g", :out=>w, :err=>"/dev/null")
-  w.close
-  r.read.strip
+  `stty -g 2>/dev/null`
 end

 def restoreStty(stty)

(Ruby 1.8.7 doesn't support out of order default args, and system has undergone several changes throughout ruby versions, so just sticking with backticks in this case seems more portable)

[bholt]

I also am just a bit irked by the warning: already initialized constant SCALA_LIB_DIR message I get every time. We could just rename the variable to be a global variable rather than constant ($scala_lib_dir).

[bholt]

One more thing: should we default to enabling -Yrepl-sync when they specify --repl? It seems like a sensible default, and it would not be something I would necessarily think to look for a flag to fix.

[bholt]

This should close #888.

[ianoc]

I removed the default arg... moving it to the end breaks maven conventions. Uses the backticks

[bholt]

lgtm, runs on 1.8.7

[ianoc]

@johnynek merge?