<AWT Dev> RFR: 8136570: Avoid setting environment variables related to /usr/dt

Martin Buchholz martinrb at google.com
Wed Sep 16 17:53:55 UTC 2015


Webrev regenerated
http://cr.openjdk.java.net/~martin/webrevs/openjdk9/usr-dt-environment/usr-dt-environment.patch
and is now the best kind of change, a pure-deletion change.

On Wed, Sep 16, 2015 at 9:00 AM, Phil Race <philip.race at oracle.com> wrote:

> That is fine by me. If any one on awt-dev knows of a reason to keep it
> they should speak up.
>
> -phil.
>
>
> On 9/15/15 5:09 PM, Martin Buchholz wrote:
>
> We would be entirely happy if the environment frobbing code were to be
> deleted.
> Should I change my code to do that?
>
> On Tue, Sep 15, 2015 at 3:49 PM, Phil Race <philip.race at oracle.com> wrote:
>
>> I don't understand that original assessment.
>> Switching to XAWT had no impact on this code except to make it pointless.
>> i.e it did not prevent its execution.
>>
>> I doubt there is any code left in the JDK that will derive any
>> benefit from it still being there. All CDE/Motif & Xt related code is
>> gone.
>> This should have been removed along with it but was doubtless not
>> in plain sight to whoever did that.
>>
>> So unless I am missing something you could go further and just delete it.
>>
>> -phil.
>>
>>
>> On 9/15/2015 3:13 PM, Martin Buchholz wrote:
>>
>>> I reported this bug 12 years ago, but it was closed Will Not Fix, and
>>> this year I see java programs crashing because of it!
>>> This is a partial fix.
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8136570
>>> https://bugs.openjdk.java.net/browse/JDK-4953367
>>> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/usr-dt-environment/
>>> <
>>> http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/usr-dt-environment/
>>> >
>>>
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20150916/71b2a58f/attachment.html>


More information about the awt-dev mailing list