XBOX 360 Media Sharing is broken.

[bernard.van.vlimmeren@…]

I recently (today) upgraded my mythtv version to the most recent code (including patches up till today). Unfortunately the upnp server does not return the correct data for the XBOX 360 anymore. As far as I can determine an xbox compatible upnp server needs to comply with: ​http://msdn.microsoft.com/en-us/windows/hardware/gg487545 However, when browsing videos (requesting containerId=15) from the xbox the server returns a list of object.item.videoitem while the xbox expects object.storagefolder. This causes the displayed list of videos to be empty.

In addition, when searching for music, the xbox requests containerId=7 which returns for some reason all recordings.

[bernard.van.vlimmeren@…]

Version

[Raymond Wagner]

Resetting this back to default priority.

[Joe Bryant <tenminjoe@…>]

Patch to make UPnP compatible with Xbox 360

[Joe Bryant <tenminjoe@…>]

I've started work on making MythTV's UPnP compatible with the Xbox 360 again.

In this patch: Viewing music "by album", viewing "all songs", playing individual songs. Only MP3s are supported - the Xbox 360 doesn't play OGG or FLAC. I haven't looked at the video support at all.

This is a testing-the-water patch as I have not worked on MythTV before. If this patch goes through okay then I intend to look at fixing up the UPnP support some more.

I hope that NON-Xbox 360 users who have UPnP devices can test against this patch to make sure it doesn't break UPnP for them.

[Joe Bryant <tenminjoe@…>]

Just noticed my patch has my home directory paths in it, betraying my total lack of experience with svn. Will try regenerating the patch correctly tomorrow.

[Jonathan <jonathan@…>]

Replying to Joe Bryant <tenminjoe@…>:

Just noticed my patch has my home directory paths in it, betraying my total lack of experience with svn. Will try regenerating the patch correctly tomorrow.

You patch seems to still have SVN revision numbers and is therefore quite out of date as the source has been moved from SVN to Git for a while now (about 8 months IIRC), see also: ​http://code.mythtv.org/trac

Perhaps you should first test the latest source from Git before diving in, it might save you chasing ghosts.

[Joe Bryant <tenminjoe@…>]

Thanks for letting me know - I'll get the git code and see what needs to be done.

It's a bit of a shame that it was so easy for me to end up working with the wrong code as a neophyte MythTV developer. I started at http://www.mythtv.org/wiki/Development_guide, which sent me to http://www.mythtv.org/docs/mythtv-HOWTO-5.html#ss5.2, which told me to get the latest source using subversion. The subversion repository is still up and running, so I assumed I was doing it right. Still, this is no doubt a discussion for elsewhere.

[Joe Bryant <tenminjoe@…>]

Revised diff, this time against latest git source

[Joe Bryant <tenminjoe@…>]

I've now recreated the patch against the latest git source (no significant changes).

[yannick.moussette@…]

Replying to Joe Bryant <tenminjoe@…>:

I've now recreated the patch against the latest git source (no significant changes).

Hi Joe, thanks for the hard work... Do you know if this ever made it to the official Mythtv tree? Using the new 0.25 and still having issues with Xbox 360...

Thanks!

Yannick

[Joe Bryant <tenminjoe@…>]

I never heard anything back after I submitted it, so I assume that, alas, it has not been included.

[Stuart Morgan <smorgan@…>]

In d8c0ad9930dd157106f651617d1217f1d6cce234/mythtv:

UPnP: Add the fraction of a second component to the 'Duration' attribute for music.

This is a no-op, it's entirely optional in the spec and we don't store
music duration to this resolution. I'm making the change because it
future proofs the upnp code should we ever decide to store music
duration down to the millisecond.

Refs #9632

[Stuart Morgan <smorgan@…>]

In 84705913ebfe904d6cb85a3b192b6eacfe6537e3/mythtv:

UPnP: Ensure resource URIs are properly encoded and formatted.

Also fixed a couple of bugs.

1. The 'medium' size artwork was being defined

twice for video and recordings.

2. The thumbnail for recordings was using PNG which is within the

spec, but DLNA requires JPG be offered at a minimum. Since no client
will support PNG but not JPG, we may as well only offer JPG for now.

Refs #9632
Fixes #12202

[stuartm]

To clarify matters, the XBox 360 does not support UPnP. It only supports streaming from Windows Media Centre using a hardcoded proprietary protocol based on a subsection of UPnP. It is not UPnP or DLNA compliant.

Given the XBox 360 has been replaced and the volume of hacks in our UPnP code just to support the XBox, I'm inclined to close this as "Won't Fix" so we can instead focus on improving support for true UPnP clients.