Should static graph respect MSBuild.SkipNonExistentProjects and MSBuild.SkipNonExistentTargets?

[cdmihai]

In msbuild, one can call targets which do not exist by setting SkipNonexistentTargets on the MSBuild task.
The static graph target protocol does not support this and will call the nonexistent targets which results in a build failure.

A possible implementation would be to support <ProjectReferenceTargets Include='Build' Targets='GetTargetFrameworks' SkipNonexistentTargets='true'> which would instruct the target propagation code to take out non existing targets.

[rainersigwald]

I'd say "no".

[cdmihai]

Unfortunately we encountered some repos depending on skipping nonexistent targets. A possible implementation was attempted in #5297, but the PR has to be revived.

[rainersigwald]

I'd say "no".

I now think this was wrong, especially while we don't have #6852.

[AArnott]

I would add that the parser should honor SkipGetTargetFrameworkProperties=true by no longer executing the GetTargetFrameworks target.

See dotnet/Nerdbank.GitVersioning#648 (comment) for context.