Skip to content

Fix build parallelism issue with documentation file generation#22232

Merged
nguerrera merged 1 commit into
dotnet:masterfrom
nguerrera:fix-doc-xml-build-race
Sep 25, 2017
Merged

Fix build parallelism issue with documentation file generation#22232
nguerrera merged 1 commit into
dotnet:masterfrom
nguerrera:fix-doc-xml-build-race

Conversation

@nguerrera
Copy link
Copy Markdown
Contributor

Use GenerateDocumentationFile=true instead of DocumentationFile=$(IntermediateOutputPath)$(AssemblyName).xml. This lets the SDK pick the appropriate path.

At the point where the explicit DocumentationFile was being evaluated, the intermediate output path had not yet gotten the TargetFramework appended. As such, parallel builds of the different TargetFrameworks of a multi-targeted project were trying to write the documentation xml to the same intermediate location.

Use GenerateDocumentationFile=true instead of DocumentationFile=
$(IntermediateOutputPath)$(AssemblyName).xml. This lets the SDK
pick the appropriate path.

At the point where the explicit DocumentationFile was being evaluated,
the intermediate output path had not yet gotten the TargetFramework
appended. As such, parallel builds of the different TargetFrameworks
of a multi-targeted project were trying to write the documentation xml
to the same intermediate location.
@nguerrera
Copy link
Copy Markdown
Contributor Author

https://ci.dot.net/job/dotnet_roslyn/job/master/job/windows_release_unit32_prtest/9614/

"xunit produced no error output but had exit code -2146232797"

Looks unrelated. Different intermittent issue?

@jasonmalinowski
Copy link
Copy Markdown
Member

@nguerrera: yeah, probably a different issue. 😦

@jaredpar
Copy link
Copy Markdown
Member

retest windows_release_unit32_prtest please

@jaredpar
Copy link
Copy Markdown
Member

retest windows_release_unit64_prtest please

@nguerrera nguerrera merged commit 49f9176 into dotnet:master Sep 25, 2017
@nguerrera nguerrera deleted the fix-doc-xml-build-race branch September 25, 2017 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants