upnp plugin is broken

Bug #198466 reported by Will Newton
2
Affects Status Importance Assigned to Milestone
rhythmbox (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: rhythmbox

The upnp plugin in current hardy (0.11.4-0ubuntu6) is broken. If you try and use it you'll get undefined variables in the Python source of the plugin. These errors are fixed in GNOME SVN - the backport is trivial. It would be nice to have it fixed for the hardy release.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. Enabling it seems to work correctly on hardy, could you describe what error you get and what change is required?

Changed in rhythmbox:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
Will Newton (will-newton) wrote : Re: [Bug 198466] Re: upnp plugin is broken

Enabling it does not fail - but when you press play you should see
Python backtraces in ~/.xsession-errors.

On Sat, Mar 8, 2008 at 3:19 PM, Sebastien Bacher <email address hidden> wrote:
> Thank you for your bug report. Enabling it seems to work correctly on
> hardy, could you describe what error you get and what change is
> required?
>
> ** Changed in: rhythmbox (Ubuntu)
> Importance: Undecided => Low
> Assignee: (unassigned) => Ubuntu Desktop Bugs (desktop-bugs)
> Status: New => Incomplete
>
>
>
> --
> upnp plugin is broken
> https://bugs.launchpad.net/bugs/198466
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for taking the time to report this bug. Unfortunately we can't fix it, because your description didn't include enough information. You may find it helpful to read "How to report bugs effectively" <http://www.chiark.greenend.org.uk/~sgtatham/bugs.html>. We'd be grateful if you would then provide a more complete description of the problem.

 We have instructions on debugging some types of problems. <http://wiki.ubuntu.com/DebuggingProcedures> Thanks!

Changed in rhythmbox:
status: Incomplete → Invalid
Revision history for this message
Sebastien Bacher (seb128) wrote :

closing the bug, having useful description is required since we get hundred bugs every week and having to ask several times to get the error message doesn't make the work easy, I've no device to try this one and without details there is no way we will be able to work on the issue

Revision history for this message
Will Newton (will-newton) wrote :

I disagee with your interpretation. If you don't want to fix it, no
problem, I can do it myself, I only reported the bug as a courtesy to
Ubuntu.

On Sat, Mar 8, 2008 at 8:55 PM, Sebastien Bacher <email address hidden> wrote:
> closing the bug, having useful description is required since we get
> hundred bugs every week and having to ask several times to get the error
> message doesn't make the work easy, I've no device to try this one and
> without details there is no way we will be able to work on the issue
>
>
>
> --
> upnp plugin is broken
> https://bugs.launchpad.net/bugs/198466
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Sebastien Bacher (seb128) wrote :

you could also describe the error you get or the change required

Revision history for this message
Will Newton (will-newton) wrote :

The first error you should encounter on trying the upnp plugin:

Traceback (most recent call last):
  File "/usr/lib/rhythmbox/plugins/upnp_coherence/__init__.py", line
43, in activate
    if url:
UnboundLocalError: local variable 'url' referenced before assignment

There are at least two other trivial coding errors of this type in the
plugin. These are fixed in these upstream changes:

Fixes this and one other unbound variable error:

http://svn.gnome.org/viewvc/rhythmbox/trunk/plugins/coherence/upnp_coherence/__init__.py?r1=5486&r2=5537

Fixes declaration of state_variable_changed method:

http://svn.gnome.org/viewvc/rhythmbox/trunk/plugins/coherence/upnp_coherence/UpnpSource.py?r1=5257&r2=5563

Due to possible dependencies between other parts of these changes I
would suggest that it's probably not possible to backport these two
changes in isolation, but update the whole upnp_coherence plugin
directory.

On Sun, Mar 9, 2008 at 10:20 AM, Sebastien Bacher <email address hidden> wrote:
> you could also describe the error you get or the change required
>
>
>
> --
> upnp plugin is broken
> https://bugs.launchpad.net/bugs/198466
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Sebastien Bacher (seb128) wrote :

the bug has been fixed in the new version uploaded in hardy

Changed in rhythmbox:
status: Invalid → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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