Mostly moot, now that we have consensus on just deleting these entirely, but my (original) code was safer because it calls putenv in fewer cases. Who knows what /usr/dt is "supposed" to look like these days? "Common" Desktop Environment not so "common" anymore...
On Wed, Sep 16, 2015 at 1:33 AM, Thomas Stüfe <thomas.stu...@gmail.com> wrote: > Hi Martin, > > would it be not sufficient to just test for existence of /usr/dt before > the two calls to setPathEnvironment() and leave the rest as it is? I also > think the intend would be clearer ("only call this if CDE is installed"). > > Kind Regards, Thomas > > > > On Wed, Sep 16, 2015 at 12:13 AM, Martin Buchholz <marti...@google.com> > 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/ >> > >