Page MenuHome

Cleanup: Windows Manifest usage.
ClosedPublic

Authored by Ray Molenkamp (LazyDodo) on Oct 27 2019, 1:19 AM.

Details

Summary

We had a manifest file, but it was seemingly not used, some settings were done using linker pargmas, some of them visual studio would set by default for us, others where not set at all.

This patch changes:

  1. Single manifest file where we can maintain all settings in a single location, removal of any linker pragmas
  1. Compatibly settings for win vista - win10, without this any call to any of the GetVersion and related functions (GetVersionEx, VerifyVersionInfo, IsWindowsXxxx) will by default say we are on vista and OS specific optimizations in external shared libraries may be missed.
  1. Rather than having it in the RC file in an #ifdef which may or may not trigger depending on the build tool used, we tell cmake to treat it as a source file and it will do the right thing for both the ninja and visual studio generators.

Diff Detail

Repository
rB Blender

Event Timeline

@Brecht Van Lommel (brecht) Out of curiosity is 2 the reason you ended up importing RtlGetVersion in cycles?

@Brecht Van Lommel (brecht) Out of curiosity is 2 the reason you ended up importing RtlGetVersion in cycles?

For that Cycles test we needed to know the actual Windows version. We'll probably remove that soon anyway since NVIDIA fixed their drivers.

This revision is now accepted and ready to land.Dec 4 2019, 1:36 AM
This revision was automatically updated to reflect the committed changes.