edf2csv 0.8.65 → 0.8.67

package/dist/convert/run.js

@@ -1287,7 +1287,8 @@ function writeHint(cause, toStdout = false) {
         ? 'What reached stdout before it failed is incomplete and should not be used. '
         : 'The files written so far are incomplete and should not be used. ';
     return (preamble +
-        destinationAdvice(cause, toStdout ? 'redirect it somewhere else' : 'choose another with --out', toStdout ? 'redirect it somewhere shorter' : 'choose a shorter destination with --out'));
+        destinationAdvice(cause, toStdout ? 'redirect it somewhere else' : 'choose another with --out', toStdout ? 'redirect it somewhere shorter' : 'choose a shorter destination with --out', 'Part of that path no longer exists; make sure nothing is removing it while the ' +
+            'conversion runs.'));
 }
 /**
  * The same question asked before anything has been written: the output directory could not be
@@ -1305,10 +1306,11 @@ function writeHint(cause, toStdout = false) {
  * between the two, so the sentences themselves are shared rather than copied.
  */
 function createHint(cause) {
-    return destinationAdvice(cause, 'choose another with --out', 'choose a shorter destination with --out');
+    return destinationAdvice(cause, 'choose another with --out', 'choose a shorter destination with --out', 'Part of that path cannot be created — a symbolic link with nothing behind it, or a ' +
+        'mount point that is not mounted; choose another with --out.');
 }
 /** The sentence an errno earns, with the two phrases that name where "somewhere else" is. */
-function destinationAdvice(cause, elsewhere, shorter) {
+function destinationAdvice(cause, elsewhere, shorter, missing) {
     const code = cause?.code;
     switch (code) {
         case 'ENOSPC':
@@ -1322,8 +1324,26 @@ function destinationAdvice(cause, elsewhere, shorter) {
             return `That filesystem is mounted read-only; ${elsewhere}.`;
         case 'EISDIR':
             return `A directory is sitting where that file belongs; remove or rename it, or ${elsewhere}.`;
+        /*
+          The one errno whose sentence is not the same question at both call sites.
+    
+          Mid-conversion, a missing component is a directory that went away under a run that had
+          already written into it, and "make sure nothing is removing it" is the thing to check.
+          Before anything is written it is the opposite: the destination's parents are created
+          recursively, so reaching ENOENT there means a component that cannot be created — a
+          symbolic link with nothing behind it, or a mount point that is not mounted.
+    
+              $ edf2csv rec.edf --out /mnt/archive/out
+              error: Cannot create "/mnt/archive/out": part of the path does not exist.
+                     Part of that path no longer exists; make sure nothing is removing it
+                     while the conversion runs.
+    
+          Nothing is removing it and nothing was there. `createHint`'s own docstring says the
+          preamble "is the only thing that differs between the two, so the sentences themselves
+          are shared rather than copied" — and this is the sentence where that was not so.
+        */
         case 'ENOENT':
-            return `Part of that path no longer exists; make sure nothing is removing it while the conversion runs.`;
+            return missing;
         case 'ENAMETOOLONG':
             return `That path is longer than the filesystem allows; ${shorter}.`;
         case 'EMFILE':