FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Publish `org.processing.core.dxf` to Maven Central (Publish libraries to maven central) by Stefterv · Pull Request #1403 · processing/processing4 · GitHub

Publish org.processing.core.dxf to Maven Central (Publish libraries to maven central) - #1403

Merged
catilac merged 4 commits into
processing:mainfrom
Stefterv:publish-dxf
Feb 6, 2026
Merged

Publish org.processing.core.dxf to Maven Central (Publish libraries to maven central)#1403
catilac merged 4 commits into
processing:mainfrom
Stefterv:publish-dxf

Conversation

Stefterv commented Feb 3, 2026
edited
Loading

Copy link
Copy Markdown
Member

Stefterv commented Feb 3, 2026

Copy link
Copy Markdown
Member Author

@avinxshKD would you be interested in applying the same to some of the other Java Libraries?

Copy link
Copy Markdown
Contributor

@Stefterv Sure, I can take a look at those.

Breakdown looks like this:

  • PDF: Good to go. It already has the Gradle setup, just needs the publishing block.

  • Net / SVG: Need to be fully ported from Ant to Gradle first, but otherwise straightforward.

  • Serial / IO: These are harder because of the native libs/JARs. We'll need to figure out how to package those correctly for Maven so we don't break cross-platform support.

One catch on this PR: build.gradle.kts lists implementation("com.lowagie:itext:2.1.7"). pretty sure that's a copy-paste error from the PDF library? DXF shouldn't depend on iText.

Stefterv commented Feb 4, 2026

Copy link
Copy Markdown
Member Author

Great! Just make individual PR's for each package starting with the easiest.

And thanks for catching the dependency!

Stefterv marked this pull request as ready for review February 4, 2026 09:36
Stefterv requested review from SableRaf and catilac February 4, 2026 09:37
Stefterv changed the title Publish libraries to maven central Publish org.processing.core.dxf to Maven Central (Publish libraries to maven central) Feb 4, 2026

catilac commented Feb 5, 2026

Copy link
Copy Markdown
Collaborator

@Stefterv now that the action has been updated (#1401) , would it be a big lift to rebase this onto main? Or actually would a pulling in the changes from main suffice?

Stefterv commented Feb 5, 2026

Copy link
Copy Markdown
Member Author

Trying with a rebase on this and the PR before

avinxshKD commented Feb 6, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

@Stefterv @catilac

working on Serial and IO libraries now. PR coming soon. :)

Copy link
Copy Markdown
Contributor

Using the same coordinates as PDF/Net/SVG does that look right for consistency? @Stefterv

Stefterv commented Feb 6, 2026

Copy link
Copy Markdown
Member Author

Yes, org.processing.core.serial and org.processing.core.io please

catilac merged commit 870d116 into processing:main Feb 6, 2026
6 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL