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

"Use apphost" checkbox missing from remodeled app designer for .NET console app project #7976

Closed
AArnott opened this issue Mar 14, 2022 · 5 comments
Assignees
Labels
Feature-Project-Properties-Designer The new project property pages which replace the legacy AppDesigner good-first-issue [up-for-grabs] Good for learning and for external contributors. Relatively self-contained. Resolution-By-Design The feature is working as it was designed. Triage-Approved Reviewed and prioritized
Milestone

Comments

@AArnott
Copy link
Contributor

AArnott commented Mar 14, 2022

Visual Studio Version

Dev17.2 Preview 3 (32309.498.main)

Summary

The app designer for a .NET Console Application has no discoverable way to disable the apphost that the project system spawns the app under (whether with Debug or Execute) as it used to.
This setting would control the UseAppHost msbuild property. It is useful for at least a few things, including working around dotnet/msbuild#7466.

Steps to Reproduce

  1. Create a new .NET Console Application
  2. Right-click on the project node in Solution Explorer and click Properties
  3. Use "search properties" to find "apphost", "host" or other variants. Or just look around every promising section manually.

Expected Behavior

Find a "Use apphost" checkbox.

Actual Behavior

No such checkbox to be found.

User Impact

Inability to workaround issues created by the app host.

@rainersigwald
Copy link
Member

I think it might be reasonable to have this checkbox but please talk to @baronfel about how interested the .NET SDK is to expose this prominently; most apps should accept the default (use an apphost so that you launch yourapp.exe instead of dotnet.exe yourapp.dll).

@drewnoakes
Copy link
Member

From https://docs.microsoft.com/en-us/dotnet/core/project-sdk/msbuild-props#useapphost is looks like we'd only want this option to be visible from .NET Core 3.0 onwards.

If @baronfel agrees with this request, it should be straightforward to add on our side.

@drewnoakes drewnoakes added Feature Request good-first-issue [up-for-grabs] Good for learning and for external contributors. Relatively self-contained. Feature-Project-Properties-Designer The new project property pages which replace the legacy AppDesigner labels Mar 15, 2022
@tmeschter tmeschter added the Triage-Approved Reviewed and prioritized label Mar 17, 2022
@tmeschter tmeschter added this to the 17.3 milestone Mar 17, 2022
@tmeschter
Copy link
Contributor

Adding the checkbox is easy; the hard part if figuring out if we should.

@baronfel
Copy link
Member

I would prefer not to expose this flag explicitly. The primary reason for this is that for the .NET 7 timeframe we are looking at making two changes that will make the use of AppHosts much more regular/predictable for end users based on their projects' use cases:

As a result, the apphost becomes an emergent property of the application's execution model instead of a top-level property.

@adamint adamint added the Resolution-By-Design The feature is working as it was designed. label Apr 15, 2022
@adamint
Copy link
Member

adamint commented Apr 15, 2022

Closing due to request not to complete this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature-Project-Properties-Designer The new project property pages which replace the legacy AppDesigner good-first-issue [up-for-grabs] Good for learning and for external contributors. Relatively self-contained. Resolution-By-Design The feature is working as it was designed. Triage-Approved Reviewed and prioritized
Projects
None yet
Development

No branches or pull requests

6 participants