Skip to content

[pthreadpool] create a new port#16342

Merged
vicroms merged 8 commits intomicrosoft:masterfrom
luncliff:port/pthreadpool
Apr 1, 2021
Merged

[pthreadpool] create a new port#16342
vicroms merged 8 commits intomicrosoft:masterfrom
luncliff:port/pthreadpool

Conversation

@luncliff
Copy link
Copy Markdown
Contributor

@luncliff luncliff commented Feb 21, 2021

What does your PR fix?

There was no port request for this project.

This is one of the 3rd party libraries for the PyTorch project. The PR will be used for future support of the libtorch port.

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

The project has quite simple code. Expected to be portable according to its CI configuration, but need more tests.

  • x64-osx
  • x64-linux
  • x64-windows, x86-windows, arm64-windows: runs static build on Windows

Work In Progress

  • Android/iOS triplets

Does your PR follow the maintainer guide?

The library has a tag, but it's too old. I snapshoted the source code with "version-string": 2021-02-21

@luncliff luncliff marked this pull request as draft February 21, 2021 08:37
@luncliff
Copy link
Copy Markdown
Contributor Author

luncliff commented Feb 21, 2021

This PR will be rebased when #16341 is merged. Will be ready for review after testing more triplets.

@luncliff luncliff mentioned this pull request Feb 21, 2021
3 tasks
@NancyLi1013 NancyLi1013 added category:new-port The issue is requesting a new library to be added; consider making a PR! depends:different-pr This PR or Issue depends on a PR which has been filed labels Feb 22, 2021
@luncliff luncliff marked this pull request as ready for review February 25, 2021 01:27
@NancyLi1013 NancyLi1013 removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Feb 26, 2021
@luncliff luncliff requested a review from NancyLi1013 February 28, 2021 05:53
@NancyLi1013 NancyLi1013 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Mar 1, 2021
@NancyLi1013
Copy link
Copy Markdown
Contributor

LGTM now, thanks for adding this port @luncliff.

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: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.

3 participants