yarle icon indicating copy to clipboard operation
yarle copied to clipboard

enexSources does not allow multiple .enex files

Open ecerulm opened this issue 1 year ago • 4 comments

Using 5.0.11

with the following in the config.json

  "enexSources": [
    "/Users/ecerulm/Downloads/evernote2023-03-18/boardgames.enex",
  ],

it works, As soon as I add a second file:

  "enexSources": [
    "/Users/ecerulm/Downloads/evernote2023-03-18/boardgames.enex",
    "/Users/ecerulm/Downloads/evernote2023-03-18/macos.enex"
  ],

then it assumes each path is a directory not an ENEX files and fails with

npx -p [email protected] yarle --configFile config_macos.json
...
node:fs:1438
  handleErrorFromBinding(ctx);
  ^

Error: ENOTDIR: not a directory, scandir '/Users/ecerulm/Downloads/evernote2023-03-18/boardgames.enex'
    at Object.readdirSync (node:fs:1438:3)
    at Object.run (/Users/ecerulm/.npm/_npx/cc64087a51a90dbd/node_modules/yarle-evernote-to-md/dist/dropTheRopeRunner.js:51:14)
    at Object.<anonymous> (/Users/ecerulm/.npm/_npx/cc64087a51a90dbd/node_modules/yarle-evernote-to-md/dist/dropTheRope.js:30:19)
    at Module._compile (node:internal/modules/cjs/loader:1275:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1329:10)
    at Module.load (node:internal/modules/cjs/loader:1133:32)
    at Module._load (node:internal/modules/cjs/loader:972:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12)
    at node:internal/main/run_main_module:23:47 {
  errno: -20,
  syscall: 'scandir',
  code: 'ENOTDIR',
  path: '/Users/ecerulm/Downloads/evernote2023-03-18/boardgames.enex'
}

ecerulm avatar Mar 19 '23 11:03 ecerulm

hi @ecerulm ,

Pls try to copy your enex files into a directory and set that directory as enexSources. In this cade Yarle should iterate over the contente of the directory and convert all the enex files in it.

akosbalasko avatar Mar 19 '23 12:03 akosbalasko

yes, I already did that to workaround it but the https://github.com/akosbalasko/yarle/blob/62d4e19b37f2b09fe8c927490eb702e296425076/README.md?plain=1#L147 should be updated to say that it's either a single ENEX file or a list of folders, the current multiple enex files, or a folder of the enex files seems misleading.

ecerulm avatar Mar 19 '23 12:03 ecerulm

@ecerulm okok, thank you, I'll fix it.

akosbalasko avatar Mar 19 '23 20:03 akosbalasko

#628 implemented this feature. 🙂

ipcjs avatar Apr 15 '24 15:04 ipcjs