Skip to content
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

Added manually defined required compilations constants #208

Merged
merged 13 commits into from
Jul 20, 2022

Conversation

Arlodotexe
Copy link
Member

@Arlodotexe Arlodotexe commented Jul 19, 2022

  • Closes Uno Specific Preprocessor Directives don't work #191
  • Splits the properties for identifying the current platform into Labs.PlatformIdentifiers.props
  • Adds WINUI2 and WINUI3 as compilation constants, to supplement (and eventually phase out) the potentially confusing WINAPPSDK compilation constant which is only available when using WINUI 3, even under Uno.
  • Adds compilation constants for WINDOWS_UWP, NETFX_CORE, WINDOWS_WINAPPSDK, __WASM__, HAS_UNO_WASM, __ANDROID__, __IOS__, __MACOS__, HAS_UNO_SKIA, __SKIA__, WINDOWS_WPF, and __GTK__.

@Arlodotexe Arlodotexe added bug 🐛 Something isn't working dev loop ➰ For issues that impact the core dev-loop of building experiments labels Jul 19, 2022
@Arlodotexe Arlodotexe changed the title Manually defined required compilations conditionals Added manually defined required compilations conditionals Jul 20, 2022
@Arlodotexe Arlodotexe changed the title Added manually defined required compilations conditionals Added manually defined required compilations constants Jul 20, 2022
Copy link
Member

@michael-hawker michael-hawker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, let's do this thing! 🙈

@michael-hawker michael-hawker merged commit cf0b12a into main Jul 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/compilation-conditionals branch July 20, 2022 04:14
Martin1994 pushed a commit to Martin1994/Labs-Windows that referenced this pull request Sep 2, 2023
…kit#208)

* Moved project identifiers into a new props file

* Moved project identifiers into a new props file

* Merge branch 'fix/compilation-conditionals' of https://github.com/CommunityToolkit/Labs-Windows into fix/compilation-conditionals

* Added WINUI2 and WINUI3 compilation conditionals

* Added platform-specific compilation conditionals

* Added comment for props modified by script

* Added __SKIA__ compilation conditional

* Added Labs.ProjectIdentifiers.props import

* Fixed ProjectIdentifiers on project heads

* Fixed incorrect IsUno value

* Fixed BindableMetadataProvider error on WPF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working dev loop ➰ For issues that impact the core dev-loop of building experiments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Uno Specific Preprocessor Directives don't work
2 participants