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

[matchit] Add first draft port #26499

Merged
merged 18 commits into from
Aug 26, 2022
Merged

[matchit] Add first draft port #26499

merged 18 commits into from
Aug 26, 2022

Conversation

daljit97
Copy link
Contributor

Add matchit, a library for pattern matching for C++17

Yes

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

I am still working on this PR

@ghost
Copy link

ghost commented Aug 24, 2022

CLA assistant check
All CLA requirements met.

@BowenFu
Copy link
Contributor

BowenFu commented Aug 24, 2022

@daljit97 maybe the tests failed because you did not run ./vcpkg x-add-version --all?

@daljit97
Copy link
Contributor Author

daljit97 commented Aug 24, 2022

@BowenFu you're right I forgot to do that, but that doesn't seem to be the issue. The problem seems to be that the files matchitConfig.cmake and matchitConfigVersion.cmake aren't generated in the buildtree of vcpkg (they are when running cmake ordinarily).

@Cheney-W Cheney-W changed the title Add first draft port for matchit [matchit] Add first draft port Aug 25, 2022
@Cheney-W Cheney-W added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Aug 25, 2022
@BowenFu
Copy link
Contributor

BowenFu commented Aug 25, 2022

@daljit97 maybe the problem is that make install was not supported in v1.0.0, will add new version v1.0.1 to fix this.

@Cheney-W
Copy link
Contributor

If this is a header-only library, you can refer to this file to modify your PR.

@daljit97
Copy link
Contributor Author

@BowenFu thanks indeed that seemed to be the issue (I probably should've checked against the 1.0.0 tag instead of head). I've tested on Linux and Windows and the port seems to work (at least locally).

Previous commit incorrectly change the license for the port libzippp. This commit fixes that.
@BowenFu
Copy link
Contributor

BowenFu commented Aug 25, 2022

Glad to see the issue finally gets resolved.

@daljit97 daljit97 marked this pull request as ready for review August 26, 2022 03:52
github-actions[bot]
github-actions bot previously approved these changes Aug 26, 2022
daljit97 and others added 2 commits August 26, 2022 14:05
Co-authored-by: Cheney Wang <38240633+Cheney-W@users.noreply.github.com>
@Cheney-W Cheney-W added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Aug 26, 2022
@JavierMatosD
Copy link
Contributor

Thank you!

@JavierMatosD JavierMatosD merged commit 024a041 into microsoft:master Aug 26, 2022
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.

None yet

4 participants