iTunes 10 breaks Firefly Media Server / mt-daapd

Hi,
I'm working with the developer of forked-daapd and another developer with ARM experience to get forked-daapd running on embedded systems that don't use newer kernels. It's coming.

GB

That's great news GB - I think a lot of people are waiting for an update! See if you can get smart playlists added as well 😉

Yes if there is a version that work on DNS323 and others with fun_plug and no compilation we might all switch on this for sure!

Hello, I found in some cases folked-daapd failed to display correct information.
I am a Japanese. So I have many mp3 files tagged with Japanese.
I did a tiny test. I edited id3 tags using easytag and iTunes10 (mac version),
and I installed forked-daapd and mt-daapd in my debian squeeze box,
they share the same few mp3 files.
I confirmed the difference of mt-dappd and forked-daapd using rhythmbox.

http://vbnm.sakura.ne.jp/tinytest/

Would you tell the developer of forked-daapd this issue?
I will go back to iTunes9 for a while.
Thank you.

Just seen the info from Flyn, looked at forked-daapd and dmapd and if you don't mind the hardcoding mt-daapd to simply state the mediakind is music then you only need two lines of code to daap.c (I'm running mt-daapd 0.2.4.2). With the changes below I can browse/stream music to iTunes 10.

Both go in daap.c:

$ diff -u ../../mt-daapd-0.2.4.2.orig/src/daap.c daap.c
--- ../../mt-daapd-0.2.4.2.orig/src/daap.c 2010-09-07 21:42:56.904105700 +0100
+ daap.c 2010-09-07 22:52:47.747506391 +0100
@@ -133,6 +133,7 @@
{ 0x0C, "arif", "daap.resolveinfo" },
{ 0x05, "aeNV", "com.apple.itunes.norm-volume" },
{ 0x01, "aeSP", "com.apple.itunes.smart-playlist" },
+ { 0x01, "aeMK", "com.apple.itunes.mediakind" },
{ 0x00, NULL, NULL }
};

@@ -586,6 +587,9 @@
// g = g && daap addchar(mlit,"asur",3); /* rating */
if(song->year && (wantsMeta(meta, metaSongYear)))
g = g && daap addshort(mlit,"asyr",song->year);
+
+ // Added for iTunes 10 - hardcode mediakind to music
+ g = g && daap addint(mlit, "aeMK", 1);
}

if(g == 0)

These are the lines to change for mt-daapd-svn-1696 (the last nightly version released):

-----

<pre>diff -u mt-daapd-svn-1696.original/src/plugins/out-daap-proto.c mt-daapd-svn-1696/src/plugins/out-daap-proto.c</pre>
<pre>--- mt-daapd-svn-1696.original/src/plugins/out-daap-proto.c 2007-08-10 00:07:23.000000000 -0400</pre>
<pre> + mt-daapd-svn-1696/src/plugins/out-daap-proto.c 2010-09-07 19:04:44.932978839 -0400</pre>
<pre>@@ -716,6 +716,8 @@</pre>
<pre> /* aeHV */</pre>
<pre> size += 9;</pre>
<pre> </pre>
<pre>+ size += 12;</pre>
<pre>+</pre>
<pre> return size;</pre>
<pre> break;</pre>
<pre> </pre>
<pre>@@ -920,6 +922,9 @@</pre>
<pre> if(daap_wantsmeta(pinfo->meta, metaSongContentRating))</pre>
<pre> current += dmap addchar(current,"ascr",</pre>
<pre> atoi(valarray\[SG_CONTENTRATING\]));</pre>
<pre>+</pre>
<pre>+ current += dmap addint(current, "aeMK", 1);</pre>
<pre>+</pre>
<pre> return 0;</pre>
<pre> break;</pre>
-----

After adding these two lines, iTunes 10 works for me.

Message was edited by: paraquat

@Flyn: Thanks! That led me to making a patch.

I've made a .deb-package for Debian Lenny, which you can find together with the patch for mt-daapd r1696 at http://www.informaarten.nl/mt-daapd-for-itunes-10

This is an interesting thread. Has anyone compiled and tested successfully the new build for Windows as yet?

Hi,

What you're seeing is partly or essentially due to less-than-stellar ID3 support in ffmpeg 0.5 (version in Squeeze).

Using ffmpeg 0.6 from experimental (it's safe) should give better results, but it requires a rebuild of forked-daapd (to pick up the new libraries). Don't hesitate to get in touch with me by mail so we can discuss this.

JB.

Hi,

Just so that everyone knows, I'll update the Debian packages with Maarten's patch in both Lenny and Squeeze this week-end. I need to review the patch, make the builds and test them, which takes some time 🙂

Not to mention I'm not exactly familiar with the code anymore. Eh.

JB.

Thanks -- I applied that to a Slug -- a Linksys NSLU2 -- running OpenWRT and it works!

A remaining problem for me is that the server initially shows up in iTunes with the name "untitled playlist". As soon as you select it, the name given to it in the mt-daaspd.conf file comes through. Anyone else got this problem, or a solution?

Not a dealbreaker, but annoying all the same...

I can confirm that these code changes work on a fully patched CentOS 5.5 box. Just add the couple of lines, recompile and stream to iTunes 10 (Mac).

Thanks, paraquat!


-MB

Where can I find Daap.c? I am browsing through scp and I can't find it in any of my folders?

Maaarten's patch works great for iTunes, but it broke my ability to connect to the server with Rhythmbox. I've modified the code posted by tecknobabble and maaarten into a patch that does a bit more checking for the actual request. I'm not familiar with the mt-daapd code itself, but this appears to work for me with iTunes 9.2 (Windows), iTunes 10 (Windows and OSX), and Rhythmbox 0.12.8 (in Ubuntu 10.04).

I've formatted the patch so that anyone wanting to make the changes right now can do so using maaarten's excellent instructions, just replace the line *wget -P debian/patches http://www.informaarten.nl/mt-daapd-for-itunes-10/13_itunes10.dpatch* with *wget -P debian/patches http://zero-factorial.com/coding/13_itunes10.dpatch*.

Hopefully this will help, and thanks everyone else who helped narrow down the changes that needed to be made!

Hi Deledrius,

Sounds promising. Is anyone working on a Windows equivalent rather than just Linux distros?

Cheers,
Darrell.