search

timob / jnigi

Golang Java JNI library
BSD 2-Clause "Simplified" License
163 stars 44 forks source link

MacOS AWT/Swing support, JDK-7131356 workaround #36

Closed dbarganski closed 3 years ago

timob commented 4 years ago

Nice 👍! I will try this out so I can understand how this works. Is it possible to post some example code that uses AWT/Swing with JNIGI?

timob commented 4 years ago

About https://github.com/timob/jnigi/issues/35, that you are fixing in this PR too. I just tried using go test and also ran this graphical program https://github.com/timob/swt/blob/master/example/example1.go .

I'm not getting the popup window or the error message. Maybe this issue has been fixed in OpenJDK.

What versions are you using? I'm using go version go1.13.5 darwin/amd64 with 

openjdk version "14.0.2" 2020-07-14
OpenJDK Runtime Environment (build 14.0.2+12-46)
OpenJDK 64-Bit Server VM (build 14.0.2+12-46, mixed mode, sharing)

on Mojave

dbarganski commented 4 years ago

Probably you have an installed JDK? In my use case i have JDK contained within project's sub-directory.

openjdk 11.0.1 2018-10-16 OpenJDK Runtime Environment 18.9 (build 11.0.1+13) OpenJDK 64-Bit Server VM 18.9 (build 11.0.1+13, mixed mode)

timob commented 4 years ago

I could have a system JDK installed, when I installed Intelli-J IDE it installed OpenJDK in ~/Library/Java/JavaVirtualMachines/openjdk-14.0.1 and this seems to be available system wide.

I'm not sure how to uninstall it. So let's just do the fix, and see how it works.

I've split the bug fix out into this PR https://github.com/timob/jnigi/pull/43, can you 👀 at that PR?

timob commented 4 years ago

JDK-7131356 fix is now in master.

timob commented 3 years ago

Most of this is merged. Please start new PR for remaining part if needed.