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 364311 - telling it to open the playing file doesn't do anything
telling it to open the playing file doesn't do anything
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Movie player
2.16.x
Other All
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2006-10-23 00:03 UTC by Sam Morris
Modified: 2007-08-26 18:13 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
strace of parent process (22.74 KB, text/plain)
2006-10-23 11:18 UTC, Sam Morris
Details
strace of launcher totem process (84.80 KB, text/plain)
2006-10-23 11:19 UTC, Sam Morris
Details

Description Sam Morris 2006-10-23 00:03:56 UTC
Please describe the problem:
If Totem is idle then double-clicking on a media file (or launching another instance of Totem to play another file) does nothing.

Steps to reproduce:
1. Launch Totem and play a media file
2. Disable 'Repeat Mode'
3. Let the file finish playing
4. In a terminal, run 'totem some-other-file.mp3'


Actual results:
Nothing

Expected results:
Totem should play some-other-file.mp3

Does this happen every time?
Yes

Other information:
Comment 1 Bastien Nocera 2006-10-23 10:58:44 UTC
Works fine for me. Any errors on the command-line? Could you get an strace of the original process after the first file has finished playing, and of the other instance of totem launching another file?

Do you get a file called /tmp/totem.<username>.<randomnumber> ?
Comment 2 Sam Morris 2006-10-23 11:14:06 UTC
Nothing on the command line and no files in /tmp.

Strace output to follow...
Comment 3 Sam Morris 2006-10-23 11:18:39 UTC
Created attachment 75234 [details]
strace of parent process
Comment 4 Sam Morris 2006-10-23 11:19:32 UTC
Created attachment 75235 [details]
strace of launcher totem process

Oh, I do have /tmp/user/1000/totem.sam.3686746664 which appears to be the socket that Totem uses to communicate with the original process
Comment 5 Bastien Nocera 2006-10-23 15:23:49 UTC
Where did you get your Totem from? That's certainly not the file that libbacon uses, your distribution is probably carrying a (broken) patch for that.
Comment 6 Sam Morris 2006-10-23 16:29:06 UTC
You mean the socket file? It probably lives there because I'm using pam-tmpdir. It creates /tmp/user/1000 (1000 being my uid) and sets the TMP and TMPDIR environment variables to point to that directory.

I just did a fresh test with TMP and TMPDIR unset, and have the same problem.
Comment 7 Bastien Nocera 2006-10-23 16:37:45 UTC
From the parent process:
lstat64("/home/sam/Final_Fantasy_10_How_Much_Longer_OC_ReMix.mp3", {st_mode=S_IFREG|0644, st_size=3420160, ...}) = 0

It seems to be working fine, until it tries to do some fu on the recent files. Does Totem hang, do nothing, whatever, when you launch the other command?
Does it add it to the recent-files? Does it remove the existing playlist? Does it add it to a new playlist?
Comment 8 Sam Morris 2006-10-23 16:55:39 UTC
Right, what I didn't realise when I filed this bug was that Totem only ignores the request to play a new file when it is the _same_ file. So I guess that Totem checks that the requested file name is the same as the one that is already in the playlist, and if so does nothing.

Perhaps it should only do the check if it currently playing (not stopped or paused)?
Comment 9 Bastien Nocera 2007-01-26 14:45:40 UTC
(In reply to comment #8)
> Right, what I didn't realise when I filed this bug was that Totem only ignores
> the request to play a new file when it is the _same_ file.

Lies in the summary then ;)
Comment 10 Bastien Nocera 2007-01-26 14:48:08 UTC
Added a note in totem.c's playlist_changed_cb():
        //FIXME we shouldn't compare the 2 URLs
        // http://bugzilla.gnome.org/show_bug.cgi?id=364311

This obviously needs proper testing...
Comment 11 Bastien Nocera 2007-08-26 18:13:05 UTC
2007-08-26  Bastien Nocera  <hadess@hadess.net>

        * src/totem-playlist.c: (treeview_row_changed),
        (totem_playlist_get_playing):
        * src/totem-playlist.h:
        * src/totem.c: (playlist_changed_cb): Fix activation (adding to
        the playlist, switching opened files, etc.) when 2 of the items in
        the playlist have the same MRL, see bug for details (Closes: #364311)