Erik:

Looks good to me as well.

Tim

On 05/04/15 08:13, Staffan Larsen wrote:
Looks good!

Thanks,
/Staffan

On 4 maj 2015, at 15:46, Erik Joelsson <erik.joels...@oracle.com> wrote:

Hello,

This bug has been marked as won't fix since the code it affects is going to be 
removed. However, until that removal happens, we can't build using VS2013 SP4. 
I'm in the process of updating the compilers used internally at Oracle to that 
specific version and to be able to continue that work, I need this to be 
buildable now. For this reason I propose Peter Brunet's suggested solution to 
the problem, just adding a C macro definition that makes it compile. I slightly 
adjusted his initial proposal.

Bug: https://bugs.openjdk.java.net/browse/JDK-8077422
Patch:
diff -r 83ff0dedf9e1 make/lib/Lib-jdk.hprof.agent.gmk
--- a/make/lib/Lib-jdk.hprof.agent.gmk
+++ b/make/lib/Lib-jdk.hprof.agent.gmk
@@ -31,7 +31,7 @@

BUILD_LIBHPROF_CFLAGS := $(addprefix -I, $(BUILD_LIBHPROF_SRC)) \
     -I$(JDK_TOPDIR)/src/demo/share/jvmti/java_crw_demo

BUILD_LIBHPROF_LDFLAGS :=

LIBHPROF_OPTIMIZATION := HIGHEST
@@ -49,6 +49,7 @@
     CFLAGS := $(CFLAGS_JDKLIB) $(CFLAGS_WARNINGS_ARE_ERRORS) \
         $(BUILD_LIBHPROF_CFLAGS), \
     CFLAGS_debug := -DHPROF_LOGGING, \
+    CFLAGS_windows := -D_WINSOCK_DEPRECATED_NO_WARNINGS, \
     MAPFILE := $(JDK_TOPDIR)/make/mapfiles/libhprof/mapfile-vers, \
     LDFLAGS := $(LDFLAGS_JDKLIB) \
         $(call SET_SHARED_LIBRARY_ORIGIN), \

/Erik

On 2015-04-23 00:02, Pete Brunet wrote:
This hack to jdk/make/lib/Lib-jdk.hprof.agent.gmk works for now:

      LANG := C, \
      OPTIMIZATION := $(LIBHPROF_OPTIMIZATION), \
      CFLAGS := $(CFLAGS_JDKLIB) $(CFLAGS_WARNINGS_ARE_ERRORS) \
+        -D_WINSOCK_DEPRECATED_NO_WARNINGS \
          $(BUILD_LIBHPROF_CFLAGS), \
      CFLAGS_debug := -DHPROF_LOGGING, \
      MAPFILE := $(JDK_TOPDIR)/make/mapfiles/libhprof/mapfile-vers, \

Pete

On 4/22/15 4:12 PM, Pete Brunet wrote:
p.s. I also had done a make reconfigure and make clean.  I'll try make
clean and reconfigure from bash.

On 4/22/15 4:06 PM, Pete Brunet wrote:
I was able to build 9 OK then I switched from VS2010 to VS2013 and now
get the following.  I tried hg tpull -u but that didn't help.

Is there something I need to do besides installing VS Pro 2013 with
Update 4?

$ make images 2>&1 | tee make-64.log
Building target 'images' in configuration
'windows-x86_64-normal-server-release'

c:/Users/Pete/JDK9/INTJDK-7615864/client/jdk/src/jdk.hprof.agent/windows/native/libhprof/hprof_md.c(86)
: error C2220: warning treated as error - no 'object' file generated
c:/Users/Pete/JDK9/INTJDK-7615864/client/jdk/src/jdk.hprof.agent/windows/native/libhprof/hprof_md.c(86)
: warning C4996: 'gethostbyname': Use getaddrinfo() or GetAddrInfoW()
instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated
API warnings
         c:\progra~2\wi3cf2~1\8.1\include\um\winsock2.h(2238) : see
declaration of 'gethostbyname'
Lib-jdk.hprof.agent.gmk:44: recipe for target
'/cygdrive/c/Users/Pete/JDK9/INTJDK-7615864/client/build/windows-x86_64-normal-server-release/support/native/jdk.h
prof.agent/libhprof_jvmti/hprof_md.obj' failedmake[3]: ***
[/cygdrive/c/Users/Pete/JDK9/INTJDK-7615864/client/build/windows-x86_64-normal-server-release/support/native/jdk.hprof.agent/libhprof_jvmti/hprof_md.obj]
Error 2
make/Main.gmk:168: recipe for target 'jdk.hprof.agent-libs' failed
make[2]: *** [jdk.hprof.agent-libs] Error 1
make[2]: *** Waiting for unfinished jobs....
/cygdrive/c/Users/Pete/JDK9/INTJDK-7615864/client/make/Init.gmk:255:
recipe for target 'main' failed
make[1]: *** [main] Error 1
/cygdrive/c/Users/Pete/JDK9/INTJDK-7615864/client/make/Init.gmk:171:
recipe for target 'images' failed
make: *** [images] Error 2

Reply via email to