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