Avoid extra copies of dotnet-watch build outputs in the SDK directory

[tmat]

DotNetWatch.targets was copied to the output dir of redist.csproj and from there to multiple other directories where they shouldn't be.

[tmat]

@dsplaisted PTAL. I wonder if other ProjectReferences in redist.csproj should also be Private=false.

[tmat]

@javiercn

[tmat]

Assuming the shipping layout is the same as in redist/dotnet directory the change affects it. It makes the SDK smaller by removing the duplicate files from the layout. There is logic in dotnet-watch to load Roslyn compiler assemblies from a common path shared with other tools. This logic is currently defeated by copying the binaries to the output directory.

If you think this is too risky I have no problem retargeting it to main.

[marcpopMSFT]

I think this is probably too risky at this point. All RC2 changes now require approval and the RTM branch won't have any more previews. We'll have a 7.0.2xx branch open in the coming weeks or main should be fine.

[tmat]

Moving to main.

[tmat]

/azp run

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).

[tmat]

@dsplaisted Rebased to 7.0.2xx