mpdevil fails if there are multiple artists tags (even multiple album artist tags) about plattenalbum

[SoongNoonien]

Ok, I'll fix this. But I have to admit, I've no idea how to create such a file for testing. Which program did you use to add multiple artist tags?

On Windows (/with WINE) you can use MP3Tag, multiple tags with NULL are delimited with an double backslash, for example the artist tag for one of the files from above would be "Melissa Benoist\\Grant Gustin". You can recheck the right parsing if you right click on the file in the left panel and select Extended tags. When there are multiple tags from the same type underneath the first one, it worked!

On Linux you can use Kid3(-qt). Multiple tags are delimited with the pipe symbol ("|").

[SoongNoonien]

Thanks! How would you expect the artists to be displayed, separated by comma like on your Lollypop screenshot?

Yes, I think that would be the format expected by the most people; separated by comma with a space after each comma. Some programs separate them by semicolon without space, but that's worse readable than comma with space, in my opinion.

[SoongNoonien]

Ok, I'll go for the commas with spaces.

[SoongNoonien]

I've committed a fix. I tested it with some .mp3 and .ogg files. To try it just install mpdevil directly from master or if you are using Arch you can use my git PKGBUILD.

# Maintainer: Martin Wagner <[email protected]>
# Contributor: Martin Wagner <[email protected]>

pkgname=mpdevil
pkgver=v0.4.4.r2.g08ff73d
pkgrel=1
pkgdesc="A small MPD client written in python"
arch=('any')
license=('GPL3')
url="https://github.com/SoongNoonien/mpdevil"
depends=('python-mpd2' 'gtk3' 'libnotify' 'python-gobject' 'python-requests' 'python-beautifulsoup4' 'python-dbus')
makedepends=('intltool')
source=('git+https://github.com/SoongNoonien/mpdevil.git')
md5sums=('SKIP')

pkgver() {
  cd "$pkgname"
  git describe --long --tags | sed 's/\([^-]*-g\)/r\1/;s/-/./g'
}

prepare() {
  cd "${srcdir}/${pkgname}"
  NOCONFIGURE=1 ./autogen.sh
}

build() {
  cd "${srcdir}/${pkgname}"
  ./configure --prefix=/usr 
  make
}

package() {
  cd "${srcdir}/${pkgname}"
  make DESTDIR="${pkgdir}" install
}

I tested it with a lot of albums, as I use multiple tags a lot. All of them worked! I'll close the issue...

[SoongNoonien]

Thanks a lot for testing!