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

Generator types are sometimes being loaded twice #63189

Closed
chsienki opened this issue Aug 3, 2022 · 2 comments
Closed

Generator types are sometimes being loaded twice #63189

chsienki opened this issue Aug 3, 2022 · 2 comments

Comments

@chsienki
Copy link
Contributor

chsienki commented Aug 3, 2022

As seen here: #63062 we're hitting an issue where the generator driver is receiving multiple copies of the same generator.

Looking at the binlog, the specific generator is only passed once, so it appears to be an issue with how we're instantiating the generators. It also seems to be somewhat non deterministic when this is happening.

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Issues and PRs which have not yet been triaged by a lead label Aug 3, 2022
@jaredpar
Copy link
Member

jaredpar commented Aug 30, 2022

Dug into this issue this morning after @JoeRobich, @RikkiGibson saw this hit again on the servicing branch.

Looking at the binlog, the specific generator is only passed once, so it appears to be an issue with how we're instantiating the generators.

This is not the case. It is definitely being passed more than once.

image

At the time of this bug at least three of us confirmed we weren't passing the generator twice into the binlog. I think the reason why is because prior to #63236 the binlog had a lot more failures. The build essentially limited along after a bootstrap failure and we ended up with lots of failed compilations. After #63236 though we essentially fail fast on the build and are left with a minimum amount of failed compilations to dig through. It makes the picture a lot clearer.

Now with the clearer picture it's easy to see this build is one with duplicate copies of the Json generator: a WPF temp project.

image

That means we're really just hitting the following bug in our build.

dotnet/wpf#6680

@jaredpar jaredpar added Area-External and removed untriaged Issues and PRs which have not yet been triaged by a lead Area-Analyzers Area-Compilers labels Aug 30, 2022
@jaredpar jaredpar added this to the 17.4 milestone Aug 30, 2022
@RikkiGibson
Copy link
Contributor

The external issue has been resolved.

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

No branches or pull requests

3 participants