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

[yomm2] new port #17886

Merged
merged 1 commit into from
May 26, 2021
Merged

[yomm2] new port #17886

merged 1 commit into from
May 26, 2021

Conversation

FabienPean
Copy link
Contributor

Add new port for library yomm2. Note that dependency of Boost libs are not resolved properly (#17882 #17848)

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

    Normally all, waiting for CI

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@JonLiu1993 JonLiu1993 self-assigned this May 13, 2021
@JonLiu1993 JonLiu1993 added the category:new-port The issue is requesting a new library to be added; consider making a PR! label May 13, 2021
@JackBoosY
Copy link
Contributor

Please get failure logs here.

@strega-nil-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

ports/yomm2/vcpkg.json Outdated Show resolved Hide resolved
@FabienPean FabienPean force-pushed the port/yomm2 branch 2 times, most recently from 2dd61e2 to 9fb552c Compare May 13, 2021 16:07
@FabienPean
Copy link
Contributor Author

Note that dependency to boost-integer and boost-move are only required because of boost-dynamic-bitset and should be removed from this port when #17882 is resolved.

@PhoebeHui PhoebeHui assigned PhoebeHui and unassigned JackBoosY and JonLiu1993 May 14, 2021
@FabienPean FabienPean force-pushed the port/yomm2 branch 2 times, most recently from 60d8b4c to 7d27897 Compare May 17, 2021 23:24
@FabienPean
Copy link
Contributor Author

I added a patch to fix uwp and osx build in vcpkg. This should be removed in the next version of yomm2 thanks to an update of the CMakeLists of the lib. The CI does not crash anymore on this port, so I think it is ready for review.

@FabienPean FabienPean marked this pull request as ready for review May 18, 2021 09:42
@JackBoosY
Copy link
Contributor

Depends on #17983.

@JackBoosY JackBoosY added depends:different-pr This PR or Issue depends on a PR which has been filed and removed depends:different-pr This PR or Issue depends on a PR which has been filed labels May 18, 2021
@JackBoosY
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY JackBoosY requested a review from PhoebeHui May 20, 2021 02:50
Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! This LGTM once my comment has been addressed.

ports/yomm2/vcpkg.json Outdated Show resolved Hide resolved
@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels May 26, 2021
@ras0219-msft ras0219-msft merged commit 5c847ac into microsoft:master May 26, 2021
@ras0219-msft
Copy link
Contributor

Thanks for the quick response!

@FabienPean FabienPean deleted the port/yomm2 branch September 2, 2023 14:25
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.

6 participants