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

Update OutputType to WinExe on -windows #2340

Merged
merged 2 commits into from
Sep 1, 2021
Merged

Conversation

Redth
Copy link
Member

@Redth Redth commented Aug 31, 2021

This happens in the sdk targets/props but not quite early enough to influence the EnableWin32Codegen property which causes [STAThreadAttribute] to be generated in the Windows app.

Also: For non-blazor projects, interpreter is fine
Blazor android projects (and possibly iOS?) will require this runtime fix: dotnet/runtime#58467
So, enabling interpreter again on non-blazor templates

This happens in the sdk targets/props but not quite early enough to influence the `EnableWin32Codegen` property which causes `[STAThreadAttribute]` to be generated in the Windows app.
@Redth Redth requested a review from Eilon as a code owner August 31, 2021 23:32
Copy link
Member

@mattleibow mattleibow left a comment

Choose a reason for hiding this comment

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

Might be good to remove the prop from the targets file to avoid confusion.

Blazor android projects (and possibly iOS?) will require this runtime fix: dotnet/runtime#58467

So, enabling interpreter again on non-blazor templates
@rmarinho rmarinho merged commit 2c01265 into release/6.0.1xx-rc1 Sep 1, 2021
@rmarinho rmarinho deleted the winui-winexe branch September 1, 2021 09:40
Csaba8472 added a commit to Csaba8472/maui that referenced this pull request Sep 6, 2021
* upstream/main: (326 commits)
  Re-enable interpreter on blazor by default
  Determine size of auto cells in GridLayout correctly when views occupy same cell (dotnet#2391)
  [main] Update dependencies from xamarin/xamarin-macios (dotnet#2326)
  [Android] Fix Horizontal Scrollbar Visibility (dotnet#2374)
  Try start vs4mac with local dotnet (dotnet#2386)
  [workoad] include missing dependencies in the workload (dotnet#2281)
  Update project templates to work with safe characters (dotnet#2368)
  [release/6.0.1xx-rc1] Update dependencies from xamarin/xamarin-android (dotnet#2369)
  Update dependencies from https://github.com/xamarin/xamarin-macios build 20210903.3 (dotnet#2384)
  Bump manifest to use api 31 (dotnet#2366)
  BoxView Handlers (dotnet#2250)
  Implement IsTextPredictionEnabled in WinUI Editor (dotnet#1478)
  Implement PlaceholderColor property in EntryHandlers (dotnet#1513)
  Automated dotnet-format update (dotnet#2355)
  Show entire search bar by default and make it not collapse (dotnet#2351)
  Remove test cloud if def for now (dotnet#2350)
  Add support for JS root components in BlazorWebView (dotnet#2293)
  Set base viewmapper to work against IViewHandler (dotnet#2339)
  NavigationView Handler for Android (dotnet#2336)
  Update OutputType to WinExe on -windows (dotnet#2340)
  ...
@Eilon Eilon added the area-templates Project templates, Item Templates for Blazor and MAUI label Sep 16, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Dec 22, 2023
@samhouts samhouts added the fixed-in-6.0.100-rc.1.7 Look for this fix in 6.0.100-rc.1.7! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-templates Project templates, Item Templates for Blazor and MAUI fixed-in-6.0.100-rc.1.7 Look for this fix in 6.0.100-rc.1.7!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants