Don't use js_binary to copy the safari driver client. Buck/crazy-fun's

implementation re-adds the closure base library, which we don't need.

(#2277)

Author: jleyba

javascript/node/selenium-webdriver/safari.js

@@ -52,7 +52,7 @@ const portprober = require('./net/portprober');
 /** @const */
 const CLIENT_PATH = isDevMode
     ? path.join(__dirname,
-        '../../../build/javascript/safari-driver/client.js')
+        '../../../buck-out/gen/javascript/safari-driver/client.js')
     : path.join(__dirname, 'lib/safari/client.js');
 
 

javascript/safari-driver/BUCK

@@ -1,8 +1,8 @@
-js_binary(name = 'client',
-  srcs = ['client.js'],
-  flags = [
-    '--compilation_level=WHITESPACE_ONLY',
-  ],
+
+export_file(
+  name = 'client',
+  src = 'client.js',
+  out = 'client.js',
   visibility = [
     '//java/client/src/org/openqa/selenium/safari:client',
   ],

javascript/safari-driver/build.desc

@@ -1,7 +1,8 @@
 # This is a snapshot generated from the same target in the safari branch.
-js_binary(
+# We don't actually need it renamed, but we do want it copied to build
+# output.
+rename(
     name = "client",
     srcs = ["client.js"],
-    flags = [
-        "--compilation_level=WHITESPACE_ONLY",
-    ])
+    out = "client.js
+)