Review Request 119170: Include ntdd* headers to make Solid build without the Windows DDK

Nicolás Alvarez nicolas.alvarez at
Sun Feb 26 00:27:49 UTC 2017

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 25, 2017, 9:27 p.m.)


This change has been discarded.

Review request for kdewin and Solid.

Repository: solid


The `ntdd*.h` headers used in the Windows backend come with MSVC2013 and MinGW, but not with MSVC2010, you have to download and install the DDK separately, or grab the headers from MinGW or kdewin. I don't know if MSVC2012 has them or not.

I added the headers to Solid, in a subdirectory. There is a CMake check using `check_cxx_source_compiles` to see if the headers are present in the system. The bundled headers are only added to the include search paths if they aren't already found in the system. This is more robust than checking the MSVC version number.


  src/solid/devices/CMakeLists.txt 9271ae1 
  src/solid/devices/backends/win/kdewin/ntddcdrm.h PRE-CREATION 
  src/solid/devices/backends/win/kdewin/ntddmmc.h PRE-CREATION 
  src/solid/devices/backends/win/kdewin/ntddstor.h PRE-CREATION 



Builds on MSVC2010 (didn't before) and MSVC2013 (did before).


Nicolás Alvarez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the Kde-windows mailing list