New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MPDroid crashes immediately on start up since mopidy update #216

Closed
scott2907 opened this Issue Oct 28, 2012 · 9 comments

Comments

3 participants
@scott2907

scott2907 commented Oct 28, 2012

No logs left and no errors reported by mopidy - stack trace on android shows playlist-refresh then playlist-changed then status-monitor before crash. syslog on android shows BINDER failed. No other users report any issue with MPDroid.

@jodal

This comment has been minimized.

Member

jodal commented Oct 28, 2012

I'm not able to test MPDroid myself, as it crashes directly after connecting to Mopidy on my phone running Android 4.1.2. The crash is caused by an java.lang.IllegalArgumentException in java.util.AbstractList.subList() (line 736). The last MPD command that is issued by MPDroid before it crashes is plchanges "-1", to which Mopidy responds with just OK.

I've reported this to the MPDroid developer some weeks ago, using the regular Android crash report mechanism.

Do you experience the same error, or something else?

@jodal

This comment has been minimized.

Member

jodal commented Oct 28, 2012

For the record, there was no relevant changes in Mopidy 0.8.0. What version did you upgrade from and to?

@scott2907

This comment has been minimized.

scott2907 commented Oct 28, 2012

Yes that sounds like the same error. I have also reported the issue via the Android report.
I'm not sure what version I upgraded from tbh, but if there are no relevant changes in 0.8.0 then I assume that it's MPDroid. I upgraded both simultaneously, and presumed Mopidy caused the issue since I assumed basic connections would have been tested well with several server types on MPDroid...

@jodal

This comment has been minimized.

Member

jodal commented Oct 28, 2012

I would love to get MPDroid to work with Mopidy again. It clearly looks as the best MPD client on Android these days.

@abarisain I guess you are the MPDroid developer? Do you have anything to add here? Is MPDroid or Mopidy to blame?

@abarisain

This comment has been minimized.

Contributor

abarisain commented Oct 28, 2012

Hey ! Glad you mentionned me, I would have never seen this otherwise.

I didn't test MPDroid with Mopidy before the release, therefore I'd say that I'm the one to blame. Somebody told me that mpdroid broke with mopidy right after my new release, and I was too busy fixing MPDroid's crashes with a vanilla MPD server.

I got a new linux box so I can try mopidy (it's really hard to install on a mac), and as it works with my other MPD clients, I'll focus my efforts on this now.

Tl;dr I think that nothing is wrong with Mopidy and that it's MPDroid that's broken.

Note : Maybe that MPDroid doesn't quite answer like MPD, but MPDroid should gracefully handle the error and just ignore it.

@jodal

This comment has been minimized.

Member

jodal commented Oct 28, 2012

I'm sure there are lots of things we can improve on the Mopidy side as well, to get Mopidy and MPDroid to work better together. This will be easier as soon as we can get our apps to talk to each other for extended amounts of time :-)

Feel free to contact us about stuff that doesn't work in MPDroid when using Mopidy.

@jodal

This comment has been minimized.

Member

jodal commented Oct 28, 2012

be5759e fixed the Mopidy side of this.

@abarisain

This comment has been minimized.

Contributor

abarisain commented Oct 28, 2012

I also fixed MPDroid's side (for retrocompatibility with Mopidy 0.8). Volume will be broken until a new release of mopidy happens.

For other bugs concerning MPDroid and mopidy, please open bugs on MPDroid's bugtracker too (or mention me, like jodal did). This way I get notified of the bugs. I'll try my best to make the next MPDroid version Mopidy compatible :)

@scott2907

This comment has been minimized.

scott2907 commented Oct 28, 2012

thanks chaps. I'll notify any further problems on both sites.

@jodal jodal closed this Oct 29, 2012

@ghost ghost assigned jodal Nov 1, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment