Skip to content

Msvc/Arm: Pass /D_ARM_WINAPI_PARTITION_DESKTOP_SDK_AVAILABLE=1 to cl.exe #349

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 20, 2018
Merged

Msvc/Arm: Pass /D_ARM_WINAPI_PARTITION_DESKTOP_SDK_AVAILABLE=1 to cl.exe #349

merged 1 commit into from
Sep 20, 2018

Conversation

jordanrh1
Copy link
Contributor

This flag must be defined to avoid a compilation error.

@alexcrichton
Copy link
Member

Thanks! Is there perhaps some documentation we could point to to say why we're passing this flag?

@jordanrh1
Copy link
Contributor Author

The documentation is the source code :) From corecrt.h:

// Verify that the ARM Desktop SDK is available when building an ARM Desktop app
#ifdef _M_ARM
    #if _CRT_BUILD_DESKTOP_APP && !_ARM_WINAPI_PARTITION_DESKTOP_SDK_AVAILABLE
        #error Compiling Desktop applications for the ARM platform is not supported.
    #endif
#endif

The check was added in Windows 8 days because only store apps were allowed on ARM, but this changed with the release of Windows 10 IoT Core. This check will be going away in future versions of the SDK, but for all released versions of the Windows SDK it is required.

@alexcrichton
Copy link
Member

Ok cool! Could you add that all in a comment next to the source here?

@jordanrh1
Copy link
Contributor Author

Sure, done.

@alexcrichton alexcrichton merged commit ee7b97e into rust-lang:master Sep 20, 2018
@alexcrichton
Copy link
Member

Thanks!

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.

2 participants