-
-
Notifications
You must be signed in to change notification settings - Fork 81
Migration of the serial
library to Gradle
#1106
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
base: main
Are you sure you want to change the base?
Conversation
- Add settings.gradle.kts - Update build.gradle.kts
…into issue1099
Hi @madhav2348 I have opened #1189 as an example of how this kind of migration should be approached. Would you be interested in adjusting your approach based on that PR? |
Sure , I'll be happy to work on that |
serial
library to Gradle
hi @Stefterv , taking your example as reference, I searched dependencies {
compileOnly(project(":core"))
implementation("org.scream3r:jssc:2.8.0")
} But it doesn't have the method processing4/java/libraries/serial/src/processing/serial/Serial.java Lines 262 to 265 in 3f36db5
Somehow I found this issue processing/processing@f0cc875 now gohai/java-simple-serial-connector is only a repo, but doesn't have any release or the solution I thought is
or fork the repo and build/upload the .jar file in release But I need your advice on this: if I'm doing it right! Thankyou |
Hi @madhav2348 thank you for flagging this and doing some of the initial research! Looking at the history it seems that the last change reverted it back to a version of https://github.com/java-native/jssc rather than a custom build. Please check if the latest (2.10.2) works or try to find out which version is currently in the repo. Otherwise just use an |
would creating an issue be the correct step ? because it is customized by a previous contributor and its depending for many years
|
Description
Migrated the serial library from Ant-based
build.xml
to Gradlebuild.gradle.kts
Resolve Issue
serial
Library to Gradle #1099Changes
settings.gradle.kts
Let me know if further adjustments are needed!