search

cheery / node-udev

Bindings to libudev
35 stars 31 forks source link

Reimplementation in N-API #33

Closed cheery closed 3 years ago

cheery commented 4 years ago

As a response to discussion on the issue #32, we examine the possibility to reimplement the library with N-API. The motivation to do this change is that this interface is purported to be ABI-stable.

The secondary objective here is to also provide WebIDL specifications and TypeScript headers for the library, to allow easier interop from numerous languages that compile to Js.

This change is allowed to be backwards-breaking, and we will bump up the version number by major. Still the library users are considered. We will document why the interface changes, for each function. We also attempt to keep the new library useful for the same things the previous library was useful for.

Resources for the project:

  1. N-API documentation. Outright it seems quite good, easy to comprehend and comprehensive documentation for the library.
  2. WebIDL spec.
  3. WebIDL wikipedia entry.
  4. WebIDL binding generator an experimental tool to generate bindings from the WebIDL specs. I'm planning to examine this one and see how well it fares.

A new branch (v1) was created for the work toward the N-API bindings. Help is welcome and we will work on this together.

GaikwadPratik commented 3 years ago

@cheery,

Thank you for taking initiative to create this ticket. I know all of us are busy. Just wanted to check if there is any progress on this since node v10 is EOL and is not being supported anymore

GaikwadPratik commented 3 years ago

@cheery ,

when are you going to release the package in npm?

cheery commented 3 years ago

I'm going to release npm later this week.

ti 27. heinäk. 2021 klo 14.43 Gaikwad Pratik @.***> kirjoitti:

@cheery https://github.com/cheery ,

when are you going to release the package in nom?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cheery/node-udev/issues/33#issuecomment-887442528, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJIXPEODOCJXL5QKPQXKU3TZ2LXLANCNFSM4RE4DDPA .

GaikwadPratik commented 3 years ago

@cheery,

any idea when would you be releasing it in nom?

cheery commented 3 years ago

I did run npm publish during the weekend. Anything elsewhere that is needed?

ti 10. elok. 2021 klo 18.03 Gaikwad Pratik @.***> kirjoitti:

@cheery https://github.com/cheery,

any idea when would you be releasing it in nom?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cheery/node-udev/issues/33#issuecomment-896107761, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJIXPFXIBJ7XGWFMQ7K53TT4E5VVANCNFSM4RE4DDPA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

GaikwadPratik commented 3 years ago

nope. Thanks!!!