Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Playlist does not work when the file resides in a music-path that is not the first #437

Closed
jmbannon opened this issue Dec 21, 2023 · 4 comments

Comments

@jmbannon
Copy link
Contributor

jmbannon commented Dec 21, 2023

gonic version: v0.16.2

When reading from a playlist where a track resides in a music-path that is not the first one, it says it cannot be found:

2023/12/21 15:04:25 error looking up path "/Music/TaggedManual/El Jazzy Chavo/[2021] Strangelands/09 - A Distant Star.mp3": not found
2023/12/21 15:04:25 error looking up path "/Music/TaggedManual/El Jazzy Chavo/[2018] Redirections/04 - Dreams In A Sewer.mp3": not found

Docker-compose:

  gonic:
    container_name: gonic
    image: sentriz/gonic:latest
    environment:
      - TZ=America/Los_Angeles
      - GONIC_MUSIC_PATH=/Music/Tagged,/Music/TaggedManual
      - GONIC_MULTI_VALUE_GENRE=multi
      - GONIC_MULTI_VALUE_ARTIST=multi
      - GONIC_MULTI_VALUE_ALBUM_ARTIST=multi
      - GONIC_LISTEN_ADDR=0.0.0.0:4747
      # optionally, see more available env vars in the readme
    ports:
      - 4747:4747
    volumes:
      - ${APP_PATH}/gonic:/data
      - ${NAS_PATH}/Music/Playlists/MusicBee/Exported Playlists:/playlists
      - ${NAS_PATH}/Music/Tagged:/Music/Tagged:ro
      - ${NAS_PATH}/Music/TaggedManual:/Music/TaggedManual:ro

Maybe also related, but in DSub when I try to view the TaggedManual folder, it results in no data. However, if a select All folders, stuff in TaggedManual is there.

@sentriz
Copy link
Owner

sentriz commented Dec 22, 2023

fixed by #439

@sentriz sentriz closed this as completed Dec 22, 2023
@jmbannon
Copy link
Contributor Author

Unfortunately the issue still persists - same error message

@jmbannon
Copy link
Contributor Author

Copy-pasting my live debugging here for verbosity

[11:10:04 PM] looks like the second path isn't getting used in the albums table
[11:10:07 PM] sqlite> select root_dir from albums group by root_dir;
[11:10:15 PM] /Music/Tagged
[11:14:05 PM] sqlite> select left_path from albums where tag_album_artist = "El Jazzy Chavo";
[11:14:19 PM] ../TaggedManual/El Jazzy Chavo/
[11:14:25 PM] :think:
[11:24:32 PM] tried stripping the /Music dir of the docker mount to just /Tagged + /TaggedManual, also didn't work
[11:30:07 PM] pretty confident its another path subset issue
[11:30:23 PM] /Tagged is the only unique root_dir in the albums table
[11:30:36 PM] which probably breaks assumptions in that query in the file we edited
[11:31:24 PM] cuz it will try /TaggedManual as the root_dir, which doesn't exist in the table
[11:36:29 PM] really dont want to have to change paths, but i probably will out of desperation tomorrow lol
[11:36:38 PM] calling it a night for now

sentriz added a commit that referenced this issue Dec 22, 2023
@sentriz
Copy link
Owner

sentriz commented Dec 22, 2023

@jmbannon ah it looks like that bug also exists in the scanner f5b6b4d fixed now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants