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

Evaluation missing from Design-Time build logs #256

Open
KirillOsenkov opened this issue Jan 13, 2020 · 5 comments
Open

Evaluation missing from Design-Time build logs #256

KirillOsenkov opened this issue Jan 13, 2020 · 5 comments

Comments

@KirillOsenkov
Copy link
Member

Evaluation is probably happening separately from design-time build, and as a result we are missing all evaluation events, and none of the files are embedded in the .binlog.

I'm filing this issue here to see if we can think of a workaround, perhaps somehow correlating the evaluation events with a design-time build that's based on a ProjectInstance.

This may be related to dotnet/msbuild#4926

@KirillOsenkov
Copy link
Member Author

Corresponding binlog viewer issue: KirillOsenkov/MSBuildStructuredLog#250

@KirillOsenkov
Copy link
Member Author

I found the related issue: #50

@panopticoncentral
Copy link
Contributor

If I recall correctly, we actually do get some evaluation events for non-SDK projects because they are in the global project collection, and we can hook them there. SDK projects, though, don't use the global project collection and we need some sort of CPS API to hook into evaluations. That would also give us the imports as well. It's been a todo on my list that I've been wanting to get back to, so I'll try to make some time for that.

@KirillOsenkov
Copy link
Member Author

@AArnott mentions:

The dataflow in CPS from evaluation to design-time builds is straightforward and I can certainly imaging plumbing something through to correlate the two. But they are distinctly separate operations, so I'm not sure how the "log" would be passed or concatenated to.

Keep in mind that the ProjectInstance can be mutated between evaluation and build. But for most purposes that can probably be ignored as it's rare (last I knew) and probably not done for things the user wants to know about.

@sharwell
Copy link
Member

sharwell commented Jun 2, 2020

I just hit this one today in 1.0.1.2010501

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants