On Fri, Jan 22, 2021 at 10:52 PM Nathan Hartman <hartman.nat...@gmail.com> wrote: > > On Fri, Oct 16, 2020 at 1:58 PM Alexandr Miloslavskiy > <alexandr.miloslavs...@syntevo.com> wrote: > > > > I have committed a series of patches for JavaHL on 'javahl-1.14-fixes' > > branch. They mostly address crashes when using TunnelAgent, but also fix > > some JNI warnings. > > > > The branch also includes automatic tests for problems I fixed. If you > > only checkout the first commit of the branch, there will be tests > > without any fixes. Adding more commits will gradually fix problems > > demonstrated by the new tests. > > > > Now I hope for some attention and reviews :) > > > > Here's how to run JavaHL tests on macOS/Linux: > > ---------------------------------------------- > > 1) Download [1] JDK and extract it to some folder, for example > > /home/user/Downloads/jdk. I have tested with versions 8 and 14, haven't > > yet tried the newest version 15. > > 2) Download [2] JUnit and put .jar to some folder, for example > > /home/user/Downloads/junit. > > 3) During 'configure' SVN build step, pass additional arguments: > > --with-jdk=/home/user/Downloads/jdk > > --with-junit=/home/user/Downloads/junit/junit-4.12.jar > > --enable-javahl > > 4) Compile with: > > make javahl > > 5) Run tests with: > > make check-apache-javahl > > > > [1] https://www.oracle.com/java/technologies/javase-jdk14-downloads.html > > [2] https://repo1.maven.org/maven2/junit/junit/4.12/junit-4.12.jar > > Just resurrecting the above from the mists of the mailing list... > > Would anyone who feels at home with Java be inclined to review the > javahl-1.14-fixes branch in time to merge/nominate it for 1.14.1? It's > supposed to fix a few crashes (see quoted message above). (What > stopped me from doing it already is that I need to diagnose why my > system wouldn't run the JavaHL tests.) > > My intention for this weekend is to: > > (1) write a regression test for the 'svn info --xml' bug and then > commit the fix from a few days ago in [1] > > (2) review/test/vote on STATUS for 1.10.x and 1.14.x > > (3) if time remains, fix whatever is broken on my system and review > that branch > > If someone could beat me to #3, I'll be very grateful. > > Cheers, > Nathan > > [1] > https://mail-archives.apache.org/mod_mbox/subversion-dev/202101.mbox/%3ccajt2ehpwng8jcz_mzr3zhqwxzxpdboi27ggaxidghm3mg12...@mail.gmail.com%3e
Ah, thanks for bringing this back up, and sorry to Alexandr for letting this linger for so long. I'll try to take a closer look this weekend. I am a Java developer in my $dayjob, but I don't know much about JNI nor about JavaHL. Anyway, I should be able to dig into it, and at least I already have the JavaHL testsuite running successfully here :-). Of course if someone else, who knows a bit about JavaHL perhaps, can beat me / us to it, I wouldn't object ;-). -- Johan
