Skip to content

Comments

Add windows build conditions#14595

Closed
PureWeen wants to merge 4 commits intomainfrom
windows_targets_updates
Closed

Add windows build conditions#14595
PureWeen wants to merge 4 commits intomainfrom
windows_targets_updates

Conversation

@PureWeen
Copy link
Member

Description of Change

If a project with UseMaui has UseForms or UseWPF specified, then don't include the WinAppSDK bits.

I tested this locally and for WPF you have to include these in your project so that we don't process the WPF XAML

    <EnableDefaultXamlItems>false</EnableDefaultXamlItems>
    <EnableDefaultMauiItems>false</EnableDefaultMauiItems>

I feel like this is ok for now. I didn't want to add too much conditional behavior in our code.

Issues Fixed

Fixes #6450

@mattleibow
Copy link
Member

Can we use windows7? net7.0-windows7?

That does not super the wasdk and we can also have that work there?

@PureWeen PureWeen marked this pull request as draft April 16, 2023 16:11
@PureWeen
Copy link
Member Author

I'm closing this PR

From what I can tell this is actually fixed on main by @mattleibow 's PR to nugetize things.

I tested the net7.0 branch with the following PR and the packs produced from that PR are able to create WPF/Forms projects that live in harmony with MAUI

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Windows] Can't include MAUI and WinForms in the same project

3 participants