Skip to content

Conversation

@strseb
Copy link
Collaborator

@strseb strseb commented Mar 5, 2025

Windows lies about the OS version to applications for compatibility
unless weexplicitly declare tested versions via manifest. Without
maxversiontested, we get fake version numbers.

Before:

before

After:

after

@strseb strseb requested a review from oskirby March 5, 2025 13:03
@strseb strseb force-pushed the basti/add_manifest branch from 8a0cbf2 to 5446ffb Compare March 5, 2025 17:23
@strseb strseb marked this pull request as draft March 6, 2025 15:39
@strseb strseb force-pushed the basti/add_manifest branch 2 times, most recently from f69d72f to 7fc918e Compare January 13, 2026 16:22
@strseb strseb changed the title Add a Manifest adding Supportinfo to .exe (Windows) Add manifest for accurate OS version detection Jan 13, 2026
Windows lies about the OS version to applications for compatibility
unless they explicitly declare tested versions via manifest. Without
maxversiontested, the app gets fake version numbers.

Changes:
- Add windows/vpn.manifest with maxversiontested for Windows 11 24H2
- Declare requestedExecutionLevel=asInvoker (GUI runs unprivileged)
- Enable PerMonitorV2 DPI awareness for proper high-DPI support
- Simplify msvc.cmake to only apply MSVC-specific flags to actual MSVC

The MSVC-only flags (/ZI for edit-and-continue) don't work with clang-cl,
so we now explicitly check for MSVC instead of excluding clang.
@strseb strseb force-pushed the basti/add_manifest branch from 7fc918e to 31020ce Compare January 13, 2026 17:35
@strseb strseb marked this pull request as ready for review January 13, 2026 18:06
@strseb strseb requested a review from artfwo January 14, 2026 14:37
@strseb strseb merged commit c6092ae into main Jan 14, 2026
150 of 152 checks passed
@strseb strseb deleted the basti/add_manifest branch January 14, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants