[PR #9706] Fix .ignore files not being honored in extras directories #12197

Closed
opened 2026-02-07 06:51:43 +03:00 by OVERLORD · 0 comments
Owner

Original Pull Request: https://github.com/jellyfin/jellyfin/pull/9706

State: closed
Merged: No


Changes

Added a patch to fix .ignore files not working in extras folders. This is done by skipping the resolving of the current extra directory files, if it contains a .ignore file.

Ideally, we would make use of the ignore file check inside the ResolvePath method that folders like season folders also go through. But due to some limitations, like limited access to some needed information from inside the FindExtras method, this couldn't be done easily without the need of a big refactor.

Issues

Fixes #9571

**Original Pull Request:** https://github.com/jellyfin/jellyfin/pull/9706 **State:** closed **Merged:** No --- <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> Added a patch to fix `.ignore` files not working in `extras` folders. This is done by skipping the resolving of the current extra directory files, if it contains a `.ignore` file. Ideally, we would make use of the ignore file check inside the `ResolvePath` method that folders like season folders also go through. But due to some limitations, like limited access to some needed information from inside the FindExtras method, this couldn't be done easily without the need of a big refactor. **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Fixes #9571
OVERLORD added the pull-request label 2026-02-07 06:51:43 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#12197