Ubuntu
gnome-media package

Playback plays currently working file in gnome-sound-recorder

Bug #1029092 reported by James M. Leddy
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnome-media (Ubuntu)
Fix Released
Low
Unassigned
Precise
Fix Released
Low
Unassigned

Bug Description

Impact:
As expressed in bug 1011436, we have a problem currently in ubuntu where if we record another file using g-s-r, it plays the first file that was recorded ever. Specifically:
I upgraded gnome-media to 3.4.0-0ubuntu4 and had a test, now the second record can play, but still has something wrong.

Test Case:

Round 1

* Open Sound Recorder, click "Record sound" button, say "This is the first record", then click "Stop sound"
* Click "Play sound", you can hear "This is the first record"

Round 2

* Without clicking save button, click "Record sound" again, it will prompt you to save the sound, choose "Save As" to save it to "1.ogg"
* Now new record is started, say "This is the second record", then click "Stop sound"
* Click "Play sound" again, but you still hear "This is the first record", not the expected "This is the second record"

Round 3

* Click "Record sound" for the third time, after prompt save the second record to "2.ogg"
* Record the sound "This is the third record", then click "Stop sound"
* Click "Play sound" and you will hear "This is the second record", not "This is the third record"

Regression potential:
when doing several recording the buffers could be wrongly handled, check that when previewing and saving records you get the correct content
....

So the play issue is fixed, but it will always play the last saved record instead of the current record after the second time.

Please help to reproduce.

See original description
Revision history for this message
James M. Leddy (jm-leddy) wrote :
Revision history for this message
James M. Leddy (jm-leddy) wrote :

    - Reset the working file at the start of a new recording,
      fixes playback so that it plays the working file and not
      an older file.

This is the commit message that I used.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "git_reset_working_file_for_record.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

Sebastien Bacher (seb128)
Changed in gnome-media (Ubuntu):
status: Triaged → Fix Committed
Changed in gnome-media (Ubuntu Precise):
status: New → In Progress
importance: Undecided → Low
Changed in gnome-media (Ubuntu):
importance: Medium → Low
description: updated
Revision history for this message
Clint Byrum (clint-fewbar) wrote : Please test proposed package

Hello James, or anyone else affected,

Accepted gnome-media into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/gnome-media/3.4.0-0ubuntu3.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

description: updated
Changed in gnome-media (Ubuntu Precise):
status: In Progress → Fix Committed
tags: added: verification-needed
Alan Pope 🍺🐧🐱 🦄 (popey)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-media - 3.4.0-0ubuntu3.1

---------------
gnome-media (3.4.0-0ubuntu3.1) precise-proposed; urgency=low

  * debian/patches/git_play_after_second_record.patch:
    - updated to fix the preview button playing the wrong buffer when
      doing a second recording, thanks James M Leddy for the update
      (lp: #1029092)
 -- Sebastien Bacher <email address hidden> Mon, 03 Sep 2012 15:40:03 +0200

Changed in gnome-media (Ubuntu Precise):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-media - 3.4.0-1ubuntu1

---------------
gnome-media (3.4.0-1ubuntu1) raring; urgency=low

  * Rebase on Debian unstable, remaining Ubuntu changes:
  * debian/control:
    - updated the dpkg build-depends, use "Pre-Depends: ${misc:Pre-Depends}"
  * debian/gnome-media.maintscript:
    - clean the deprecated autostart for the sound control
  * debian/patches/02_better_record_titles.patch:
    - "Use names based on date instead of wort "Untitled"."
  * debian/patches/03_set_pulse_role.patch:
    - set the pulse media role for the indicator sound controls
  * debian/patches/git_play_after_second_record.patch:
    - "Properly reinitialize when starting a new recording"

  [ Sebastien Bacher ]
  * debian/patches/git_play_after_second_record.patch:
    - updated to fix the preview button playing the wrong buffer when
      doing a second recording, thanks James M Leddy for the update
      (lp: #1029092)
 -- Robert Ancell <email address hidden> Thu, 15 Nov 2012 15:09:28 +1300

Changed in gnome-media (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.