browser-serialport icon indicating copy to clipboard operation
browser-serialport copied to clipboard

Merge with node-serialport via bindings

Open reconbot opened this issue 8 years ago • 8 comments

Hi Folks, I'm working towards getting our c++ bindings isolated enough that we could switch out serialport implementations in an easier way. This could include the chrome serialport object. I think this is something to talk about because I'm going to be releasing some breaking changes to serialport to improve it's api and it's silly to implement everything twice. I'm thinking maybe we could merge projects.

I've opened https://github.com/voodootikigod/node-serialport/issues/739 to explore the option.

reconbot avatar Apr 07 '16 18:04 reconbot

Awesome idea @reconbot! Let me know if I can help.

garrows avatar Apr 07 '16 22:04 garrows

yes! you very much can - I don't have a roadmap together yet, but the idea, is a minor bug release (With a major amount of bugs fixed), and then a major api change release, and then somewhere in there separating out and cleaning up the bindings. We could work chrome.serial in after the bug release I think.

reconbot avatar Apr 08 '16 01:04 reconbot

So I've got 4.0 in release candidate status. We can either bring those changes here or work to merge sooner than later.

The upgrade guide talks about the changes directly.

reconbot avatar Jun 17 '16 14:06 reconbot

This would also allow for future support of the proposed WebSerial API.

djipco avatar Nov 02 '16 11:11 djipco

It will be really nice to have a unified serial api for Javascript

weiway avatar Nov 03 '16 23:11 weiway

I've almost completed a spike on a chrome bindings object. I'm a lost on how to use the tests with it, but I'm almost there.

https://github.com/EmergingTechnologyAdvisors/node-serialport/compare/web

reconbot avatar Jan 15 '17 06:01 reconbot

I just want to poke this issue, the aforementioned branch is way out of date. However SerialPort 5 is about to be released. A binding interface for this would now be pretty strait forward to make but hard to test in a CI.

The project as a whole is not something I can take on at this time. However if anyone wants to give it a try I'd be available to support their efforts!

reconbot avatar Jul 21 '17 15:07 reconbot

@reconbot thanks for the bump. I'm planning on reviving this repo next month and I'd love to do the merge.

garrows avatar Jul 23 '17 22:07 garrows