SonarrAutoImport icon indicating copy to clipboard operation
SonarrAutoImport copied to clipboard

Params -dry-run | -v | \ in path doesn't work on Windows

Open sunnyd24 opened this issue 2 years ago • 2 comments

It seems the windows binary v1.5 doesn't accept -dry-run or -v as params, as per readme file excerpt below:

There are also some optional params:
-v Enables verbose logging
-dry-run will scan the folder for video files, but not call the Sonarr API

The command I am using is below with the error output:

SonarrAutoImport -dry-run
[11:21:52.373--INF] === Sonarr Auto Import Log Started ===
[11:21:52.430--INF] LogLevel: Information
[11:21:52.517--ERR] Settings not found: ry-run

I have found that the readme needs updating to reflect 3 things if using windows:

  1. use --dry-run instead of -dry-run
  2. use --v instead of -v
  3. use double backslashes in paths

Also, the readme could do with a good description as to the difference between "mappingPath" and "downloadsFolder". This has been provided in a separate issue by @ jortan with thanks.

Originally posted by @jortan in https://github.com/Webreaper/SonarrAutoImport/issues/10#issuecomment-1216620365

downloadsFolder is the path to media files to be processed from the perspective of SonarrAutoImport script
mappingPath is the path to media files to be processed from the perspective of Sonarr

These should point to the same destination. In some scenarios (docker) the same destination might require a different path.

sunnyd24 avatar Jan 10 '24 10:01 sunnyd24

Sounds good - want to submit a PR with the appropriate readme changes?

Webreaper avatar Jan 10 '24 10:01 Webreaper

Sounds good - want to submit a PR with the appropriate readme changes?

Happy to but I haven't done this before. I think this is the process looking at GitHub help, could you confirm?

  1. You need to assign me to this issue
  2. I create a pull request
  3. I make changes to readme.md
  4. I send to you for review/approval
  5. You merge changes and up-issue

sunnyd24 avatar Jan 10 '24 11:01 sunnyd24