Skip to content

[boost] pulling has_synchro bits from https://github.com/microsoft/vcpkg/pull…#30836

Merged
JavierMatosD merged 28 commits intomicrosoft:masterfrom
JavierMatosD:boost_pulling_has_synchro
Jun 16, 2023
Merged

[boost] pulling has_synchro bits from https://github.com/microsoft/vcpkg/pull…#30836
JavierMatosD merged 28 commits intomicrosoft:masterfrom
JavierMatosD:boost_pulling_has_synchro

Conversation

@JavierMatosD
Copy link
Copy Markdown
Contributor

@JavierMatosD JavierMatosD commented Apr 13, 2023

@Neumann-A I want to pull out the has_synchro bits from #27694

@JavierMatosD JavierMatosD changed the title pulling has_synchro bits from https://github.com/microsoft/vcpkg/pull… [boost] pulling has_synchro bits from https://github.com/microsoft/vcpkg/pull… Apr 13, 2023
@JonLiu1993 JonLiu1993 added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Apr 14, 2023
@JonLiu1993
Copy link
Copy Markdown
Contributor

@BillyONeal, Could you please take a review?

Copy link
Copy Markdown
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

The changes in here look good. However, clearly scripts/boost/generate-ports.ps1 has not been run, which will erase a bunch of these changes. I believe you and @ras0219-msft were going to propose changes to generate-ports.ps1, but in that case I think you're on the hook to have those changes in hand before we proceed. (That is, in an otherwise unchanged repo generate-ports.ps1 needs to result in no changes)

@JonLiu1993
Copy link
Copy Markdown
Contributor

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

@JonLiu1993 JonLiu1993 added the depends:different-pr This PR or Issue depends on a PR which has been filed label May 23, 2023
@JonLiu1993
Copy link
Copy Markdown
Contributor

Depends on #31572.

# Conflicts:
#	ports/boost-atomic/vcpkg.json
#	versions/b-/boost-atomic.json
#	versions/b-/boost-modular-build-helper.json
@JonLiu1993 JonLiu1993 removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Jun 12, 2023
@JavierMatosD JavierMatosD marked this pull request as ready for review June 13, 2023 18:52
@JavierMatosD JavierMatosD requested a review from BillyONeal June 13, 2023 18:53
@JonLiu1993 JonLiu1993 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jun 16, 2023
@JavierMatosD JavierMatosD merged commit c6c59e3 into microsoft:master Jun 16, 2023
Copy link
Copy Markdown
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

This approval is assuming generate-ports.ps1 makes no changes.

@JavierMatosD JavierMatosD deleted the boost_pulling_has_synchro branch October 31, 2023 23:10
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.

4 participants