code review request for 7153050 remove crufty '_g' support
Daniel D. Daugherty
daniel.daugherty at oracle.com
Fri Dec 14 10:19:45 PST 2012
Thanks Tim!
Dan
On 12/14/12 10:57 AM, Tim Bell wrote:
> This looks fine to me from a build and build-infra perspective.
>
> Thanks for taking this on - code deletion is all good.
>
> Tim
>
> On 12/13/12 15:33, Daniel D. Daugherty wrote:
>> Trying again... now my e-mail client is complaining...
>>
>> Adding back in the other OpenJDK aliases...
>>
>> Ron's e-mail has gone off-line (no e-mail for him since 1440 MT)...
>>
>> Coleen,
>>
>> Thanks for the review!
>>
>> This bug (7153050) is just for the Makefile changes so that the
>> reviews are more focused. Ron has separate bugs for the other parts
>> of the hotspot repo that need to be fixed. I believe he is going to
>> attack the src/... paths in his second fix.
>>
>> Dan
>>
>>
>> On 12/13/12 4:13 PM, Coleen Phillimore wrote:
>>>
>>> Ron, I'm so happy this is going away. The code changed looks
>>> good. There are some comments in some files that refer to the old
>>> jvm_g.dll and libjvm_g.so. Can you remove these too?
>>>
>>> carrs% ygrep -l libjvm_g
>>> ./os/bsd/vm/os_bsd.cpp
>>> ./os/linux/vm/os_linux.cpp
>>> ./os/solaris/vm/os_solaris.cpp
>>> carrs% ygre jvm_g.dll
>>> ygre: Command not found.
>>> carrs% ygrep jvm_g.dll
>>> ./os/windows/vm/os_windows.cpp:// Find the full path to the current
>>> module, jvm.dll or jvm_g.dll
>>> ./share/tools/ProjectCreator/ProjectCreator.java: +
>>> "jvm.dll and jvm_g.dll; no trailing slash>");
>>>
>>> Thanks,
>>> Coleen
>>>
>>> On 12/13/2012 03:53 PM, Ron Durbin wrote:
>>>> Thx again
>>>> -----Original Message-----
>>>> From: Daniel D. Daugherty
>>>> Sent: Thursday, December 13, 2012 1:51 PM
>>>> To: build-dev; serviceability-dev at openjdk.java.net;
>>>> hotspot-runtime-dev at openjdk.java.net; Ron Durbin
>>>> Subject: Re: code review request for 7153050 remove crufty '_g'
>>>> support
>>>>
>>>> Serguei,
>>>>
>>>> Thanks for the fast review!
>>>>
>>>> I could swear that Ron had removed lines 41 and 45 in an earlier
>>>> version. Sigh... Will be fixed.
>>>>
>>>> Dan
>>>>
>>>>
>>>> On 12/13/12 1:22 PM, serguei.spitsyn at oracle.com wrote:
>>>>> Dan,
>>>>>
>>>>> It is nice fix and simplified many places.
>>>>>
>>>>> Just one minor comment:
>>>>>
>>>>> make/bsd/makefiles/dtrace.make
>>>>>
>>>>> 41 #LIBJVM_DB = libjvm_db.dylib
>>>>> 42 LIBJVM_DB = libjvm_db.dylib
>>>>>
>>>>> 45 #LIBJVM_DTRACE = libjvm_dtrace.dylib
>>>>> 46 LIBJVM_DTRACE = libjvm_dtrace.dylib
>>>>>
>>>>> The lines #41 and #45 can be removed.
>>>>>
>>>>>
>>>>> Thanks,
>>>>> Serguei
>>>>>
>>>>>
>>>>> On 12/13/12 10:53 AM, Daniel D. Daugherty wrote:
>>>>>> Greetings,
>>>>>>
>>>>>> I'm sponsoring this code review request from Ron Durbin. This change
>>>>>> is targeted at JDK8/HSX-25 in the RT_Baseline repo. Please make sure
>>>>>> you include Ron on any e-mail replies since he is not yet on the
>>>>>> OpenJDK aliases.
>>>>>>
>>>>>> Dan
>>>>>>
>>>>>>
>>>>>> Intro:
>>>>>>
>>>>>> This set of changes removes the makefile support for generation of
>>>>>> debug versions that follow _g semantics.
>>>>>>
>>>>>> Defect:
>>>>>>
>>>>>> 7153050 "remove crufty '_g' support from HotSpot repo"
>>>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7153050
>>>>>> https://jbs.oracle.com/bugs/browse/JDK-7153050
>>>>>>
>>>>>>
>>>>>> Webrev
>>>>>>
>>>>>> http://cr.openjdk.java.net/~dcubed/for_rdurbin/7153050-webrev/0
>>>>>>
>>>>>> Details:
>>>>>>
>>>>>> Many makefiles have been modified to remove all reference and
>>>>>> support
>>>>>> for debug versions that follow _g semantics.
>>>
>>
>
>
>
>
More information about the serviceability-dev
mailing list