Hi, I don't think I'll be able to check this out and fix it anytime soon. Would it work for you if I add you as a co-maintainer so you can push some fixes?
Search Criteria
Package Details: mpd-sacd 0.23.13-1
Package Actions
Git Clone URL: | https://aur.archlinux.org/mpd-sacd.git (read-only, click to copy) |
---|---|
Package Base: | mpd-sacd |
Description: | MPD with patches for SACD and DVDA ISO playback. (DVDA ISO playback temporary disabled) |
Upstream URL: | https://sourceforge.net/p/sacddecoder/mpd/MPD.git/ci/master/tree/ |
Keywords: | dsd dvda mpd sacd |
Licenses: | GPL-2.0-or-later |
Conflicts: | mpd |
Provides: | mpd |
Submitter: | melvinvermeeren |
Maintainer: | None |
Last Packager: | Archttila |
Votes: | 8 |
Popularity: | 0.117992 |
First Submitted: | 2016-02-08 18:52 (UTC) |
Last Updated: | 2024-04-28 17:29 (UTC) |
Dependencies (53)
- alsa-lib
- audiofile (audiofile-gitAUR)
- expat (expat-gitAUR)
- faad2 (faad2-gitAUR)
- ffmpeg4.4
- flac (flac-gitAUR)
- fluidsynth (fluidsynth-gitAUR)
- gcc-libs (gcc-libs-gitAUR, gccrs-libs-gitAUR, gcc11-libsAUR, gcc-libs-snapshotAUR)
- hicolor-icon-theme (hicolor-icon-theme-gitAUR)
- icu (icu-gitAUR)
- jackAUR (jack2-gitAUR, pipewire-full-jack-gitAUR, pipewire-jack-gitAUR, jack2, pipewire-jack)
- lame (lame-svnAUR, lame-altivecAUR)
- libao (libao-gitAUR)
- libcdio (libcdio-gitAUR)
- libcdio-paranoia (libcdio-paranoia-gitAUR)
- libgme (libgme-vgmplay-gitAUR, game_music_emu-kode54-gitAUR, libgme-gitAUR)
- libid3tag (libid3tag-gitAUR)
- libmad
- libmikmod
- libmms (libmms-gitAUR)
- Show 33 more dependencies...
Required by (57)
- cantata (requires mpd) (optional)
- cantata-git (requires mpd) (optional)
- cantata-qt6 (requires mpd) (optional)
- cantata-qt6-git (requires mpd) (optional)
- davis (requires mpd) (optional)
- dmscripts-git (requires mpd) (optional)
- emms-git (requires mpd) (optional)
- evad (requires mpd)
- evad-svn (requires mpd)
- impd-git (requires mpd)
- inori (requires mpd)
- kunst-git (requires mpd)
- mcg (requires mpd) (optional)
- mixramp (requires mpd) (optional)
- mmtc (requires mpd)
- moped (requires mpd)
- mpd-rich-presence-bin (requires mpd)
- mpdcovergrabber-git (requires mpd)
- mpdfav-git (requires mpd) (optional)
- mpdlcd (requires mpd)
- Show 37 more...
Sources (4)
melvinvermeeren commented on 2020-10-06 14:14 (UTC)
brucehsieh commented on 2020-10-06 09:37 (UTC)
This package doesn't work.
https://git.code.sf.net/p/sacddecoder/mpd/MPD.git#commit=3a0c90a1155ecfd2eb57e172818153c1c95ec363
This version has compiling errors.
https://git.code.sf.net/p/sacddecoder/mpd/MPD.git
The mater branch has linking errors.
brucehsieh commented on 2019-11-16 06:56 (UTC)
It needs to be patched. For your reference.
https://github.com/MusicPlayerDaemon/MPD/commit/818b7e0641f25f55c46bf654d8edb7911c10d584
mistic commented on 2019-01-06 22:58 (UTC)
Hi
Great!!!. Now all works fine... And last but not least, I have been able to understand, where the fault came from...
Best regards and thank's a lot again
melvinvermeeren commented on 2019-01-04 05:05 (UTC) (edited on 2019-01-06 13:27 (UTC) by melvinvermeeren)
Edit: fixed upstream also in 0.21.5-1
src/lib/sacdiso/sacd_dsdiff.cpp:464
id3_byte_t* dsdid3 = (id3_byte_t*)&id3tags[_track_index].data[0];
With some odd/malformed tags, this may be NULL. the crash happens after when id3_tag_parse() is called with this NULL value.
id3_byte_t* dsdid3 = (id3_byte_t*)&id3tags[_track_index].data[0];
if (dsdid3 == nullptr) {
return;
}
I simply changed it to this and this time the database finished updating properly with 0.21.4-1, skipping the poor tag causing the issue. I will notify upstream about this bug tomorrow.
@mistic can you check if your error is the same if you run with mpd --no-daemon --stderr --verbose
?
melvinvermeeren commented on 2019-01-04 03:55 (UTC) (edited on 2019-01-04 04:21 (UTC) by melvinvermeeren)
Happy new year to you too. :)
I have updated the pkg to the latest upstream commit, which also has changed the build system. Perhaps the bug was fixed, I notice in the changelog of WIP 0.21.4:
- database
- inotify: fix crash bug "terminate called after throwing ..."
For me however it discarded the database first and during scanning it crashed with the following: mpd: tag.c:640: id3_tag_parse: Assertion 'data' failed.
0.21.0-10 works fine for me.
You can try 0.20.12-1 with PKGBUILD rev 9f1893d43d96
and then change the git://git.musicpd.org/manisiutkin/mpd.git source to the new repo home. However you may also need an old version of libcdio, unless of course you disable the cdio-paranoia compilation option.
Let me know how it goes if possible, thanks.
Edit: After reverting to 0.21.0-10 it fails with the same error, appears something else is causing this. Will check later.
mistic commented on 2019-01-04 00:17 (UTC)
First, I wish you a happy new year... Sadly semms mpd-sacd-0.21.0-10 makes corrupted database, at least for me. Perhaps because this mpd version itself, not the manisiutkin patch.
It would be possible to access sources in order to compile an older mpd-sacd version? example: mpd-sacd-0.20.12-1, that works fine
Thank's a lot for your efforts and share with us
melvinvermeeren commented on 2018-02-12 16:46 (UTC)
Took a while but I found the new home for the sources:
https://sourceforge.net/p/sacddecoder/mpd/MPD.git/ci/master/tree/
pkg updated
melvinvermeeren commented on 2018-02-05 11:43 (UTC)
@delleceste I still had an up-to-date local repo, which can be found here:
https://git.mel.vin/mirror/mpd-sacd
I have been searching for the repo's new home but cannot find it. If it doesn't pop up within a few weeks I'll try to contact Maxim.
Pinned Comments
Archttila commented on 2024-04-28 17:34 (UTC) (edited on 2024-05-01 14:10 (UTC) by Archttila)
Please do not mark the package as out of date, because mpd-sacd is a fork of regular MPD