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

[serial-transport] [WIP] jSerialComm - Initial Contribution #4081

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

mvalla
Copy link
Contributor

@mvalla mvalla commented Feb 5, 2024

Alternative Serial Port Provider based on https://github.com/Fazecast/jSerialComm.

From website: jSerialComm is a Java library designed to provide a platform-independent way to access standard serial ports without requiring external libraries, native code, or any other tools. It is meant as an alternative to RxTx and the (deprecated) Java Communications API, with increased ease-of-use, an enhanced support for timeouts, and the ability to open multiple ports simultaneously.

Regarding jSerialComm :

  • it supports Apple Silicon
  • there is already another OH binding using it: ProteusEcoMeter (not using OH Serial Transport).

This is a PoC of a new transport based on jSerialComm.
Few features still have to be sorted out as the mapping with OH serial tranport / RxTx is not always straight forward.

Related to #3982

@mvalla mvalla requested a review from a team as a code owner February 5, 2024 16:08
@mvalla mvalla marked this pull request as draft February 16, 2024 23:45
<parent>
<groupId>org.openhab.core.bundles</groupId>
<artifactId>org.openhab.core.reactor.bundles</artifactId>
<version>4.2.0-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<version>4.2.0-SNAPSHOT</version>
<version>4.3.0-SNAPSHOT</version>

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

Successfully merging this pull request may close these issues.

2 participants