[depends][xbmc][fix] Get rid of wdk requirement

[Paxxi]

Bump sdk to Win 10 as cmake defaults to that for addons anyway. Add the few headers we need from wdk as a package to simplify setup

[afedchin]

is there any benefits from sdk10? and why cmake uses it as default?

[Paxxi]

Not for us, no. I assume cmake just picks the newest, or runs vcvars.bat which defaults to newest. There's also no downside to using it. I was a bit confused when doing the 2015 upgrade and thought it's best to target the lowest possible. In hindsight this has only caused trouble with having to install an extra sdk.
This PR's not very technical but rather to make the dev setup on windows easier and less error-prone.

[afedchin]

thanks for clarifications

[Paxxi]

jenkins build this please

[garbear]

I still need WDK 8.1 installed to compile Kodi on Windows 7. The headers in mini_wdk-win32.7z seem to use structs and typedefs that aren't available on windows 7

[Paxxi]

If you have sdk 10.x installed it should be fine

[garbear]

Can you make a note of this on the wiki?

On Aug 4, 2016 10:30 PM, "Pär Björklund" notifications@github.com wrote:

If you have sdk 10.x installed it should be fine

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10143 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgcGi1xOoG8igI2ZL87Cgo4RszAFHJEks5qcsp7gaJpZM4JQB_L
.

[Paxxi]

I updated the wiki when this went in I think or maybe it was for another change but it does say to install sdk 10

[garbear]

lol you mean the big image that takes up the entire screen? my powers of observation are incredible :)

[Paxxi]

haha yeah that one :)