Skip to content

[gtest] Add a patch to fix build error in GCC 11#15661

Merged
dan-shaw merged 3 commits intomicrosoft:masterfrom
PragmaTwice:fix-gtest
Jan 15, 2021
Merged

[gtest] Add a patch to fix build error in GCC 11#15661
dan-shaw merged 3 commits intomicrosoft:masterfrom
PragmaTwice:fix-gtest

Conversation

@PragmaTwice
Copy link
Copy Markdown
Contributor

@PragmaTwice PragmaTwice commented Jan 15, 2021

Describe the pull request

  • What does your PR fix?
    Fixes [gtest] build failure while using GCC 11 (trunk) #15617.
    To summarize, gtest 1.10.0 failed to compile in GCC 11, and no other versions will be released due to their policy (live in head, no release tags), so a patch is the only way to solve it.

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    No change.

  • Does your PR follow the maintainer guide?
    Yes.

@vicroms
Copy link
Copy Markdown
Member

vicroms commented Jan 15, 2021

/azp run

@microsoft microsoft deleted a comment from azure-pipelines bot Jan 15, 2021
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY JackBoosY added the category:port-bug The issue is with a library, which is something the port should already support label Jan 15, 2021
Copy link
Copy Markdown
Contributor

@NancyLi1013 NancyLi1013 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also bump Port-Version as 3 in CONTROL file?

@NancyLi1013 NancyLi1013 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Jan 15, 2021
@NancyLi1013
Copy link
Copy Markdown
Contributor

LGTM now, thanks for your PR @PragmaTwice.

@dan-shaw dan-shaw merged commit 6df7d66 into microsoft:master Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[gtest] build failure while using GCC 11 (trunk)

5 participants