From 0ca3d32c82a74a1ec9db3f0519a32493539ae1b6 Mon Sep 17 00:00:00 2001
From: Martyn Gigg <martyn.gigg@stfc.ac.uk>
Date: Sat, 8 Dec 2018 09:50:50 +0000
Subject: [PATCH] Add more detail to developer docs r.e. VS2017 setup

---
 dev-docs/source/GettingStarted.rst       | 4 ++++
 dev-docs/source/JenkinsConfiguration.rst | 5 +++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/dev-docs/source/GettingStarted.rst b/dev-docs/source/GettingStarted.rst
index e5e48eafce7..3de44933d1c 100644
--- a/dev-docs/source/GettingStarted.rst
+++ b/dev-docs/source/GettingStarted.rst
@@ -21,6 +21,10 @@ Install the following:
 * `Visual Studio 2017 Community Edition <https://visualstudio.microsoft.com/downloads/>`_.
 
   * When asked about installation workloads choose ``Desktop development with C++``
+  * Under the "Installation details" section verify that the following are checked:
+
+    * ``Windows 8.1 SDK and UCRT SDK``
+    * The latest Windows 10 SDK
 
 
 * `Git <https://git-scm.com/>`_.
diff --git a/dev-docs/source/JenkinsConfiguration.rst b/dev-docs/source/JenkinsConfiguration.rst
index ddc5cb25736..6695beb8daa 100644
--- a/dev-docs/source/JenkinsConfiguration.rst
+++ b/dev-docs/source/JenkinsConfiguration.rst
@@ -43,8 +43,9 @@ Set up a local ``builder`` account that will be used by the slave.
 Install the :ref:`required prerequisites <GettingStarted>` for the relevant OS.
 
 .. note::
-   For Windows the `Command line Visual C++ build tools <http://landinghub.visualstudio.com/visual-cpp-build-tools>`__
-   may be used in place of a full Visual Studio install from version 2017 onwards (the 2015 tools contain a broken `vcvarsall.bat`).
+   For Windows the `Command line Visual C++ build tools <https://visualstudio.microsoft.com/downloads/>`__
+   may be used in place of a full Visual Studio install from version 2017 onwards (the 2015 tools contain a broken `vcvarsall.bat`). The same
+   options should be used as for the full install.
 
 Windows
 -------
-- 
GitLab