Hi Bryan, Thanks for your first diff! I'll be able to commit when your name shows up on the OCA list. I think it is updated every two weeks.
You may have noticed I committed some low-hanging fruit diffs in hotspot and jdk sub-trees. The iconv diff I'm holding until I hear back from Greg for FreeBSD. With that diff I get up to the part of the jdk build where it fails due to OpenBSD's sound support being different. I have one comment about the diff inline below. Regards, -Kurt On Wed, 2015-05-06 at 17:07 -0400, Bryan C. Everly wrote: > However, if someone would like to submit this diff on my behalf until > my name shows up on the list, I was able to get a bit farther on > OpenBSD 5.7: > > > $ hg diff common/autoconf/libraries.m4 > > diff -r 36cd3cebf1de common/autoconf/libraries.m4 > --- a/common/autoconf/libraries.m4 Tue May 05 18:06:47 2015 -0700 > +++ b/common/autoconf/libraries.m4 Wed May 06 17:05:17 2015 -0400 > @@ -73,6 +73,7 @@ > if test "x$OPENJDK_TARGET_OS" = xbsd; then > AC_MSG_CHECKING([what is not needed on BSD?]) > PULSE_NOT_NEEDED=yes > + ALSA_NOT_NEEDED=yes > AC_MSG_RESULT([pulse]) > fi This chunk above needs to be OpenBSD specific since at least FreeBSD has support for ALSA. > > @@ -249,6 +250,10 @@ > # A CSW package seems to be installed! > CUPS_FOUND=yes > CUPS_CFLAGS="-I/opt/csw/include" > + elif test -s /usr/local/include/cups/cups.h; then > + # Standard location for OpenBSD > + CUPS_FOUND=yes > + CUPS_CFLAGS="-I/usr/local/include" > fi > AC_MSG_RESULT([$CUPS_FOUND]) > fi > > > If folks aren't comfortable, my hope is I make it through the process > in the next week or so and I can do it then. > > Thanks, > Bryan
