improve dependencies

bin/parse.py

@@ -7,11 +7,10 @@ import json
 import string
 import itertools
 
-build_dir = os.environ.get("BUILD_DIR", "")
+scad_fn = sys.argv[1]
+d_fn = sys.argv[2]
 
-fn = sys.argv[1]
-
-with open(fn) as f:
+with open(scad_fn) as f:
     scad_text = f.read()
 
 poly_reqs = re.findall(r'<(.+\.gen\.scad)>', scad_text)
@@ -23,29 +22,33 @@ modules = re.findall(r"module\s+([^\s\(]+)\s*\("
                      r"wardings_k\s*=\s*(\[[^\]]+\])"
                      , scad_text, flags=re.S)
 
-modules_parsed = [(n, json.loads(o), json.loads(w)) for (n, o, w) in modules]
+if len(modules) == 0:
+    print("Could not find any compatible modules in {}".format(scad_fn), file=sys.stderr)
+    sys.exit(1)
+
+module = modules[0]
+
+(n, os, ws) = module
+
+# Dirty hack to parse OpenSCAD lists
+os = json.loads(os)
+ws = json.loads(ws)
 
 def sanitize(s):
     return "".join([c for c in s.lower() if c in string.ascii_lowercase + string.digits])
 
-all_keys = [(n, o, w) for (n, os, ws) in modules_parsed for o in os for w in ws]
+all_keys = [(n, o, w) for o in os for w in ws]
 
 def stl_filename(n, o, w):
     return "$(STL_DIR)/{n}_{o_s}_{w_s}.stl".format(n=n, o=o, w=w, o_s=sanitize(o), w_s=sanitize(w))
 
 all_keys_makefile = ["{stl_fn}: {scad_fn} {deps}\n\t$(SCAD) $(SCADFLAGS) -D 'outline=\"{o}\"' -D 'warding=\"{w}\"' {scad_fn} -o $@"
     .format(n=n, o=o, w=w,
-    scad_fn=fn, stl_fn=stl_filename(n, o, w), deps=" ".join(["$(POLY_DIR)/{}".format(r) for r in poly_reqs]))
+    scad_fn=scad_fn, stl_fn=stl_filename(n, o, w), deps=" ".join(["$(POLY_DIR)/{}".format(r) for r in poly_reqs]))
     for (n, o, w) in all_keys]
 all_stl = [stl_filename(n, o, w) for (n, o, w) in all_keys]
 all_keys_makefile.append("STL_OBJ += {}".format(" \\\n ".join(all_stl)))
 
-def stripfn(fn):
-    fn = os.path.basename(fn)
-    if fn.endswith(".scad"):
-        fn = fn[:-5]
-    return fn
-
-with open(os.path.join(build_dir, "{}.d".format(stripfn(fn))), "w") as f:
+with open(d_fn, "w") as f:
     print("\n".join(all_keys_makefile), file=f
 )