Skip to content

Build fix 20200222#11150

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-20200222
Closed

Build fix 20200222#11150
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-20200222

Conversation

@levitte
Copy link
Copy Markdown
Member

@levitte levitte commented Feb 22, 2020

A couple of incorrect things went through in merges of diverse pieces of code.

URGENT! We have builds failing because of this

@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer severity: urgent Fixes an urgent issue (exempt from 24h grace period) labels Feb 22, 2020
Copy link
Copy Markdown
Member

@slontis slontis left a comment

Choose a reason for hiding this comment

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

Take it back - travis is still failing

Property "default" no longer exists, replace "default=yes" with
"provider=default"
@levitte
Copy link
Copy Markdown
Member Author

levitte commented Feb 23, 2020

Turns out I overdid this PR, one fix cancelled out the other error. It's now been reduced to something that should work

@paulidale
Copy link
Copy Markdown
Contributor

Travis timeout isn't relevant.
Merged to master.

@paulidale paulidale closed this Feb 24, 2020
@paulidale paulidale added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Feb 24, 2020
openssl-machine pushed a commit that referenced this pull request Feb 24, 2020
Property "default" no longer exists, replace "default=yes" with
"provider=default"

Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from #11150)
@levitte levitte deleted the fix-20200222 branch February 24, 2020 05:08
@paulidale
Copy link
Copy Markdown
Contributor

Ah, the PING! was deleted :)
I was feeling confused.

@levitte
Copy link
Copy Markdown
Member Author

levitte commented Feb 24, 2020

Yeah, I got confused because the tab I had this in wasn't updated before I posted...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch severity: urgent Fixes an urgent issue (exempt from 24h grace period)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants