Skip to content

Migration of the dxf library from Ant to Gradle#1192

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.

Sign up for GitHub

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

Jump to bottom
Merged
catilac merged 2 commits into from
Sep 12, 2025
Merged

Migration of the dxf library from Ant to Gradle #1192

catilac merged 2 commits into from
Sep 12, 2025

Conversation

@npNSU
Copy link

@npNSUnpNSU commented Jul 28, 2025•
edited
Loading

Resolves

This PR resolves#981

Changes

  • Replicated the code from Pull Request #1189
  • Edited the files: java/libraries/dxf/build.gradle.kts & java/build.gradle.kts
  • Took out the dependency lines(17-21) in /java/libraries/dxf/build.gradle.kts that appeared in PR 1189.
  • Modified line 89: val libraries = arrayOf(“dxf") in java/build.gradle.kts

Tests

  • We ran ./gradlew test, built successfully.
  • Ran ./gradlew run, built IDE successfully.
  • Tested the creation of a .dxf file from a sketch using the following the sketch code: /Users/nia/processing4/java/libraries/dxf/examples/SimpleExport/SimpleExport.pde
  • Tested opening outputted .dxf file in ShareCAD; displayed as expected

Screenshots

Opened IDE after changes with test code and sketch
IDE Image

Produced .dxf file opened in ShareCAD
image

Thank you for your guidance @Stefterv! Please let us know if we need to make further adjustments!

@np158@awood0727
@kaybcodes@SilasVM
Worked on migration of the dxf library from Ant to Gradle
20425fd
Signed-off-by: [Nia Perez] <[email protected]> Co-authored-by: Amante' Woodley <[email protected]> Co-authored-by: Kayla Bobo <[email protected]> Co-authored-by: Silas Morgan <[email protected]>
@SableRafSableRaf requested a review from SteftervJuly 28, 2025 20:15
@Stefterv
Copy link
Collaborator

Stefterv commented Jul 29, 2025

Thank you @npNSU, looks good to me!

We will have to wait to merge these changes until we merge my PR (and probably resolve the conflict)

@SteftervStefterv changed the title Worked on migration of the dxf library from Ant to GradleMigration of the dxf library from Ant to GradleJul 29, 2025
@SteftervStefterv added this to the 4.4.8 milestone Sep 8, 2025
@SteftervStefterv moved this to In progress in Processing RoadmapSep 8, 2025
@catilac
Copy link
Collaborator

catilac commented Sep 12, 2025

I resolved the merge conflict!

@catilaccatilac merged commit 2c94b38 into processing:mainSep 12, 2025
@github-project-automationgithub-project-automationbot moved this from In progress to Completed in Processing RoadmapSep 12, 2025
@catilaccatilac mentioned this pull request Sep 12, 2025
Closed
@catilac
Copy link
Collaborator

catilac commented Sep 12, 2025

Thank you @npNSU ! 🎉

@SableRaf
Copy link
Collaborator

SableRaf commented Sep 12, 2025

@all-contributors please add @npNSU for code

@allcontributorsallcontributorsbot mentioned this pull request Sep 12, 2025
Merged
@allcontributors
Copy link
Contributor

allcontributorsbot commented Sep 12, 2025

@SableRaf

I've put up a pull request to add @npNSU! 🎉

@CodebyDipanCodebyDipan mentioned this pull request Sep 15, 2025
Closed
Stefterv added a commit to Stefterv/processing4 that referenced this pull request Sep 22, 2025
@Stefterv
Squashed commit of the following:
bae6533
commit b2f79f6 Author: Stef Tervelde <[email protected]> Date: Mon Sep 22 14:21:27 2025 +0200 Include root-level Java files in source set Uncommented and updated the configuration to include all .java files at the project root in the source set. This ensures that Java files located directly in the project directory are recognized and compiled. commit 4aeff01 Merge: 046a21672db9b7 Author: Stef Tervelde <[email protected]> Date: Mon Sep 22 14:07:25 2025 +0200 Merge branch 'main' into gradle-runner commit 72db9b7 Merge: 0c43ebfaa9fd93 Author: Moon <[email protected]> Date: Fri Sep 19 08:24:30 2025 -0400 Merge pull request processing#1251 from Stefterv/lsp-libraries-fix Make sure LSP recognises contributed libraries commit aa9fd93 Author: Stef Tervelde <[email protected]> Date: Fri Sep 19 11:19:37 2025 +0200 Make sure contributions are loaded in the lsp Trigger locating the sketchbook folder so contributed libraries are loaded commit 0c43ebf Merge: 9ca92bc5a108da Author: Moon <[email protected]> Date: Thu Sep 18 09:01:09 2025 -0400 Merge pull request processing#1248 from processing/all-contributors/add-SuganthiThomas docs: add SuganthiThomas as a contributor for code commit 5a108da Author: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Date: Thu Sep 18 13:00:49 2025 +0000 docs: update .all-contributorsrc [skip ci] commit f3198d7 Author: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Date: Thu Sep 18 13:00:48 2025 +0000 docs: update README.md [skip ci] commit 9ca92bc Merge: 13e697788b9e04 Author: Moon <[email protected]> Date: Thu Sep 18 08:54:28 2025 -0400 Default DEBUG env value to false, or read actual value instead of only checking for existence commit 88b9e04 Author: Suganthi Thomas <[email protected]> Date: Wed Sep 17 22:12:07 2025 -0500 Removed commented lines commit 014fd93 Author: Suganthi Thomas <[email protected]> Date: Tue Sep 16 17:43:01 2025 -0500 DEBUG should be set to the actual value. commit 13e6977 Merge: bdbd025a741775 Author: Stef Tervelde <[email protected]> Date: Tue Sep 16 09:12:10 2025 +0200 Merge pull request processing#1239 from Stefterv/develop-menu Add 'Develop' menu for debug mode with update check commit a741775 Author: Stef Tervelde <[email protected]> Date: Fri Sep 12 21:51:13 2025 +0200 Added Language key for `menu.develop.check_for_updates` commit bdbd025 Merge: 354e5609b44035 Author: Moon <[email protected]> Date: Fri Sep 12 14:30:06 2025 -0400 Merge pull request processing#1242 from processing/all-contributors/add-npNSU docs: add npNSU as a contributor for code commit 9b44035 Author: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Date: Fri Sep 12 18:26:32 2025 +0000 docs: update .all-contributorsrc [skip ci] commit 2800725 Author: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Date: Fri Sep 12 18:26:31 2025 +0000 docs: update README.md [skip ci] commit c3b2c6b Merge: d3f3611354e560 Author: Stef Tervelde <[email protected]> Date: Fri Sep 12 17:54:13 2025 +0200 Merge branch 'processing:main' into develop-menu commit 354e560 Merge: 2c94b389507241 Author: Moon <[email protected]> Date: Fri Sep 12 11:50:16 2025 -0400 Merge pull request processing#1236 from Stefterv/main Fix the issue with the pre-processor test commit 2c94b38 Merge: 34a85df4495cc5 Author: Moon <[email protected]> Date: Fri Sep 12 10:54:55 2025 -0400 Merge pull request processing#1192 from npNSU/dxfMigration Migration of the `dxf` library from Ant to Gradle commit 4495cc5 Merge: 20425fd34a85df Author: Moon <[email protected]> Date: Fri Sep 12 10:39:45 2025 -0400 Merge branch 'main' into dxfMigration commit 34a85df Merge: 2e855a13fbfddb Author: Stef Tervelde <[email protected]> Date: Fri Sep 12 15:34:22 2025 +0200 Merge pull request processing#1222 from hamoid/addDependencyUpdates Add Gradle task to check for dependency updates commit 2e855a1 Merge: 275d7893814307 Author: Raphaël de Courville <[email protected]> Date: Fri Sep 12 14:43:31 2025 +0200 Merge pull request processing#1226 from Stefterv/pixel-density-warning Add warning for default pixelDensity behavior commit 3814307 Author: Raphaël de Courville <[email protected]> Date: Fri Sep 12 14:42:36 2025 +0200 Update the warning message for pixelDensity Made the warning message more helpful commit 9507241 Author: Stef Tervelde <[email protected]> Date: Fri Sep 12 07:46:40 2025 +0200 Update bug1532 test and resource file Adjusted expected exception position in ParserTests for bug1532 and removed QuickTime session initialization from bug1532.pde to reflect updated test requirements. commit 3fbfddb Merge: 0e18502275d789 Author: Stef Tervelde <[email protected]> Date: Thu Sep 11 16:09:43 2025 +0200 Merge branch 'main' into addDependencyUpdates commit d3f3611 Author: Stef Tervelde <[email protected]> Date: Thu Sep 11 16:02:49 2025 +0200 Add 'Develop' menu for debug mode with update check Introduces a new 'Develop' menu that appears when debug mode is enabled. commit 0e18502 Author: Abe Pazos <[email protected]> Date: Thu Sep 11 14:09:19 2025 +0200 Gradle libs.versions.toml: inline "versions" version commit 275d789 Merge: 11ba3b4faf5997 Author: Moon <[email protected]> Date: Tue Sep 9 15:03:41 2025 -0400 Merge pull request processing#1189 from Stefterv/fix-pdf-library Migration of the `pdf` library to Gradle commit 11ba3b4 Merge: 43ba46b00e4243 Author: Moon <[email protected]> Date: Tue Sep 9 15:02:51 2025 -0400 Merge pull request processing#1232 from Stefterv/gradle-runner-preproc Moving the `java:preprocessor` to be fully Gradle native commit 43ba46b Merge: c570b6d7e109bd Author: Moon <[email protected]> Date: Tue Sep 9 15:01:52 2025 -0400 Merge pull request processing#1228 from Stefterv/format-cli Adding the sketch format command commit c570b6d Merge: e56e2818f418a2 Author: Moon <[email protected]> Date: Mon Sep 8 14:41:47 2025 -0400 Merge pull request processing#1227 from awood0727/debugFixUpdate Updated fix for the debugger issue commit 8f418a2 Author: Amante' Woodley <[email protected]> Date: Fri Sep 5 09:53:39 2025 -0400 Updated fix for the dubugger issue Signed-off-by: Amante' Woodley <[email protected]> Co-authored-by: Nia Perez <[email protected]> Co-authored-by: Kayla Bobo <[email protected]> Co-authored-by: Silas Morgan <[email protected]> commit 7e109bd Author: Stef Tervelde <[email protected]> Date: Fri Sep 5 14:28:03 2025 +0200 Adding the sketch format command commit 906ab17 Author: Stef Tervelde <[email protected]> Date: Thu Sep 4 10:34:45 2025 +0200 Add warning for default pixelDensity behavior Introduces a warning message when pixelDensity defaults to 2x to match the display's pixel density. The warning prompts users to explicitly set pixelDensity in settings() to avoid the message. commit 452fbce Author: Abe Pazos <[email protected]> Date: Sat Aug 30 00:15:17 2025 +0200 Add Gradle task to check for dependency updates commit 20425fd Author: [Nia Perez] <[email protected]> Date: Mon Jul 28 14:30:12 2025 -0400 Worked on migration of the dxf library from Ant to Gradle Signed-off-by: [Nia Perez] <[email protected]> Co-authored-by: Amante' Woodley <[email protected]> Co-authored-by: Kayla Bobo <[email protected]> Co-authored-by: Silas Morgan <[email protected]> commit faf5997 Author: Stef Tervelde <[email protected]> Date: Wed Jul 23 06:58:14 2025 +0200 Migration of the `pdf` library to Gradle
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SteftervSteftervStefterv approved these changes

Assignees

No one assigned

Labels

None yet

Projects

Processing Roadmap
Status: Completed

Milestone

4.4.11

Development

Successfully merging this pull request may close these issues.

Move dxf Library to Gradle

5 participants

@npNSU@Stefterv@catilac@SableRaf@np158