Bug 1322498 - [abrt] mpdris2: mpDris2:753:_fetch_object:TypeError: _writecommand() takes at least 2 arguments (1 given)
Summary: [abrt] mpdris2: mpDris2:753:_fetch_object:TypeError: _writecommand() takes at...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: mpdris2
Version: 23
Hardware: x86_64
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Ankur Sinha (FranciscoD)
QA Contact: Fedora Extras Quality Assurance
URL: https://retrace.fedoraproject.org/faf...
Whiteboard: abrt_hash:3ad24133a3ce0de9df5e7448e04...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-30 15:06 UTC by Vadim Raskhozhev
Modified: 2016-05-07 11:59 UTC (History)
3 users (show)

Fixed In Version: mpdris2-0.7-2.20160422gita3af302.fc23 mpdris2-0.7-2.20160422gita3af302.fc24
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-05-02 18:53:39 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
File: backtrace (539 bytes, text/plain)
2016-03-30 15:06 UTC, Vadim Raskhozhev
no flags Details
File: environ (1.23 KB, text/plain)
2016-03-30 15:06 UTC, Vadim Raskhozhev
no flags Details

Description Vadim Raskhozhev 2016-03-30 15:06:20 UTC
Version-Release number of selected component:
mpdris2-0.7-1.fc23

Additional info:
reporter:       libreport-2.6.4
cmdline:        python /bin/mpDris2
executable:     /bin/mpDris2
kernel:         4.4.6-300.fc23.x86_64
runlevel:       N 5
type:           Python
uid:            1000

Truncated backtrace:
mpDris2:753:_fetch_object:TypeError: _writecommand() takes at least 2 arguments (1 given)

Traceback (most recent call last):
  File "/bin/mpDris2", line 426, in timer_callback
    was_idle = self.idle_leave()
  File "/bin/mpDris2", line 415, in idle_leave
    self._fetch_object()
  File "/bin/mpDris2", line 753, in _fetch_object
    return self.client._writecommand(*args)
TypeError: _writecommand() takes at least 2 arguments (1 given)

Local variables in innermost frame:
self: <__main__.MPDWrapper object at 0x7f523746ccd0>
args: ()

Comment 1 Vadim Raskhozhev 2016-03-30 15:06:24 UTC
Created attachment 1141784 [details]
File: backtrace

Comment 2 Vadim Raskhozhev 2016-03-30 15:06:26 UTC
Created attachment 1141785 [details]
File: environ

Comment 3 Ankur Sinha (FranciscoD) 2016-03-30 17:20:28 UTC
Thank you for the bugreport. Looking into this now.

Cheers,
Ankur

Comment 4 Ankur Sinha (FranciscoD) 2016-03-30 17:21:31 UTC
Any chance you have a set of steps that can reproduce the bug?

Comment 5 Vadim Raskhozhev 2016-03-30 17:32:18 UTC
Afraid I do not have such a set. The only thing I'm sure of is that this abrt alert appeared right after I logged in after reboot. Guess this won't help much, sorry.

Comment 6 Ankur Sinha (FranciscoD) 2016-03-31 09:27:42 UTC
(In reply to Vadim Raskhozhev from comment #5)
> Afraid I do not have such a set. The only thing I'm sure of is that this
> abrt alert appeared right after I logged in after reboot. Guess this won't
> help much, sorry.

No worries. I'll look into it. Cheers!

Comment 7 Sebastian Dyroff 2016-04-22 09:29:21 UTC
Hey,

I had to downgrade mpdris2 because of this. My media player indicator extension for gnome stopped working. Steps to reproduce on my machine are:

1. log in to gnome mpdris2 starts with the session
2. start mpd in the terminal
3a. either use the media player indicator buttons (nothing will happen, but the traceback is logged)
3b. or use  dbus-send --print-reply --dest=org.mpris.MediaPlayer2.mpd /org/mpris/MediaPlayer2 org.mpris.MediaPlayer2.Player.PlayPause to trigger the bug

Regards

Comment 8 Ankur Sinha (FranciscoD) 2016-04-22 10:23:31 UTC
Thanks for the info Sebastian. I'll look into it - can you please check with the latest commit of the media player extension? Both mpdris2 and the media player extension are written by the same upstream:

https://github.com/eonpatapon/gnome-shell-extensions-mediaplayer
https://github.com/eonpatapon/mpDris2

Cheers,
Ankur

Comment 9 Sebastian Dyroff 2016-04-22 10:32:37 UTC
Stays the same with the current master

Comment 10 Ankur Sinha (FranciscoD) 2016-04-22 12:25:38 UTC
Ah- fixed after release here. I'll push an update ASAP.

https://github.com/eonpatapon/mpDris2/commit/a3af3028650812898e201a6f7718126ac43a57a6

Comment 11 Fedora Update System 2016-04-22 13:29:00 UTC
mpdris2-0.7-2.20160422gita3af302.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-c15b56ad9c

Comment 12 Fedora Update System 2016-04-22 13:29:07 UTC
mpdris2-0.7-2.20160422gita3af302.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-743ff6ab06

Comment 13 Fedora Update System 2016-04-24 01:24:53 UTC
mpdris2-0.7-2.20160422gita3af302.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-743ff6ab06

Comment 14 Fedora Update System 2016-04-24 06:23:07 UTC
mpdris2-0.7-2.20160422gita3af302.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-c15b56ad9c

Comment 15 Sebastian Dyroff 2016-04-27 07:49:05 UTC
Thanks, works great

Comment 16 Fedora Update System 2016-05-02 18:53:36 UTC
mpdris2-0.7-2.20160422gita3af302.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.

Comment 17 Fedora Update System 2016-05-07 11:59:08 UTC
mpdris2-0.7-2.20160422gita3af302.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.