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

doesn't work with node-ffi v1.0 API #16

Closed
justinfreitag-zz opened this issue Jun 27, 2012 · 7 comments
Closed

doesn't work with node-ffi v1.0 API #16

justinfreitag-zz opened this issue Jun 27, 2012 · 7 comments

Comments

@justinfreitag-zz
Copy link

the dependency node-ffi 0.5.4 won't build for me on node 0.8,

also, when I manually switch node-ffi 0.5.4 for 1.0.2, NodObjc doesn't pass its tests, failing with:

core/blocks.js
/Users/justin/dev/NodObjC/lib/core.js:18
, uintptr_t = ffi.Bindings.POINTER_SIZE == 8 ? 'uint64' : 'uint32'
^
TypeError: Cannot read property 'POINTER_SIZE' of undefined
at Object. (/Users/justin/dev/NodObjC/lib/core.js:18:29)

@TooTallNate
Copy link
Owner

Yes, I'm working on getting NodObjC updated to the node-ffi v1.0 API, but it's a big refactor. You can track the progress in the ref branch./

@justinfreitag-zz
Copy link
Author

thanks, appreciated :)

@TooTallNate
Copy link
Owner

Until then you will have to try an older version of Node, like v0.6.19.

@TooTallNate
Copy link
Owner

@justinfreitag I actually back ported the necessary fixes for node-ffi v0.5.x, so you should be able to use the current version of NodObjC on node v0.8.x still.

@justinfreitag-zz
Copy link
Author

great, thanks.

On Sat, Jul 7, 2012 at 12:05 PM, Nathan Rajlich <
reply@reply.github.com

wrote:

@justinfreitag I actually back ported the necessary fixes for node-ffi
v0.5.x, so you should be able to use the current version of NodObjC on node
v0.8.x still.


Reply to this email directly or view it on GitHub:
#16 (comment)

@trevorlinton
Copy link
Collaborator

Fixed on https://github.com/TooTallNate/NodObjC/tree/0.1.0, waiting for test/docs/merge

@trevorlinton
Copy link
Collaborator

Fixed and merged.

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

No branches or pull requests

3 participants