Skip to content

build: work around bug in MSBuild v16.10.0 - #38873

Merged
aduh95 merged 1 commit into
nodejs:masterfrom
targos:tmp-fix-msbuild
May 31, 2021
Merged

build: work around bug in MSBuild v16.10.0#38873
aduh95 merged 1 commit into
nodejs:masterfrom
targos:tmp-fix-msbuild

Conversation

@targos

Copy link
Copy Markdown
Member

This allows to build Node.js at the (temporary) cost of longer build
times.

Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373

@github-actionsgithub-actionsBot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. windows Issues and PRs related to the Windows platform. labels May 31, 2021
@targostargos added the fast-track PRs that do not need to wait for 48 hours to land. label May 31, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @targos. Please 👍 to approve.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@targos

Copy link
Copy Markdown
MemberAuthor

Requesting fast-track because GitHub actions fail permanently since they upgraded their runners today.

This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: nodejs#38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: nodejs#38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
@aduh95

Copy link
Copy Markdown
Contributor

Landed in 3457130

@aduh95
aduh95 merged commit 3457130 into nodejs:masterMay 31, 2021
@targos
targos deleted the tmp-fix-msbuild branch June 1, 2021 05:06
danielleadams pushed a commit that referenced this pull request Jun 1, 2021
This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: #38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
@danielleadamsdanielleadams mentioned this pull request Jun 1, 2021
targos added a commit that referenced this pull request Jun 5, 2021
This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: #38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
targos added a commit that referenced this pull request Jun 5, 2021
This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: #38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
targos added a commit that referenced this pull request Jun 5, 2021
This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: #38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
@targostargos mentioned this pull request Jun 6, 2021
targos added a commit that referenced this pull request Jun 11, 2021
This allows to build Node.js at the (temporary) cost of longer build
times.
Refs: #38872
Refs: https://github.com/dotnet/msbuild/releases/tag/v16.10.0
Refs: dotnet/msbuild#6465
Refs: dotnet/msbuild#6373
PR-URL: #38873
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.fast-trackPRs that do not need to wait for 48 hours to land.needs-ciPRs that need a full CI run.windowsIssues and PRs related to the Windows platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@targos@nodejs-github-bot@aduh95@lpinca@Flarna@bl-ue