Skip to content

Add IntelliJ build troubleshooting tips (JDK version, GUI launch, duplicate roots) to BUILD.md #1095

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

Closed
marioguzzzman opened this issue May 15, 2025 · 5 comments · Fixed by #1096
Assignees

Comments

@marioguzzzman
Copy link
Contributor

Hello,

First, thanks for the clear steps in BUILD.md, they helped a lot!

While I just went through the build process using on my system, I ran into a few issues that might be common for new contributors, especially when using IntelliJ.

I'm using:

  • IntelliJ IDEA 2024.3.4.1 Community Edition
  • Windows 11 (amd64)

And here are some suggestions I'd love to contribute:

What I experienced:

  • The build didn't work until I manually selected Temurin 17.0.15 as the JDK inside IntelliJ. Other JDKs (ms-17, etc.) didn't launch the app or caused errors. I’m still not 100% sure why, so if there’s a technical reason for this preference, it could be good to mention that directly?
  • I also got this warning in IntelliJ:
    "Duplicate content roots detected" things still worked fine, but it might help to say it's safe to ignore.

Would it make sense to add a small visual guide to BUILD.md with these things?

What I’d love to work on:

  • Add a Troubleshooting and Setup Tips section to BUILD.md
  • Optionally include screenshots of IntelliJ (JDK selection, warning popup, manual run config setup)
  • Maybe even a visual guide that can be launched from the Processing GUI itself? This might become a whole new issue.

Let me know if it would be helpful.

Thanks again!

cc @SableRaf , since you recently updated the BUILD.md, I figured you might have thoughts on this. Happy to draft something and open a PR if that sounds good.

@Stefterv Stefterv assigned Stefterv and marioguzzzman and unassigned Stefterv May 15, 2025
@Stefterv
Copy link
Collaborator

Hi @marioguzzzman Sound good to me, thank you for your initiative!

@SableRaf
Copy link
Collaborator

Thanks @marioguzzzman!

– Add a Troubleshooting and Setup Tips section to BUILD.md
– Optionally include screenshots of IntelliJ (JDK selection, warning popup, manual run config)

I agree with Stef, please go ahead with the above and make a PR. Let me know if you need any input.

– Maybe even a visual guide that can be launched from the Processing GUI?

I like the idea of letting people know they can build Processing from source, but I'd rather not maintain two separate sources of truth. Let's hold off on that for now and revisit it when we have a chance to do a larger overhaul of the contributor docs.

@marioguzzzman
Copy link
Contributor Author

Thanks @Stefterv and @SableRaf!

I’ll go ahead and prepare the PR with the troubleshooting section and screenshots for the BUILD.md file only. I agree with keeping one-source for now.

Appreciate the support and will share the PR shortly!

@SableRaf
Copy link
Collaborator

@all-contributors please add @marioguzzzman for documentation

Copy link
Contributor

@SableRaf

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

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 a pull request may close this issue.

3 participants