Skip to content
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

Jump to bottom

Illegal reflective access by processing.app.platform.LinuxPlatform #207

Closed
benfry opened this issue Jun 18, 2021 · 5 comments
Closed

Illegal reflective access by processing.app.platform.LinuxPlatform #207

benfry opened this issue Jun 18, 2021 · 5 comments
Labels
Help Wanted We have very little time and would like some help Linux Issue is specific to Linux

Comments

@benfry
Copy link
Member

benfry commented Jun 18, 2021

The X11 WM_CLASS property which is used as the application name by Gnome3 and other window managers, so in response to processing/processing#2534, we added code to modify the name. It relies on a hack that makes a field public, so Java 11+ complains about it.

To fix, we need a more reliable way to specify the application name so that it doesn't show up as Processing.app.Base on Linux. This could be as simple as modifying the launcher script to set the WM_CLASS environment variable to Processing on startup, but someone would need to spend some time looking into it and testing any possible solution on various Linux distros.

There may also be an alternative using JNA, but on a quick look, it seemed to required a contrib library for JNA, which we probably don't want to rely on for sake of longevity.
@benfry benfry changed the title Illegal reflective access by processing.app.platform.LinuxPlatform Illegal reflective access by processing.app.platform.LinuxPlatform Jun 18, 2021
@benfry benfry changed the title Illegal reflective access by processing.app.platform.LinuxPlatform Illegal reflective access by processing.app.platform.LinuxPlatform Jun 18, 2021
@benfry benfry added the Help Wanted We have very little time and would like some help label Jun 18, 2021
@benfry benfry mentioned this issue Jun 18, 2021
Open
@benfry benfry added the Linux Issue is specific to Linux label Jun 18, 2021
@benfry benfry mentioned this issue Aug 4, 2021
Closed
@mario6097
Copy link

mario6097 commented Jan 6, 2022 •
edited

Is this issue the reason for #333?
Is there a workaround, since otherwise I can not use Processing4 on my machine F35?
It would be great if somebody fix it.

@benfry
Copy link
Member Author

benfry commented Jan 10, 2022

I don't think so—I believe this is a benign problem that we just need to deal with at some point.

@benfry benfry mentioned this issue Jan 15, 2022
Closed
@benfry
Copy link
Member Author

benfry commented Jan 15, 2022

To be clear: this is a warning (just like the message says in all caps, I guess), not an error that will break the software (for now). But I'd still like to address the issue it's complaining about.

benfry added a commit that referenced this issue Jan 16, 2022
@benfry
update desktop file and try suppressing warning from #207
8e54334
@benfry benfry mentioned this issue Jan 16, 2022
Open
@benfry
Copy link
Member Author

benfry commented Jan 16, 2022

Warning fixed with 8e54334 but the deeper issue is still unresolved. See #207.

@benfry benfry closed this as completed Jan 16, 2022
@github-actions
Copy link

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Help Wanted We have very little time and would like some help Linux Issue is specific to Linux
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@benfry @mario6097