After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 753314 - No external subtitles for Panasonic again
No external subtitles for Panasonic again
Status: RESOLVED FIXED
Product: rygel
Classification: Applications
Component: IOP
unspecified
Other Linux
: Normal normal
: ---
Assigned To: rygel-maint
rygel-maint
Depends on: 753382
Blocks:
 
 
Reported: 2015-08-06 12:43 UTC by jan
Modified: 2015-08-09 11:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Difference in response between v0.24.2 and v0.26.1 (6.43 KB, text/plain)
2015-08-06 12:43 UTC, jan
  Details
server: Fix rewriting of subtitle uris in pv tags (1.50 KB, patch)
2015-08-08 07:05 UTC, Jens Georg
committed Details | Review
server: Add PV subtitle to all resources (1.71 KB, patch)
2015-08-09 11:18 UTC, Jens Georg
committed Details | Review

Description jan 2015-08-06 12:43:59 UTC
Created attachment 308847 [details]
Difference in response between v0.24.2 and v0.26.1

External subtitles has been added for Panasonic, see: https://bugzilla.gnome.org/show_bug.cgi?id=661228#c13

However:

Somewhere between v0.24.2 (Fedora 21) and v0.26.1 (Fedora 22) subtitles stopped working on my Panasonic TX-P50GT60.

I captured the difference between the versions, the most obvious differences:

The "pv:subtitleFileType" and "pv:subtitleFileUri" attributes are missing from the first res element.
24: <res size="3132991716" protocolInfo="http-get:*:video/mp4:DLNA.ORG_OP=01" pv:subtitleFileType="SRT" pv:subtitleFileUri="http://192.168.1.171:34239/MediaExport/i/(...)/sub/0.srt">http://192.168.1.171:34239/MediaExport/i/(...).mp4</res>
26: <res size="3132991716" protocolInfo="http-get:*:video/mp4:DLNA.ORG_OP=01">http://192.168.1.171:52468/MediaExport/i/(...)/res/primary_http.mp4</res>

The other "pv:subtitleFileUri" attributes seem to have an invalid uri (file:///... instead of http://...)
24: <res protocolInfo="http-get:*:video/mpeg:DLNA.ORG_PN=MPEG_TS_SD_EU_ISO;DLNA.ORG_CI=1;DLNA.ORG_FLAGS=01700000000000000000000000000000" resolution="720x576" pv:subtitleFileType="SRT" pv:subtitleFileUri="http://192.168.1.171:34239/MediaExport/i/(...)/sub/0.srt">http://192.168.1.171:34239/MediaExport/i/(...)/tr/MPEG_TS_SD_EU_ISO.mpg</res>
26: <res resolution="720x576" protocolInfo="http-get:*:video/mpeg:DLNA.ORG_PN=MPEG_TS_SD_EU_ISO;DLNA.ORG_OP=10;DLNA.ORG_CI=1;DLNA.ORG_FLAGS=01700000000000000000000000000000" pv:subtitleFileType="SRT" pv:subtitleFileUri="file:///home/jan/Videos/wds04/Whatever.srt">http://192.168.1.171:52468/MediaExport/i/(...)/res/MPEG_TS_SD_EU_ISO.mpg</res>
Comment 1 Jens Georg 2015-08-07 17:15:50 UTC
Uch. ouch. How could I miss that. Sorry about that
Comment 2 Jens Georg 2015-08-08 07:05:09 UTC
Created attachment 308933 [details] [review]
server: Fix rewriting of subtitle uris in pv tags

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 3 Jens Georg 2015-08-08 12:21:20 UTC
Missing subtitle on first resource is due to bug 753382
Comment 4 Jens Georg 2015-08-09 11:18:24 UTC
Created attachment 308972 [details] [review]
server: Add PV subtitle to all resources

Work-around issue with resources without DLNA profile losing its dlna_flags.

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 5 Jens Georg 2015-08-09 11:33:38 UTC
Attachment 308933 [details] pushed as 542e5d2 - server: Fix rewriting of subtitle uris in pv tags
Attachment 308972 [details] pushed as eede589 - server: Add PV subtitle to all resources