Skip to content
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

Install on darwin #12

Closed
reconbot opened this issue Sep 16, 2013 · 4 comments · Fixed by #13
Closed

Install on darwin #12

reconbot opened this issue Sep 16, 2013 · 4 comments · Fixed by #13

Comments

@reconbot
Copy link
Contributor

Hi,
I'm using onoff for FirmataPi and have hit a snag. I've been developing on osx, running tests on osx and linux, and have "production" on my raspberry pi. I know onoff can't work on anything but linux but in my tests I'm mocking out onoff as I don't need to test its functionality, just that I'm calling it correctly.

So here's my question, how can I install it on other platforms?

@fivdi
Copy link
Owner

fivdi commented Sep 16, 2013

Well, at the moment you can't, but removing these three lines and wrapping all the code in gpiowatcher.cc with #ifdef linux ... #endif to avoid compile errors might do it. I haven't got osx so I can't try it out. If you can try it out, verify that it works, and make a pull request I'll merge it in.

@reconbot
Copy link
Contributor Author

Taking the OS out of the package.json was enough to get it to install on
osx. I haven't tried using it, but I can require it.


Francis Gulotta
wizard@roborooter.com

On Mon, Sep 16, 2013 at 4:02 PM, fivdi notifications@github.com wrote:

Well, at the moment you can't, but removing these three lineshttps://github.com/fivdi/onoff/blob/master/package.json#L21-L23and wrapping all the code in
gpiowatcher.cchttps://github.com/fivdi/onoff/blob/master/src/gpiowatcher.ccwith #ifdef linux ... #endif to avoid compile errors might do it. I
haven't got osx so I can't try it out. If you can try it out, verify that
it works, and make a pull request I'll merge it in.


Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-24539841
.

@fivdi
Copy link
Owner

fivdi commented Sep 17, 2013

v0.1.7 with the modifications is now available on npm. Good luck with FirmataPi.

@reconbot
Copy link
Contributor Author

Thanks!
On Sep 17, 2013 3:04 PM, "fivdi" notifications@github.com wrote:

v0.1.7 with the modifications is now available on npmhttps://npmjs.org/package/onoff.
Good luck with FirmataPi https://github.com/firmata/firmata-pi.


Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-24614080
.

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

Successfully merging a pull request may close this issue.

2 participants