-
Notifications
You must be signed in to change notification settings - Fork 70
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
No introspectable error in 0.2.22 #159
Comments
not sure whether it's related to #150 could you post sample code here? thanks! |
Alternately, could it be an issue because of the API changes that I introduced? |
Hmm, yes we are using the Dbus constructor, which I see is now deprecated. When was that introduced? |
The DBus constructor still works in v0.2.22, should be other change related. |
From 0.2.19 -> 0.2.22, the major API that changed, that is probably relevant to you, is f6d7370 But that doesn't explain why you're getting a "Not Introspectable" error. The DBus constructor was deprecated in 1.0.0. You can see all of those changes at Migrating. But like Shouqun said, that didn't land until 1.0.0, so is probably not your issue. Can you narrow it down by trying your code with 0.2.21? |
Ah sorry I didn't see you'd bumped to v1 👍 Cool, we'll do some more digging and get back to you guys. |
For API compatibility, I just published 0.2.23 from branch https://github.com/Shouqun/node-dbus/tree/0.2.23 which is API compatible with 0.2.19 (without change f6d7370 and related code). and for API 1.x, I'll publish 1.0.0 later. @bryanburgers |
@craig-mulligan I can't write CoffeeScript, nor do I have However, I tried to rewrite your dbus-promise using the new API, and it seems to work fine against Just playing around at this point to see what I can figure out. |
Edit: This was trivial. The I am also running Node v6.x.x, with node-dbus v1.0.0 and getting the var DBus = require('../');
var bus = DBus.getBus('system');
bus.getInterface('fi.w1.wpa_supplicant1', '/fi/w1/wpa_supplicant1', 'fi.w1.wpa_supplicant1.Interface', function(err, iface) {
if( err ) {
console.error( err )
}
}); Any ideas? |
Hi i have the same error, how can i fix that.
` and this is the client code
|
Hey guys, I found the cause of this. You can only load the system bus once const bus1 = DBus.getBus("system");
const bus2 = DBus.getBus("system");
bus1.getInterface(); // will work
bus2.getInterface(); // will throw that error posted this answer here too |
Is there an intend to fix this? |
node v: 6
v: 0.2.22
Reverted to
0.2.19
and it works.The text was updated successfully, but these errors were encountered: