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 509020 - Won't open DVD ISOs automatically
Won't open DVD ISOs automatically
Status: RESOLVED FIXED
Product: totem-pl-parser
Classification: Core
Component: General
Old
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2008-01-12 20:29 UTC by Sven Arvidsson
Modified: 2008-06-03 17:41 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
Truncated ISO image (100.00 KB, application/x-cd-image)
2008-01-15 14:55 UTC, Matt Kraai
  Details
use AC_SYS_LARGEFILE (967 bytes, patch)
2008-01-15 20:15 UTC, Christian Persch
none Details | Review

Description Sven Arvidsson 2008-01-12 20:29:25 UTC
[ From http://bugs.debian.org/460434 by Matt Kraai ]

"When I select a DVD ISO image in Nautilus and choose 'Open with "Movie Player (Xine)" from its context menu, Totem starts up and displays the following dialog:

 Totem could not play file 'file:///...'.

 There is no plugin to handle this movie.

It displays the same dialog if I first start Totem, then choose Movie -> Open... and select a DVD ISO.

I wish it would play the movie in both cases.

It is able to do so if I start it from the command line as follows:

 totem-xine dvd:///...

I have a similar problem using Totem with the GStreamer backend."


Unless I'm misreading bug 381261, this has been supported since 2.18?
Comment 1 Bastien Nocera 2008-01-13 10:09:59 UTC
What's the size of the ISO image? Which version of shared-mime-info does he have?
Comment 2 Matt Kraai 2008-01-14 18:08:28 UTC
(In reply to comment #1)
> What's the size of the ISO image? Which version of shared-mime-info does he
> have?

The ISO image is 4651546624 bytes.  After you asked, I found that it is able to play an ISO that's 2140274688 bytes.  I have shared-mime-info 0.22-2.
Comment 3 Bastien Nocera 2008-01-15 10:55:30 UTC
My guess is that the ISO image doesn't have a label, or the function we use to parse the title doesn't handle it.

What's the output of "file" on the ISO image? Does the file finish with ".iso" in its name? Could you please attach the first 100kB of the file in this bug as well?

use "dd if=myfile.iso of=truncated-file.iso bs=512 count=200"
Comment 4 Matt Kraai 2008-01-15 14:55:22 UTC
Created attachment 102903 [details]
Truncated ISO image
Comment 5 Matt Kraai 2008-01-15 14:57:09 UTC
(In reply to comment #3)
> My guess is that the ISO image doesn't have a label, or the function we use to
> parse the title doesn't handle it.
> 
> What's the output of "file" on the ISO image?

$ file
PRIME_SUSPECT_2.ISO: UDF filesystem data (version 1.5) 'PRIME_SUSPECT_2                '

>                                               Does the file finish with ".iso"
> in its name?

It finishes with ".ISO".  The behavior is unchanged if I lowercase the extension.

>              Could you please attach the first 100kB of the file in this bug as
> well?
> 
> use "dd if=myfile.iso of=truncated-file.iso bs=512 count=200"

Done above.
Comment 6 Bastien Nocera 2008-01-15 17:10:58 UTC
./test-parser file:///home/hadess/Desktop/Downloads/foo.iso 

###################### parsing ################

added URI 'vcd:///home/hadess/Desktop/Downloads/foo.iso'
        title = 'PRIME_SUSPECT_2'

Which is correct given that the file is < 700 megs. If it was over 700 megs, if would read:
added URI 'dvd:///home/hadess/Desktop/Downloads/foo.iso'
        title = 'PRIME_SUSPECT_2'

Best bet is to try with an updated Totem (2.20.3 is the latest stable release), and if that fails, I'd need the output of test-parser from the src/ dir on that file.
Comment 7 Matt Kraai 2008-01-15 18:40:22 UTC
(In reply to comment #6)
> Best bet is to try with an updated Totem (2.20.3 is the latest stable release),
> and if that fails, I'd need the output of test-parser from the src/ dir on that
> file.

I am (and have been) using Totem 2.20.3 (via Debian's totem-xine 2.20.3-1 package).  Here's the output of test-parser on the original ISO:

 ./test-parser file:///tmp/PRIME_SUSPECT_2.ISO 
 
 ###################### parsing ################
 
 url 'file:///tmp/PRIME_SUSPECT_2.ISO' unhandled
Comment 8 Matt Kraai 2008-01-15 19:45:42 UTC
(In reply to comment #7)
> I am (and have been) using Totem 2.20.3 (via Debian's totem-xine 2.20.3-1
> package).  Here's the output of test-parser on the original ISO:
> 
>  ./test-parser file:///tmp/PRIME_SUSPECT_2.ISO 
> 
>  ###################### parsing ################
> 
>  url 'file:///tmp/PRIME_SUSPECT_2.ISO' unhandled

The fopen in totem_pl_parser_iso_get_title fails with EFBIG, presumably because the file is bigger than 2^31.  If I replace fopen with fopen64, it displays the following instead:

 ###################### parsing ################
 
 added URI 'dvd:///tmp/PRIME_SUSPECT_2.ISO'
 	title = 'PRIME_SUSPECT_2'
Comment 9 Bastien Nocera 2008-01-15 19:55:18 UTC
Arf, that'll teach me. I was using an x86-64 so didn't see the problem :)

2008-01-15  Bastien Nocera  <hadess@hadess.net>

        * plparse/Makefile.am: Use 64-bit versions of the
        functions when possible, fixes DVD ISO not being recognised
        (Closes: #509020)

Thanks for finding that one! Fixed in trunk and stable branch.
Comment 10 Christian Persch 2008-01-15 20:14:43 UTC
I don't think that's correct to directly and always add those defines to CPPFLAGS. Does AC_SYS_LARGEFILE not work ?
Comment 11 Christian Persch 2008-01-15 20:15:05 UTC
Created attachment 102934 [details] [review]
use AC_SYS_LARGEFILE
Comment 12 Bastien Nocera 2008-01-15 20:21:29 UTC
(In reply to comment #10)
> I don't think that's correct to directly and always add those defines to
> CPPFLAGS. Does AC_SYS_LARGEFILE not work ?

gnome-vfs has done it for at least 6 or 7 years, so it's probably ok :)
Comment 13 Philip Withnall 2008-06-03 17:41:05 UTC
Mass-move from totem to totem-pl-parser. You can remove all messages by searching for this comment.