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

Log location of intrinsic tasks #6397

Merged
merged 2 commits into from
May 28, 2021
Merged

Conversation

KirillOsenkov
Copy link
Member

@KirillOsenkov KirillOsenkov commented May 1, 2021

Log the Line and Column of add or remove items in ItemGroups inside targets. Previously we could only recover the file from the parent target, but not exactly which line in the file corresponded to the add/remove.

Now we can exactly pinpoint which add/remove is executing. No need to log the file because we can recover this information from the parent target. This saves space.

IntrinsicTaskLocation2

Log the Line and Column of add or remove items in ItemGroups inside targets. Previously we could only recover the file from the parent target, but not exactly which line in the file corresponded to the add/remove.

Now we can exactly pinpoint which add/remove is executing. No need to log the file because we can recover this information from the parent target. This saves space.
@KirillOsenkov KirillOsenkov added Area: Logging Area: Debuggability Issues impacting the diagnosability of builds, including logging and clearer error messages. labels May 1, 2021
@KirillOsenkov
Copy link
Member Author

Note that this doesn't solve #2473, because it logs only during target execution. Evaluation ItemGroups are not logged.

@ladipro ladipro added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label May 10, 2021
@Forgind Forgind merged commit 8861fa0 into main May 28, 2021
@KirillOsenkov KirillOsenkov deleted the dev/kirillo/intrinsicTaskLocation branch May 29, 2021 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Debuggability Issues impacting the diagnosability of builds, including logging and clearer error messages. Area: Logging merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants