After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 740173 - JNI Version returned is 1.2 but should be at least 1.5
JNI Version returned is 1.2 but should be at least 1.5
Status: RESOLVED FIXED
Product: java-atk-wrapper
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: java-atk-wrapper maintainer(s)
java-atk-wrapper maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-11-15 15:00 UTC by Magdalen Berns (irc magpie)
Modified: 2015-02-27 16:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to throw error if version is not found. (974 bytes, patch)
2014-11-15 15:24 UTC, Magdalen Berns (irc magpie) 		committed Details | Review
Patch to address problem (725 bytes, patch)
2015-02-24 15:32 UTC, Magdalen Berns (irc magpie) 		committed Details | Review

Description Magdalen Berns (irc magpie) 2014-11-15 15:00:08 UTC 
The build script specifies that the earliest java runtime version compatible is 1.5 yet in AtkWapper.c 1_2 or above is returned.
Comment 1 Magdalen Berns (irc magpie) 2014-11-15 15:24:21 UTC 
Created attachment 290757 [details] [review]
Patch to throw error if version is not found.

Simply changing to JNI_VERSION_1_5 does not work so for now I am just adding a version check which fails if the version sought, is not found. I got some useful tips on that from reading through the following thread:

https://community.oracle.com/thread/2337165?tstart=0

I will leave this bug open.
Comment 2 Magdalen Berns (irc magpie) 2015-02-24 15:32:31 UTC 
Created attachment 297776 [details] [review]
Patch to address problem

JNI_VERSION_1_6 is now used so it seems appropriate to do a minimum version check for this in configure.ac instead of 1.5.[1]

[1] https://git.gnome.org/browse/java-atk-wrapper/tree/jni/src/jawutil.c#n436
Comment 4 André Klapper 2015-02-27 16:56:53 UTC 
[Moving at-spi/java-atk-wrapper bugs to separate product. See bug 740075]