Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove mpris from build & prevent mpris from firing functions#877

Merged
weblancaster merged 3 commits intoSoundnode:masterfrom
jakejarrett:mpris
Sep 24, 2016
Merged

Remove mpris from build & prevent mpris from firing functions#877
weblancaster merged 3 commits intoSoundnode:masterfrom
jakejarrett:mpris

Conversation

@jakejarrett
Copy link
Member

Removes mpris from the build process & will check if mprisService is truthy (It returns false if its not supported so the if statements will not be truthy currently)

This prevents the app from throwing exceptions when mpris functions are not defined.
Going to approach mpris support without building modules
This prevents npm install from failing / causing issues
@jakejarrett
Copy link
Member Author

@weblancaster This should revert all mpris functionality (making linux builds work if built outside of a linux environment)

@weblancaster
Copy link
Member

awesome @jakejarrett

@weblancaster weblancaster merged commit 263037d into Soundnode:master Sep 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants