Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[openssl] Remove unused patches #14477

Merged
merged 2 commits into from
Nov 11, 2020
Merged

[openssl] Remove unused patches #14477

merged 2 commits into from
Nov 11, 2020

Conversation

NancyLi1013
Copy link
Contributor

@NancyLi1013 NancyLi1013 commented Nov 9, 2020 •
edited
Loading

Describe the pull request

Remove unused patches

ports/openssl/unix/ConfigureIncludeQuotesFix.patch
ports/openssl/unix/EmbedSymbolsInStaticLibsZ7.patch
ports/openssl/unix/STRINGIFYPatch.patch
ports/openssl/uwp/fix-uwp-configure-unicode.patch

These patches has been deprecated in version 1.1.1d via PR #8566

Note: No feature needs to test.

@NancyLi1013 NancyLi1013 added info:internal This PR or Issue was filed by the vcpkg team. category:port-update The issue is with a library, which is requesting update new revision labels Nov 9, 2020
@NancyLi1013 NancyLi1013 mentioned this pull request Nov 9, 2020
Closed
@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Nov 9, 2020
@Neumann-A
Copy link
Contributor

since you are doing cleanup you could try to merge uwp/windows. Since the portfiles are doing very similar stuff.

@strega-nil
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

strega-nil
strega-nil reviewed Nov 10, 2020
View reviewed changes
ports/openssl/CONTROL Outdated Show resolved Hide resolved
@strega-nil strega-nil merged commit 2c3ded4 into microsoft:master Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@strega-nil strega-nil strega-nil left review comments

@JackBoosY JackBoosY JackBoosY approved these changes

Assignees

@JackBoosY JackBoosY

Labels
category:port-update The issue is with a library, which is requesting update new revision info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[openssl] remove obsolete patches
4 participants
@NancyLi1013 @Neumann-A @strega-nil @JackBoosY