firmware: Ignore EV3 error on GetVersion.

We don't use this datum for anything, and the rest of the flashing
process apparently proceeds fine even if this fails. This could use
further investigation, though.

Author: jaguilar

src/firmware/sagas.ts

@@ -1221,12 +1221,19 @@ function* handleFlashEV3(action: ReturnType<typeof firmwareFlashEV3>): Generator
 
     defined(version);
 
-    console.debug(
-        `EV3 bootloader version: ${version.getUint32(
-            0,
-            true,
-        )}, HW version: ${version.getUint32(4, true)}`,
-    );
+    // For reasons that we do not currently understand, some EV3s do not return
+    // anything to our GetVersion command. We don't actually use the version
+    // for anything so we will just ignore this error.
+    try {
+        console.debug(
+            `EV3 bootloader version: ${version.getUint32(
+                0,
+                true,
+            )}, HW version: ${version.getUint32(4, true)}`,
+        );
+    } catch (err) {
+        console.error(`Failed to parse ev3 version response: ${ensureError(err)}`);
+    }
 
     // FIXME: should be called much earlier.
     yield* put(didStart());