RFR: JDK-8068735 Configure fails on Windows if Visual Studio $LIB/$INCLUDE is lower case
Erik Joelsson
erik.joelsson at oracle.com
Fri Jan 9 15:44:50 UTC 2015
Looks ok to me.
/Erik
On 2015-01-09 16:41, Magnus Ihse Bursie wrote:
> When detecting the Visual Studio environment, we check for the
> variables $LIB and $INCLUDE. However, on Windows, environment
> variables are not case sensitive.
>
> Most of the time, these are named $LIB and $INCLUDE, but sometimes
> they are named $lib and $include instead. Currently, we fail if this
> is the case.
>
> I made a simple fix to cover this case. Ideally, we'd like to have
> proper, full support of case insensitivity. Unfortunately, that is
> hard to fix and the current lines are the result of much blood and
> tears. I'm not very much inclined to try to "improve" them.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8068735
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8068735-support-lowercase-vs-env/webrev.01
>
> /Magnus
More information about the build-dev
mailing list