-
-
Notifications
You must be signed in to change notification settings - Fork 49
Move dxf
Library to Gradle
#981
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
Comments
@Stefterv Can you assign this issue to me |
2 similar comments
@Stefterv Can you assign this issue to me |
@Stefterv Can you assign this issue to me |
- Removed ant.importBuild("build.xml") - Added standalone Gradle build logic using Kotlin DSL - Configured Java 17 toolchain - Added core.jar presence check - Created custom dxfJar and clean tasks This removes reliance on Ant for the DXF library as part of the migration effort described in processing#981
Hi! @Stefterv I recently explored migrating the DXF library from Ant to Gradle and opened PR #1031 with my initial implementation. The changes included: Replacing the build.xml with a build.gradle.kts file Ensuring core.jar is properly handled via Gradle Successfully building the DXF jar using :java:libraries:dxf:dxfJar I understand the importance of getting assigned before starting work and apologize for jumping ahead. I'm really interested in contributing and learning through this process. If you're open to it, I'd love to be officially assigned to this issue so I can refine the migration further based on feedback and reopen the PR. Looking forward to your thoughts! |
Most relevant area for this enhancement?
Build process
Feature to enhance
Build system
Desired enhancement
In PR #888 I took a little shortcut with the existing Java based libraries and made them still reliant on the old Ant build system.
We would need the ant based
build.xml
to be replaced by a properbuild.gradle.kts
version to further remove our reliance on AntPotential benefits
No response
Possible drawbacks
No response
Additional context
No response
Would you like to work on the issue?
Someone else can work on this
The text was updated successfully, but these errors were encountered: