Issue 4552 - Drop jshydra dependency (buildtools)

packagerChrome.py

Index: packagerChrome.py
diff --git a/packagerChrome.py b/packagerChrome.py
index b81b665aff62c1fafb1c577632105a8258a3bec9..22bcac5a6a0e6adf4c7948fe73a3ff192ab09c5f 100644
--- a/packagerChrome.py
+++ b/packagerChrome.py
@@ -144,30 +144,40 @@ def createInfoModule(params):
 
 
 def convertJS(params, files):
-    from jshydra.abp_rewrite import rewrite_js
-
     output_files = collections.OrderedDict()
-    args = collections.defaultdict(list)
+    args = collections.defaultdict(dict)
 
     for item in params['metadata'].items('convert_js'):

[Wladimir Palant, 2016/11/30 14:54:44]

Nit: `for name, value in ...`? This should make the code below easier to
understand.

[kzar, 2016/11/30 15:26:17]

Well we need item still for `item.source`. How about this?

         filename, arg = re.search(r'^(.*?)(?:\[(.*)\])?$', item[0]).groups()
         if arg is None:
             output_files[filename] = (item[1].split(), item.source)
         else:
-            args[filename].append('{}={}'.format(arg, item[1]))
+            args[filename][arg] = item[1]

[Wladimir Palant, 2016/11/30 11:39:32]

Nit: Frankly, I'd prefer `args.setdefault(filename, {})[arg] = item[1]` instead
of the obscure games with defaultdict (yes, you didn't introduce that code).

[kzar, 2016/11/30 14:29:44]

Done.

+
+    template = getTemplate('modules.js.tmpl', autoEscape=True)

[Wladimir Palant, 2016/11/30 11:39:32]

Please remove autoEscape=True, this isn't an HTML or XML document. Escaping
entities makes no sense in JavaScript context.

[kzar, 2016/11/30 14:29:44]

Done.

 
     for filename, (input_files, origin) in output_files.iteritems():
         if '/' in filename and not files.isIncluded(filename):
             continue
 
+        args["filename"]["autoload"] = set(args.get("autoload", "").split(","))

[Wladimir Palant, 2016/11/30 11:39:32]

This line has three bugs, I assume that it hasn't been tested. How about you
store args[filename] in a local variable? This would elegantly squash the first
two bugs (accessing the wrong dictionary). The third issue is that
set("".split(",")) won't produce an empty set as you probably expected. Plus,
autoload order might be relevant - this shouldn't be a set. Altogether this
should probably be:

  current_args = args.get(filename, {})
  current_args['autoload'] = [
    module for module in current_args.get('autoload', '').split(',') if module
!= ''
  ]

Also: single quotation marks please (affects a bunch of other code parts as
well). You should run tox, it will point out these PEP-8 compliance issues.

[kzar, 2016/11/30 14:29:44]

Ouch, done.
Done.
In fairness I did run tox, unfortunately due to issue #4575 I didn't get any
useful output. (I had assumed I would get the useful output too, but apparently
it was masked by that issue instead.)

+
         base_dir = os.path.dirname(origin)
-        jshydra_args = ['--arg', ' '.join(args[filename])]
+        modules = []

[kzar, 2016/11/18 17:25:48]

List of tuples instead of a dict since the module names are occasionally the
same. For example "common" could mean a few different files. Only matters when
not using the module system of course however this way seemed simpler than
having two code paths. (Also order might matter.)

 
         for input_filename in input_files:
-            jshydra_args.append(os.path.join(base_dir, input_filename))
+            module_name = os.path.splitext(os.path.basename(input_filename))[0]
+            prefix = os.path.basename(os.path.dirname(input_filename))
+            if prefix != "lib":
+                module_name = "{}_{}".format(prefix, module_name)
+            with open(os.path.join(base_dir, input_filename), 'r') as f:

[Wladimir Palant, 2016/11/30 11:39:32]

Nit: file rather than f?

[kzar, 2016/11/30 14:29:44]

Done.

+                modules.append((module_name, f.read().decode('utf-8')))
             files.pop(input_filename, None)
 
-        files[filename] = rewrite_js(jshydra_args)
+        files[filename] = template.render(
+            args=args[filename],
+            modules=modules
+        ).encode('utf-8')
 
 
 def toJson(data):