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

Update to minidlna 1.3.3 + Couple extra fixes #28

Merged
merged 13 commits into from
Sep 30, 2024
Merged

Update to minidlna 1.3.3 + Couple extra fixes #28

merged 13 commits into from
Sep 30, 2024

Conversation

HeavenVolkoff
Copy link
Contributor

@HeavenVolkoff HeavenVolkoff commented Sep 30, 2024

Hello,

This PR pulls the latest upstream updates (1.3.1, 1.3.2, 1.3.3). As well as some extra fixes from a couple of Debian patches. I also had to make a few small changes to the password logic for it to be compatible with some of the recent changes, and besides that, I applied a patch that adds support for generating on-the-fly thumbnails with libffmpegthumbnailer for video files. As far as I could test, everything seems to be working fine.

Closes: #27

HeavenVolkoff and others added 13 commits September 29, 2024 23:18
This prevents creation of multiple album containers for compilation
albums (i.e. one separate album for each artist) when adding files via
inotify, which causes the scanner cache to be invalidated all the time.

OTOH, different albums with identical names ("Greatest Hits", "Love
Songs") are now listed as just a single album in the "Album" container.
This might be solved using some heuristic based on music file location
(same directory / different directories).

The new behaviour is usually the same as encountered when doing a full
rescan, because there's a scanner cache that is used during full rescans
and that cache ignores artist names as well.

Debian patch: https://sources.debian.org/patches/minidlna/1.3.3%2Bdfsg-1.1/07-fix-multi-artist-album-handling.patch
@NathanaelA NathanaelA merged commit b1377ce into NathanaelA:master Sep 30, 2024
@NathanaelA
Copy link
Owner

Wow, thanks for all the hard work!!!

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

Successfully merging this pull request may close these issues.

./autogen.sh ERROR
6 participants