> On 22 Oct 2013, at 20:19, Mike Duigou <mike.dui...@oracle.com> wrote: > > This is the same change I am using locally. It wasn't clear why the SCTP > handling of errors was different than the rest of the libs.
The SCTP native code is clean enough to enable Werror, not sure if others areas are but they certainly do not build with Werror. In this specific case there is no possibility of fixing this warning, so suppression seems the best option. -Chris. > > Mike > >> On Oct 22 2013, at 11:44 , Chris Hegarty <chris.hega...@oracle.com> wrote: >> >> The SCTP native code complies with -Werror. gcc 4.8.1 reports unused >> parameters as warnings, and this causes the build to fail. gcc 4.8.1 is the >> defaul on Ubuntu 13.10. >> >> These parameters are required as the functions are JNI functions. The >> simplest solution is to ignore/suppress these unused param warnings. >> >> See discussion on: >> http://mail.openjdk.java.net/pipermail/build-dev/2013-July/009513.html >> >> diff --git a/makefiles/lib/NioLibraries.gmk b/makefiles/lib/NioLibraries.gmk >> --- a/makefiles/lib/NioLibraries.gmk >> +++ b/makefiles/lib/NioLibraries.gmk >> @@ -150,7 +150,7 @@ >> >> ifneq ($(OPENJDK_TARGET_OS), macosx) >> >> - SCTP_WERROR := -Werror >> + SCTP_WERROR := -Werror -Wno-error=unused-parameter >> ifeq ($(OPENJDK_TARGET_CPU_ARCH), ppc) >> SCTP_WERROR := >> endif >> >> With this change it may be possible to remove the special casing of ppc? I >> would need to check, but this could be handled separately. >> >> -Chris. >
