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

Microsoft.VisualStudio.LanguageServices.DevKit.csproj attempted to copy files to output directory during design time build #74801

Closed
sharwell opened this issue Aug 19, 2024 · 1 comment · Fixed by #74812
Labels
Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead

Comments

@sharwell
Copy link
Member

sharwell commented Aug 19, 2024

Version Used: ee863ed

Steps to Reproduce:

Open Roslyn.sln in Visual Studio.

Expected Behavior:

Design time build does not attempt to copy files to output (bin) directory.

Actual Behavior:

image

The caller appears to be target _CopyDevKitExtensionFiles in Microsoft.CodeAnalysis.LanguageServer.UnitTests.csproj. Design time build is broken, leading to lack of IDE features during development.

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Aug 19, 2024
@tmeschter
Copy link
Contributor

There are two issues here:

  1. Since this is an error reported through the normal MSBuild mechanisms (as opposed to an uncaught exception or a missing target or something like that) it is reasonable to expect that it will be shown in the VS Error List.
  2. Regardless of the error reporting you probably don't want _CopyDevKitExtensionFiles to run during a design-time build. The definition of the target should be updated to include Condition="'$(DesignTimeBuild)' != 'true'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants