Merge pull request #9 from NoRedInk/drop-sed

Drop sed

native_package_install.py

@@ -2,11 +2,11 @@
 
 from __future__ import print_function
 
-import collections
 import argparse
+import collections
+import fnmatch
 import json
 import os
-import subprocess
 import sys
 import tarfile
 import urllib2
@@ -141,29 +141,39 @@ def get_source_dirs(vendor_dir, package):
     return data['source-directories']
 
 
+def replace_in_file(filePath, src, target):
+    """ find replace in a file """
+    output = ""
+    with open(filePath) as infile:
+        output = infile.read().replace(src, target)
+    with open(filePath, 'w') as outfile:
+        outfile.write(output)
+
+
+def find_all_native_files(path):
+    """ recursivly find all js files in a package """
+    native_files = []
+    for root, dirnames, filenames in os.walk(path):
+        if "Native" not in root:
+            continue
+        for filename in fnmatch.filter(filenames, '*.js'):
+            native_files.append(os.path.join(root, filename))
+    return native_files
+
+
 def munge_names(vendor_dir, repository, packages):
     """
     Replaces the namespaced function names in all native code by the namespace from the given elm-package.json.
     """
     namespace, name = namespace_from_repo(repository)
     for package in packages:
-        subprocess.Popen((
-          "find",
-          package_dir(vendor_dir, package),
-          "-type",
-          "f",
-          "-exec",
-          "sed",
-          "-i",
-          "",
-          "-e",
-          "s/{0}/{1}/g".format(
-              format_native_name(package['namespace'], package['name']),
-              format_native_name(namespace, name)
-          ),
-          "{}",
-          ";"
-        ))
+        native_files = find_all_native_files(package_dir(vendor_dir, package))
+        for native_file in native_files:
+            replace_in_file(
+                native_file,
+                format_native_name(package['namespace'], package['name']),
+                format_native_name(namespace, name)
+            )
 
 
 def update_elm_package(vendor_dir, configs, packages):