GeezerVison is showing github.com — Adding a Gradle Build system to the Processing IDE by Stefterv · Pull Request #888 · processing/processing4
Conversation
Stefterv
commented
Dec 14, 2024
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
Here's the first pull-request adding a new Gradle build system to the Processing IDE.
The GitHub Actions will create distributable binaries for macOS (x64, aarch64), windows (x64) and linux (x64, aarch64). The other platforms will need their own CI/CD system, but I think the repo will already build on the other platforms.
In terms of compatibility I've come pretty far but I'm sure there is some functionality that I have missed.
New Features
- Installers for macOS (dmg), Widows (msi), linux (deb)
- Windows: Start menu item
- Jetpack Compose
- Kotlin Support for the IDE
- Splash screen Refactor + Animation
- Additional pde:// schema functions (To be documented)
Functionality to test and verify
- Examples
- Tweaks mode
- P2D
- Debugger
- Languages
- Contributed libraries
- Autocomplete
- MovieMaker
- Exporting Applications
-
processing-java
- java mode default libraries (dxf,svg,io,net,serial,pdf)
- Python mode (BROKEN, already on 4.3.1 macOS aarch64, other platforms okay)
- Android mode
To add during this pull request
To improve during this pull request
- Legacy resource handling (rn it copies all assets to temp folder on launch)
- Move all dependencies to libs.versions.toml
- Version and Build number
After pull request
To consider
In any case, this pull request does not break the existing Ant build system so we can either wait with the merge or we can have the Gradle version available as a preview.
The bundle size has increased quite a bit since we are shipping a JRE and a JDK instead of using the JDK to run the IDE
|linux-x64 | 257 MB |
|macos-aarch64 | 294 MB |
|macos-x64 | 289 MB |
|windows-x64 | 275 MB |
P.S. this also includes the pre-processor as a Maven library, so merging this PR will add processing:java:prepocessor as a Maven package
Fixes #924
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
micycle1 reacted with rocket emoji
- Disable system look & feel on macOS for readability
- Cleaned unnecessary space
- Cleaned finished TODO
We can either re-open the editor after changing of the theme or wait until we switch that part over to Jetpack Compose
If that's not too much work, I'd say let's see how the re-opening of the editor feels.
Alternatively, how about we inform the user that the full change will take effect after they restart Processing? Could be a pop up message.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work on this PR @Stefterv! This is a huge leap forward for Processing 💙
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Stefterv and Taenerys reacted with heart emoji
|
Thank you, @Stefterv !! You worked very hard on this!
|
Stefterv, SableRaf, and Taenerys reacted with heart emoji
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
This was referenced May 15, 2025
Reviewers
SableRaf
SableRaf approved these changes
edwinRNDR
Awaiting requested review from edwinRNDR
+1 more reviewer
Reviewers whose approvals may not affect merge requirements
Uh oh!
There was an error while loading. Please reload this page.
Milestone
No milestone
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
|