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.

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

[libprotobuf-mutator] Add new port #13691

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

lebdron
Copy link
Contributor

@lebdron lebdron commented Sep 23, 2020

Describe the pull request

@lebdron lebdron marked this pull request as draft September 23, 2020 21:15
@lebdron lebdron force-pushed the port/libprotobuf-mutator branch from 0765f67 to 0c38ed6 Compare September 23, 2020 22:07
@lebdron lebdron marked this pull request as ready for review September 23, 2020 22:24
@lebdron lebdron force-pushed the port/libprotobuf-mutator branch from 0c38ed6 to 1662a79 Compare September 23, 2020 22:37
@NancyLi1013 NancyLi1013 added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Sep 24, 2020
ports/libprotobuf-mutator/vcpkg.json Outdated Show resolved Hide resolved
ports/libprotobuf-mutator/vcpkg.json Outdated Show resolved Hide resolved
@NancyLi1013
Copy link
Contributor

Hi @lebdron
Could you please address the review suggestions?

@lebdron lebdron force-pushed the port/libprotobuf-mutator branch 2 times, most recently from 0d1db0d to 9351c74 Compare November 1, 2020 23:12
@lebdron lebdron requested a review from NancyLi1013 November 1, 2020 23:16
@lebdron lebdron force-pushed the port/libprotobuf-mutator branch from 9351c74 to d5db90b Compare November 9, 2020 15:59
@lebdron lebdron requested a review from NancyLi1013 November 9, 2020 16:13
@NancyLi1013
Copy link
Contributor

LGTM, only one thing needs to be confirmed now.

I noticed that there are no any released version for the new port. So is it possible to release a stable version for
libprotobuf-mutator?

@lebdron
Copy link
Contributor Author

lebdron commented Nov 13, 2020

I am afraid I have to ping @vitalybuka again as I cannot make a release, and also google/libprotobuf-mutator#186 is a relevant issue.

@lebdron lebdron force-pushed the port/libprotobuf-mutator branch 2 times, most recently from a40b50e to 1f0f362 Compare November 15, 2020 01:50
@lebdron lebdron force-pushed the port/libprotobuf-mutator branch from 1f0f362 to 69deac3 Compare November 15, 2020 09:51
@lebdron
Copy link
Contributor Author

lebdron commented Nov 15, 2020

Hello @NancyLi1013 . Everything is done 😊 . Libarchive should be fixed in #14571

@NancyLi1013
Copy link
Contributor

Thanks for your so quick response @lebdron. I will rerun this PR once PR #14571 merged. Besides this , everything seems good now.

Also thanks for your help and support @vitalybuka.

@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Nov 17, 2020
@strega-nil strega-nil merged commit 0b7e8c9 into microsoft:master Nov 17, 2020
@lebdron lebdron deleted the port/libprotobuf-mutator branch November 18, 2020 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants