DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=31613>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=31613 [EMAIL PROTECTED] changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | ------- Additional Comments From [EMAIL PROTECTED] 2005-01-04 18:46 ------- jclere: Thanks for getting these patches in. But I believe there are still a couple of issues: (1) You say JNI_VERSION_1_4 is not needed, but in my testing I found that if JNI_VERSION_1_2 is requested, Apple will actually load JVM 1.3.1 (for compatibility) rather than the latest JVM available. So unless I'm missing something, it is indeed necessary to set JNI_VERSION_1_4 unless one wants the invoked java code to live under JVM 1.3.1, which is not desirable. Have I missed something? (2) You decided not to put in the defaulting of JAVA_HOME, which is okay. But the change to the resolution of $JAVA_HOME/../Headers really is needed. The code in configure.in as it sits does this: cd $JAVA_HOME cd .. cd Headers This fails to correctly cd to the Headers directory on Mac OS X, since the typical setting for JAVA_HOME (/Library/Java/Home) is a symlink. The following does correctly resolve to the proper location: cd $JAVA_HOME/../Headers While it seems, intuitively, that those statements should be equivilant, they are not, at least in the presense of symlinks. Besides, the proper form is shorter anyway! Can you apply those changes? Thanks again for getting these patches in, and let me know if you've got any questions. -jdb -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]