In [JDK-8344030](https://bugs.openjdk.org/browse/JDK-8344030) the configure 
option --with-tools-dir was deprecated. There are however still references to 
it in the configure script. Specifically toolchain_microsoft is still using the 
variable it was setting up, and there are error messages telling the user to 
try using it.

Digging further into this, the way we use `--with-tools-dir` to find Visual 
Studio is quite convoluted. It used to be that you would point to something 
like ".../Community/VC/bin" or ".../Community/VC/bin/amd64", which would make 
sense as those were directories with the compiler executables. Configure would 
then walk up two or three levels to find the root of the Visual Studio 
installation. Modern Visual Studio installations have a much deeper and more 
complex directory structure, so this doesn't work as it used to. However, 
because the configure logic just checks two and three levels above the 
directory given to `--with-tools-dir`, you can actually supply any existing 
directory inside the installation at the appropriate depth, e.g. 
".../Community/VC/Auxiliary/Build".

This really isn't a great situation and the proper fix would be some kind of 
overhaul, probably a new explicit configure parameter pointing to the Visual 
Studio root to use. However, I don't have the time to invest in such a big 
rewrite at this time, so I will limit this bug to just restoring 
--with-tools-dir for it's current functionality on Windows, with a help text 
that describes what it currently does. This will at least let users currently 
relying on this option to continue building with Visual Studio in a nonstandard 
location.

-------------

Commit messages:
 - JDK-8372943

Changes: https://git.openjdk.org/jdk/pull/28618/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28618&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8372943
  Stats: 6 lines in 1 file changed: 5 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/28618.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28618/head:pull/28618

PR: https://git.openjdk.org/jdk/pull/28618

Reply via email to