Bug 527484

Summary:

CD burning broken in Rhythmbox

Product:

[Fedora] Fedora

Reporter:

jmccann

Component:

brasero

Assignee:

Xavier Lamien <lxtnow>

Status:

CLOSED RAWHIDE

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

medium

Docs Contact:

Priority:

low

Version:

rawhide

CC:

bnocera, cschalle, lxtnow, mclasen

Target Milestone:

---

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2009-10-07 09:19:45 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
test playlist	none

Description jmccann 2009-10-06 15:23:01 UTC

CD burning doesn't work in rawhide.

1. Make playlist
2. Click burn
3. nada

Comment 1 Matthias Clasen 2009-10-06 15:39:48 UTC

When I do this, I get 

(11:37:24) [0x12a5060] [source_burn] rb-disc-recorder-plugin.c:468: Burning playlist test
(11:37:24) [0x12a5060] [source_burn] rb-disc-recorder-plugin.c:472: Creating audio project
(11:37:24) [0x12a5060] [source_burn] rb-disc-recorder-plugin.c:484: Starting brasero


but brasero is not actually started, and nothing happens after that

Comment 2 Bastien Nocera 2009-10-06 17:05:15 UTC

In my instance, I can see brasero getting launched:
brasero -r /tmp/brasero-tmp-project-OMU50U -x 94371844

Using this (it's slow enough that you can see it):
watch -n 1 "ps aux | grep brasero" > foo

And then brasero removes the file in question (which it probably doesn't know how to handle) and exits.

Worked fine with brasero 2.26.x from F11.

Comment 3 Bastien Nocera 2009-10-06 17:07:03 UTC

This line does nothing:
brasero -r /tmp/brasero-tmp-project-QD320U
But remove the file.

Comment 4 Bastien Nocera 2009-10-06 17:08:13 UTC

Created attachment 363860 [details]
test playlist

brasero /tmp/testfile
works
brasero -r /tmp/testfile
doesn't

My guess is that they're removing the file before reading it :)

Comment 5 Bastien Nocera 2009-10-06 17:09:24 UTC

Additional warnings when the files in the playlist don't exist come free of charge:

(brasero:1536): GLib-GIO-CRITICAL **: g_file_new_for_uri: assertion `uri != NULL' failed

(brasero:1536): GLib-GIO-CRITICAL **: g_file_query_info: assertion `G_IS_FILE (file)' failed

(brasero:1536): GLib-CRITICAL **: g_propagate_error: assertion `src != NULL' failed

(brasero:1536): GLib-GIO-CRITICAL **: g_file_get_uri: assertion `G_IS_FILE (file)' failed

(brasero:1536): GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

Comment 6 Matthias Clasen 2009-10-07 02:28:47 UTC

I've fixed this 'the wrong way' for now by making rhythmbox and totem pass uris. 
If brasero behaviour gets fixed to expect paths again, those patches will have to be backed out.

Comment 7 Bastien Nocera 2009-10-07 09:19:45 UTC

Fixed in 2.28.1-2 in F-13 and F-12, waiting on tagging.

I also requested for totem and rhythmbox to have their work-arounds removed.