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

zxing-cpp: add version 2.3.0 #26295

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

axxel
Copy link
Contributor

@axxel axxel commented Jan 2, 2025

Summary

Changes to recipe: zxing-cpp/2.3.0

I'm the maintainer of this library and just released a new version.

I'm not a conan user myself and hence, I did not test anything related to this conan packaging.


This is a blind edit, lets see if it works...
@axxel
Copy link
Contributor Author

axxel commented Jan 3, 2025

@toge and @SpaceIm or anyone interested: I guess I need your help on this. I don't know how to make the linux build use a newer cmake (>=3.16), see build log. Neither do I feel comfortable coming up with a new _compiler_cpp_support block for c++20 in conanfile.py. It is strongly recommended to build 2.3.0 with c++20 to get a fully featured build.

It might also be worth considering adding new config options, like "enable C-API" and/or "enable experimental API".

Feel free to add to this PR or a come up with your own.

@axxel
Copy link
Contributor Author

axxel commented Jan 6, 2025

Fixed the remaining compilation issues myself. Seems ready to merge from my perspective.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant