-
Notifications
You must be signed in to change notification settings - Fork 119
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
SyntaxError: In src/server/server.coffee, reserved word "static" on line 5 #12
Comments
It looks like ce76984 fixed this issue. Try pulling and giving it another go. |
I suspected that change may have fixed it - sorry I'm not well versed in node/npm - what's the process for installing the latest from Git? edit - nevermind, figured out the instructions from this other issue comment: I now have it running, though getting the 'subscribe' error mentioned in that discussion. Are the changes in your mpd git branch being migrated into the main mpd branch, and/or will those messages be muted a bit in a future release for users of the stable mpd release? |
Ok. This is like the 4th time people have run into the "subscribe" error. We need to make the readme clearer. @superjoe30 The readme currently points to Also, I'm putting notes in the readme now that alert people that we depend on new features of mpd. |
There used to be an error in the master of git://git.musicpd.org/master/mpd.git but there isn't one now. You can compile from either. I should just release another version of groovebasin which will solve the "static" problem. |
I released package version 0.0.6, should solve this problem. |
Thanks! The readme was indeed a bit unclear about a few of these points. The issue is the first line says it depends on the version from source (now clearer by stating .17.x), but then the second line (Line 53) states that certain features are only available if you compile from source - implying that .16.x should still work, just without all features. And aside from having to click 'ok' to get past the subscribe error it did seem to work fine with .16.x. I didn't get a chance to beat on it that much just yet, but the first impressions have been great. |
Thanks for the feedback. I'm pretty sure we don't support interfacing with mpd 0.16. If it seems to work a little, it's a coincidence. I've updated the readme again in light of your suggestions. |
Hi, trying to install groovebasin on Puppy Linux, I built node and npm from scratch on the system. Getting the error in the issue title when I try to install. I'm not sure if this is a legitimate bug or if I'm missing some key dependency.
Complete install log:
The text was updated successfully, but these errors were encountered: