dotnet/msbuild

Preprocess flag allows some irrelevant logging to escape

Open

#1,799 opened on 2017年3月6日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)C# (5,062 stars) (1,364 forks)batch import
Area: Debuggabilityhelp wantedtriaged

説明

<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
    <Target Name="Build" BeforeTargets="bt" AfterTargets="at"/>  
</Project>

msbuild /pp:out.txt test.proj

results in

C:\git\corefx>msbuild /pp:out.txt test.proj
The target "bt" listed in a BeforeTargets attribute at "C:\git\corefx\test.proj (3,26)" does not exist in the project,
and will be ignored.
The target "at" listed in an AfterTargets attribute at "C:\git\corefx\test.proj (3,45)" does not exist in the project,
and will be ignored.
C:\git\corefx>

When using /preprocess, no logging is expected except for cases where /preprocess cannot do its job (for example syntax error or import of nonexistent file)

コントリビューターガイド

技術スタック
csharp
領域
build systemtooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
basic knowledge of MSBuildunderstanding of preprocess flag
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50
調査方針
Investigate the preprocessing code path in MSBuild to identify where logging messages are emitted for non existent targets. The issue is that warnings for BeforeTargets/AfterTargets should be suppressed when /pp (preprocess) flag is used. Look at the ProjectEvaluation or preprocessing logic in the source code (likely under src/Build/ or src/MSBuild/). A fix might involve adding a condition to check if preprocessing is enabled before logging those warnings. The issue has no comments, so the fix can be straightforward.