Update Windows build readme to match current VS2017 and java installers

[matthuisman]

Description

Few minor changes I noticed today as building Kodi

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of change

• Bug fix (non-breaking change which fixes an issue)
• Clean up (non-breaking change which removes non-working, unmaintained functionality)
• Improvement (non-breaking change which improves existing functionality)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that will cause existing functionality to change)
• Cosmetic change (non-breaking change that doesn't touch code)
• None of the above (please explain below)

Checklist:

• My code follows the Code Guidelines of this project
• My change requires a change to the documentation, either Doxygen or wiki
• I have updated the documentation accordingly
• I have read the Contributing document
• I have added tests to cover my change
• All new and existing tests passed

[phunkyfish]

Thanks @matthuisman

[matthuisman]

ooops, deleted this branch by accident but then restored

[Paxxi]

Looks good to me except for the comment. If that is tested then go ahead and merge.

Thanks @matthuisman 😀

[Paxxi]

Did you close this by accident again @matthuisman ?

[phunkyfish]

I’m going to assume a yes 😉

[matthuisman]

After the comments - i wasnt actually sure if those arm libraries where needed or not.
So maybe we just keep them as is.

I never even opened VS. I just followed the build commands and did it in command prompt.
There isn't anything there about using the .sln to build (also nothing about binary addons?)

All I know at moment is I didn't need them for the below to make a working kodi.exe
cmake -G "Visual Studio 15 Win64" -T host=x64 %userprofile%\kodi
cmake --build . --config "Release"

I'll try run a BuildSetup.bat now without them and see if that completes
Nope, Unresolved external symbol dav1d_picture_unref referenced in function libdav1d

What else is interesting, when I did my first build of kodi.exe and copied into nightly directory, the nightly didn't have libdvdnav.dll (needed to copy that over). So not sure what is happening there.

Also, the docs make it sound like the kodi.exe should just run in it's own folder.
But it doesn't - just a bunch of missing dlls.
So maybe more of a docs overhaul needed

Should I update this PR to remove the (if compiling for ARM) notes?

[Paxxi]

I think either remove it or add the suggestions I made to change it to (if not compiling arm or uwp). You should be able to accept the edits right on the page under the changes tab.

You don't have to test it, I tested it myself yesterday

[matthuisman]

done :)

[Paxxi]

Thanks again 😀