Fix Failing CI setup (#58)

* use detected.name and detected.arch instead of .classifier which has better luck * Update reflect-config with new module structure from Larky * Missed larky.modules in front of larky.types Larky
verygoodsecurity · Mar 30, 2021 · 3527e41 · 3527e41
1 parent 377a976
commit 3527e41
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 28 deletions.
diff --git a/pom.xml b/pom.xml
@@ -116,7 +116,7 @@
             <groupId>org.conscrypt</groupId>
             <artifactId>conscrypt-openjdk</artifactId>
             <version>${org.conscrypt.version}</version>
-            <classifier>${os.detected.classifier}</classifier>
+            <classifier>${os.detected.name}-${os.detected.arch}</classifier>
         </dependency>
 
         <dependency>
@@ -161,6 +161,14 @@
     </dependencies>
 
     <build>
+        <extensions>
+            <extension>
+              <groupId>kr.motd.maven</groupId>
+              <artifactId>os-maven-plugin</artifactId>
+              <version>${os-maven-plugin.version}</version>
+            </extension>
+        </extensions>
+
         <plugins>
 
             <plugin>
@@ -214,19 +222,6 @@
                 <artifactId>maven-source-plugin</artifactId>
                 <version>${maven.source.plugin.version}</version>
             </plugin>
-            <plugin>
-                <groupId>kr.motd.maven</groupId>
-                <artifactId>os-maven-plugin</artifactId>
-                <version>${os-maven-plugin.version}</version>
-                <executions>
-                    <execution>
-                        <phase>initialize</phase>
-                        <goals>
-                            <goal>detect</goal>
-                        </goals>
-                    </execution>
-                </executions>
-            </plugin>
         </plugins>
     </build>
 

diff --git a/runlarky/pom.xml b/runlarky/pom.xml
@@ -85,7 +85,12 @@
                 <version>${protobuf-maven-plugin}</version>
                 <configuration>
                     <protocArtifact>
-                        com.google.protobuf:protoc:${protobuf.version}:exe:${os.detected.classifier}
+                    <!--
+                        See https://github.com/verygoodsecurity/starlarky/pull/58 for why
+                        ${os.detected.classifier} didn't work in some cases for my environment but
+                        ${os.detected.name}-${os.detected.arch} works better!
+                    -->
+                        com.google.protobuf:protoc:${protobuf.version}:exe:${os.detected.name}-${os.detected.arch}
                     </protocArtifact>
                 </configuration>
                 <executions>

diff --git a/runlarky/src/main/resources/reflect-config.json b/runlarky/src/main/resources/reflect-config.json
@@ -1,102 +1,110 @@
 [
   {
-    "name": "com.verygood.security.larky.nativelib.LarkyGlobals",
+    "name": "com.verygood.security.larky.modules.globals.LarkyGlobals",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.PythonBuiltins",
+    "name": "com.verygood.security.larky.modules.globals.PythonBuiltins",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.std.Json",
+    "name": "com.verygood.security.larky.modules.JsonModule",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.std.Proto",
+    "name": "com.verygood.security.larky.modules.ProtoBufModule",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.std.C99Math",
+    "name": "com.verygood.security.larky.modules.C99MathModule",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.std.Hashlib",
+    "name": "com.verygood.security.larky.modules.HashModule",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.stdtypes.structs.MutableStruct",
+    "name": "com.verygood.security.larky.modules.RegexModule",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.stdtypes.structs.SimpleStruct",
+    "name": "com.verygood.security.larky.modules.types.structs.MutableStruct",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.nativelib.LarkyProperty",
+    "name": "com.verygood.security.larky.modules.types.structs.SimpleStruct",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "net.starlark.java.eval.Dict",
+    "name": "com.verygood.security.larky.modules.types.Property",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "net.starlark.java.eval.StringModule",
+    "name": "com.verygood.security.larky.modules.types.Partial",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "net.starlark.java.eval.StarlarkList",
+    "name": "net.starlark.java.eval.Dict",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,
     "allDeclaredConstructors": true,
     "allPublicConstructors": true
   },
   {
-    "name": "com.verygood.security.larky.stdtypes.hashing.HashObject",
+    "name": "net.starlark.java.eval.StringModule",
+    "allDeclaredFields": true,
+    "allDeclaredMethods": true,
+    "allPublicMethods": true,
+    "allDeclaredConstructors": true,
+    "allPublicConstructors": true
+  },
+  {
+    "name": "net.starlark.java.eval.StarlarkList",
     "allDeclaredFields": true,
     "allDeclaredMethods": true,
     "allPublicMethods": true,