-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Apparent race condition copying files from sync.cmd #26858
Comments
@ViktorHofer these failures look related to your binplacepackages.depproj changes from dotnet/corefx#30897.
I've seen a number of official builds and CI legs hitting this randomly. @ericstj it seems like it is copying over the dependencies or something from all the different configurations. Which is racing with other depproj's like the netstandard one. |
All configurations CI leg in dotnet/corefx#31160 failed https://ci3.dot.net/job/dotnet_corefx/job/master/job/windows-TGroup_all+CGroup_Debug+AGroup_x64+TestOuter_false_prtest/14278/ with similar issue. |
Hmm. It seems the assembly references of System.Memory race with the other depproj files. This only happens for the following configurations as other configuration don't have assembly references for System.Memory:
Looking further... |
IIRC our depproj's shouldn't be doing a deep copy and should only copy the direct PackageReferences but that doesn't seem to be happening. |
Right, I just checked and |
I just submitted dotnet/corefx#31167 to fix the deep copy for binplacepackages and harvestpackages projects which shouldn't be deep. |
Opened on behalf of @MattGal
Possibly related to these recent PRs?
dotnet/corefx#31097
dotnet/corefx#31139
(similar symptoms, BuildVersion.props called out in one log)
Warnings: 1
Status Message: failed
Build : Master - 20180718.04 (Product Build)
Failing configurations:
Mission Control Build Info
The text was updated successfully, but these errors were encountered: