[PATCH] Review Request: 7076644 TEST_BUG: test/java/io/File/Basic.java fails with cygwin

2012-08-07 Thread Eric Wang
Hi Alan, Can you please help to review the fix below for bug http://monaco.us.oracle.com/detail.jsf?cr=7076644? http://dl.dropbox.com/u/90659131/fixes/7076644/webrev/index.html As discussed, it is caused a known issue of cygwin that chmod doesn't work, see http://cygwin.com/faq/faq-nochunks.

Process.exec() hung on Windows, how to tell what happened from dmp file?

2012-08-07 Thread Dawid Weiss
Hi everyone, Seems like the right list to ask, but redirect me if there's a better place, please. I have a Java process that forks a process and waits for it. When repeatedly running on: Windows (64 bit) java version "1.7.0_05" Java(TM) SE Runtime Environment (build 1.7.0_05-b05) Java HotSpot(TM

Re: [PATCH] Review Request: 7076644 TEST_BUG: test/java/io/File/Basic.java fails with cygwin

2012-08-07 Thread Alan Bateman
On 07/08/2012 08:00, Eric Wang wrote: Hi Alan, Can you please help to review the fix below for bug http://monaco.us.oracle.com/detail.jsf?cr=7076644? http://dl.dropbox.com/u/90659131/fixes/7076644/webrev/index.html As discussed, it is caused a known issue of cygwin that chmod doesn't work,

Re: Improve registering signal handlers in java.lang.Terminator.setup()

2012-08-07 Thread Alan Bateman
On 07/08/2012 04:35, David Holmes wrote: The change is harmless but as far as I can see you would have to customize the VM before this change would have any affect. The signals involved here are the SHUTDOWNn_SIGNAL values (which for linux/solaris are HUP, INT and TERM). JVM_RegisterSignal si

Re: PING: [PATCH FOR REVIEW] System Zlib Support

2012-08-07 Thread Andrew Hughes
- Original Message - > On 08/06/2012 11:26 AM, Andrew Hughes wrote: > > - Original Message - > >> Hi Andrew, > >> > >> I meant if we are going to put SYSTEM_ZLIB=true as default for > >> linux > >> as > >> Alan suggested, > >> we might need to update the build document as well to in

hg: jdk8/tl/jdk: 7076644: TEST_BUG: test/java/io/File/Basic.java fails with cygwin

2012-08-07 Thread alan . bateman
Changeset: e0ef14d89741 Author:alanb Date: 2012-08-07 12:47 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e0ef14d89741 7076644: TEST_BUG: test/java/io/File/Basic.java fails with cygwin Reviewed-by: alanb Contributed-by: Eric Wang ! test/ProblemList.txt ! test/java/io/File/

Re: [PATCH FOR REVIEW] Potential Buffer Overflow in java_props_md.c

2012-08-07 Thread Andrew Hughes
- Original Message - > Thanks Andrew. 7189533 created for this. > Thanks. Ok to push then? :-) > David > - > > On 3/08/2012 8:03 AM, Andrew Hughes wrote: > > > > > > - Original Message - > >> Hi Andrew, > >> > >> On 2/08/2012 7:35 PM, Andrew Hughes wrote: > >>> - Origin

Re: Improve registering signal handlers in java.lang.Terminator.setup()

2012-08-07 Thread Neil Richards
On Tue, 2012-08-07 at 10:15 +0100, Alan Bateman wrote: > On 07/08/2012 04:35, David Holmes wrote: > > > > The change is harmless but as far as I can see you would have to > > customize the VM before this change would have any affect. The signals > > involved here are the SHUTDOWNn_SIGNAL values (

Re: [PATCH FOR REVIEW] Potential Buffer Overflow in java_props_md.c

2012-08-07 Thread Xueming Shen
Andrew, Since we are here:-) are we also supposed to "free" the old_temp at #250 and old_temp and old_ev at the end? -Sherman On 08/07/2012 04:49 AM, Andrew Hughes wrote: - Original Message - Thanks Andrew. 7189533 created for this. Thanks. Ok to push then? :-) David - On

Re: Improve registering signal handlers in java.lang.Terminator.setup()

2012-08-07 Thread Alan Bateman
On 07/08/2012 15:02, Neil Richards wrote: : > From a Java Class Library point of view, it seems to me that the desire is to register shutdown hooks for any of these signals (HUP, INT and TERM) whose use has not been restricted by the VM. So an attempt to register for each of these signals should

Re: PING: [PATCH FOR REVIEW] System Zlib Support

2012-08-07 Thread Kumar Srinivasan
Hi, Please make sure that the jdk/test/tools/* tests are run and validated. Both the java launcher and the pack200 tools are dependent on zlib entry points for inflate/deflate and other things. Kumar - Original Message - On 08/06/2012 11:26 AM, Andrew Hughes wrote: - Original Me

Re: Improve registering signal handlers in java.lang.Terminator.setup()

2012-08-07 Thread Neil Richards
On Tue, 2012-08-07 at 18:01 +0100, Alan Bateman wrote: > On 07/08/2012 15:02, Neil Richards wrote: > > : > > > From a Java Class Library point of view, it seems to me that the desire > > is to register shutdown hooks for any of these signals (HUP, INT and > > TERM) whose use has not been restricted

Re: [PATCH FOR REVIEW] Potential Buffer Overflow in java_props_md.c

2012-08-07 Thread David Holmes
On 8/08/2012 2:40 AM, Xueming Shen wrote: Andrew, Since we are here:-) are we also supposed to "free" the old_temp at #250 and old_temp and old_ev at the end? No. They are aliases for temp and encoding_variant, which are freed at the end. There are only ever at most two things to free. And

Re: Improve registering signal handlers in java.lang.Terminator.setup()

2012-08-07 Thread David Holmes
On 8/08/2012 4:07 AM, Neil Richards wrote: On Tue, 2012-08-07 at 18:01 +0100, Alan Bateman wrote: On 07/08/2012 15:02, Neil Richards wrote: : From a Java Class Library point of view, it seems to me that the desire is to register shutdown hooks for any of these signals (HUP, INT and TERM) who

Re: [PATCH FOR REVIEW] Potential Buffer Overflow in java_props_md.c

2012-08-07 Thread Xueming Shen
On 8/7/12 5:33 PM, David Holmes wrote: On 8/08/2012 2:40 AM, Xueming Shen wrote: Andrew, Since we are here:-) are we also supposed to "free" the old_temp at #250 and old_temp and old_ev at the end? No. They are aliases for temp and encoding_variant, which are freed at the end. There are onl

hg: jdk8/tl/corba: Added tag jdk8-b50 for changeset d20d9eb9f093

2012-08-07 Thread lana . steuck
Changeset: 9b0f841ca9f7 Author:katleman Date: 2012-08-02 15:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/9b0f841ca9f7 Added tag jdk8-b50 for changeset d20d9eb9f093 ! .hgtags

hg: jdk8/tl/jaxp: Added tag jdk8-b50 for changeset 2791ec55f66b

2012-08-07 Thread lana . steuck
Changeset: dc1ea77ed9d9 Author:katleman Date: 2012-08-02 15:35 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/dc1ea77ed9d9 Added tag jdk8-b50 for changeset 2791ec55f66b ! .hgtags

hg: jdk8/tl: Added tag jdk8-b50 for changeset 2fd67618b9a3

2012-08-07 Thread lana . steuck
Changeset: 57c0aee73090 Author:katleman Date: 2012-08-02 15:32 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/rev/57c0aee73090 Added tag jdk8-b50 for changeset 2fd67618b9a3 ! .hgtags

hg: jdk8/tl/jaxws: Added tag jdk8-b50 for changeset bdab72e87b83

2012-08-07 Thread lana . steuck
Changeset: 1a70b6333ebe Author:katleman Date: 2012-08-02 15:35 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/1a70b6333ebe Added tag jdk8-b50 for changeset bdab72e87b83 ! .hgtags

hg: jdk8/tl/langtools: 2 new changesets

2012-08-07 Thread lana . steuck
Changeset: c4cd4cab2220 Author:katleman Date: 2012-08-02 15:37 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/c4cd4cab2220 Added tag jdk8-b50 for changeset b2d8a270f5f2 ! .hgtags Changeset: cfa70d7ac944 Author:lana Date: 2012-08-07 20:24 -0700 URL: http

hg: jdk8/tl/hotspot: 37 new changesets

2012-08-07 Thread lana . steuck
Changeset: 54e66510c9cd Author:amurillo Date: 2012-07-13 14:16 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/54e66510c9cd 7184050: new hotspot build - hs24-b17 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 8150fa46d2ed Author:jiangli Date: 2012-06-26

hg: jdk8/tl/jdk: 14 new changesets

2012-08-07 Thread lana . steuck
Changeset: 79c9847d4a5f Author:katleman Date: 2012-08-02 15:36 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/79c9847d4a5f Added tag jdk8-b50 for changeset e4bae5c53fca ! .hgtags Changeset: 28665fa73b4a Author:rupashka Date: 2012-07-19 19:09 +0400 URL: http:/