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

Race condition in usage of GenerateAssemblyInfo can cause test build breaks #27801

Closed
MattGal opened this issue Nov 2, 2018 · 3 comments · Fixed by dotnet/corefx#33224
Closed

Comments

@MattGal
Copy link
Member

MattGal commented Nov 2, 2018

@dotnet-mc-bot commented on Fri Nov 02 2018

There were a set of failures during this build. Here is a summary of these:

@jcagme, @markwilkie


@MattGal commented on Fri Nov 02 2018

Reclassifying as Source Code and moving to CoreFX. We've seen lots of these parallel build issues and it usually comes from slight build tweaks (or just a latent bug and bad luck)

@MattGal
Copy link
Member Author

MattGal commented Nov 2, 2018

@ViktorHofer FYI, I've definitely seen you chase down and fix similar issues.

@MattGal MattGal changed the title Misc (SourceCode) build failures in 'official/corefx/master/' - '20181102.05' Race condition in usage of GenerateAssemblyInfo can cause test build breaks Nov 2, 2018
@ViktorHofer
Copy link
Member

cc @weshaggard who brought the generate assembly info infrastructure over to corefx.

@weshaggard
Copy link
Member

Looks like we have 2 assemblies with the same name which is causing this issue (https://github.com/dotnet/corefx/search?q=ForwardedTypesAssembly&unscoped_q=ForwardedTypesAssembly) We need to rename one or both of those projects to fix this race.

@msftgits msftgits transferred this issue from dotnet/corefx Jan 31, 2020
@msftgits msftgits added this to the 3.0 milestone Jan 31, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants