Bug 1322498

Summary: [abrt] mpdris2: mpDris2:753:_fetch_object:TypeError: _writecommand() takes at least 2 arguments (1 given)
Product: [Fedora] Fedora Reporter: Vadim Raskhozhev <iamdexpl>
Component: mpdris2Assignee: Ankur Sinha (FranciscoD) <sanjay.ankur>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: fedora, iamdexpl, sanjay.ankur
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
URL: https://retrace.fedoraproject.org/faf/reports/bthash/f9b51fc5aa7280f3911eabb4e8ea8a06ee7452a7
Whiteboard: abrt_hash:3ad24133a3ce0de9df5e7448e04008c4aaf5a6c9;
Fixed In Version: mpdris2-0.7-2.20160422gita3af302.fc23 mpdris2-0.7-2.20160422gita3af302.fc24 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-02 18:53:39 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
File: backtrace
none
File: environ none

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.