search

RandomNinjaAtk / docker-lidarr-extended

lidarr-extended :: Lidarr application packaged with multiple scripts to provide additional functionality
GNU General Public License v3.0
279 stars 25 forks source link

[feat]: Script Incompatibility when Plex not in Docker #178

Open JacquelineHyde opened 1 year ago

JacquelineHyde commented 1 year ago

The PlexNotify & MetadataPostProcess bash files aren't accounting for the mapped volumes in the docker-compose.

To Reproduce Steps to reproduce the behavior:

  1. Test script
  2. See error

Expected behavior lidarrRootFolderPath="$(dirname "$lidarr_artist_path")" should pick up on the volume mount from the docker container mapping (e.g. /volume3/data:/data).

Log Data 2023-01-06 19:45:13 :: MetadataPostProcess :: 1.0.015 :: Processing :: . :: ERROR :: /data/media/music/General/They Might Be Giants not found within "." :: Exiting...

Docker Info:

Additional context When I changed the PlexNotify script line nine to lidarrRootFolderPath="/volume3$(dirname "$lidarr_artist_path")" it worked. Unfortunately, I'm not familiar enough with .bash scripting to fix the Metadata one for my machine, though. Changing line 43 didn't work.

RandomNinjaAtk commented 1 year ago

This is only designed to work with Plex in a container.

Edit: I don’t have a personal use case for this, so I will not be implementing this… However, PR welcome for a fix that also maintains existing compatibility…