Slider jumps around at the end of songs

thanks for the report, i don't see that issue here, could you get a rhythmbox log by running: rhythmbox --debug &> rhythmbox-debug.txt ; reproduce the issue and attach that resulting log file to the report? Thanks in advance.

This is a log of me opening rhythmbox. Playing a file, seeking to near the end, waiting for it to end, and then it does the jump around bit and goes to the next track.

I plan on having a poke around in the source later, I've fixed a similar bug in rhythmbox before, so I'm marginally familiar with the code.

This is a patch (against the upstream git master... I thought this had been forwarded upstream, but I was confusing this with a different bug). The main issue is that the playbin2 gstreamer element doesn't actually tell you when it finishes one stream and begins the next. This patch works around that for the moment by assuming that when the stream position goes backwards, we have moved to the next stream.

So, now instead of having the position slider jump around to different spots, it now either works correctly, or jumps to 0:00 of the next track in the time period where it would previously have been jumping around.

The previous behavior had the position slider moving to the elapsed time for the current stream, but scaled according to the length of the next stream, so it was often somewhere in the middle.

I've spoken to a gstreamer guy on irc, and there are plans to fix playbin2 in the future.

Oh, I forgot to mention, if someone other than me can actually reproduce this (it is broken on my amd64 and x86 machines), then forwarding this upstream seems like the correct thing to do (since I can also reproduce it in upstream git). I'm not sure if there is some launchpad magic for doing this properly though.

Thank you for your bug report, do you have an upstream bug about the issue? The change seems complicated for a stable update, do you know if upstream plans to fix that issue in the lucid cycle?

It is now in gnome bugzilla at:

https://bugzilla.gnome.org/show_bug.cgi?id=601524

Sorry, I seem to have messed up slightly. As this seems to be marked as a gstreamer bug at the moment. I think the gstreamer issue (lack of "finished" signal to go along with "nearly-finished") should be in a separate bug. I'm not quite sure how to fix it though.

is that still an issue in lucid? could you ping upstream to get the patch commited?

I don't have the bandwidth to be able to test it in lucid yet, but I suspect it will still be an issue.

The patch I gave was a workaround though (it was a slightly nicer hack than what was already there).

There is now a fix in gstreamer which makes a non-hackish fix possible:

https://bugzilla.gnome.org/show_bug.cgi?id=584987

So, if it hasn't been done yet, then someone should make a patch which uses this (I tried to make my workaround such that once this bit was done, it would be trivial to drop in the non-hackish version, so it might be an ok starting point).

That's fixed upstream now.

Reassigning to match the product of the upstream bug.

This bug was fixed in the package rhythmbox - 0.12.6git20100218-0ubuntu1

---------------
rhythmbox (0.12.6git20100218-0ubuntu1) lucid; urgency=low

  * Updated to a git version to give some testing to the upstream code before
    the next tarball, the udate fixes those issues:
    - "The play button's tool tip "Stop playback" is misleading" (lp: #97442)
    - "double-click the cover art to open it" (lp: #178517)
    - "Rhythmbox crashes when opening song info box" (lp: #399052)
    - "Slider jumps around at the end of songs" (lp: #474139)
    - "rhythmbox freezes randomly" (lp: #487932)
    - "CBC Radio 1 & 2 Live Feeds need updating" (lp: #521474)
  * debian/control.in:
    - updated the pygobject and gtk requirements
  * debian/patches/81_not_using_localmodlibs.patch,
    debian/patches/91_git_change_use_new_totemplparser.patch:
    - dropped, the changes are in the new version
  * debian/patches/82_rhythmbox-indicators.patch:
    - update to the new version by Jan Arne Petersen
  * debian/rhythmbox-plugins.install:
    - updated to list replaygain and sendto
 -- Sebastien Bacher <email address hidden> Thu, 18 Feb 2010 15:34:50 +0100