From mboxrd@z Thu Jan 1 00:00:00 1970 From: alexandre.montplaisir@polymtl.ca (Alexandre Montplaisir) Date: Wed, 08 Sep 2010 11:31:01 -0400 Subject: [ltt-dev] LTTV BUG: Recent upgrades broke the JNI interface Message-ID: <4C87AC35.8000001@polymtl.ca> (It seems LTTV can't let go of me...) I was trying to reinstall TMF, which includes the compilation of LTTV with "./configure --with-jni-interface". However, make was now getting build errors: jni_interface.c: In function ?Java_org_eclipse_linuxtools_lttng_jni_JniMarkerField_ltt_1getOffset?: jni_interface.c:1005: error: ?struct marker_field? has no member named ?offset? jni_interface.c: In function ?Java_org_eclipse_linuxtools_lttng_jni_JniMarkerField_ltt_1getSize?: jni_interface.c:1012: error: ?struct marker_field? has no member named ?size? jni_interface.c: In function ?Java_org_eclipse_linuxtools_lttng_jni_JniMarkerField_ltt_1printMarkerField?: jni_interface.c:1049: error: ?struct marker_field? has no member named ?offset? jni_interface.c:1050: error: ?struct marker_field? has no member named ?size? jni_interface.c: In function ?Java_org_eclipse_linuxtools_lttng_jni_JniParser_ltt_1getParsedData?: jni_interface.c:1134: error: ?struct marker_field? has no member named ?offset? Me and Michael were getting the same problem. Sun-java, openjdk gcj so were all giving the same errors. We ended up doing a git bisect, which gave the following commit as being responsible: 1184dc37d4cee465b9888fcec352c4ebe86d4246 is the first bad commit commit 1184dc37d4cee465b9888fcec352c4ebe86d4246 Author: Mathieu Desnoyers Date: Fri Aug 20 12:02:42 2010 -0400 ltt traceread: fix offsets calculation, should be per tracefile Signed-off-by: Mathieu Desnoyers :040000 040000 b655cf92cab4b1a11f4000e27663426da1e1d1dd 4e660bf0e8d8ad2ff0262e1c3223490d3f400178 M ltt I'll look into it this afternoon, but I suppose the changes in that commit were made for a reason, so I'm not too confident touching it. Thanks, Alexandre ps. On a side note, it seems building liblttvtraceread with OpenJDK still works with TMF, so there might be hope for a liblttvtraceread-jni package after all...