conan-center-index
conan-center-index copied to clipboard
conan-io
qtkeychain: initial
Summary
Changes to recipe: lib/qtkeychain
Motivation
fixes #19046
Details
Introduce new recipe qtkeychain
- [x] Read the contributing guidelines
- [x] Checked that this PR is not a duplicate: list of PRs by recipe
- [ ] Tested locally with at least one configuration using a recent version of Conan
- Tested locally with at least one configuration using a recent version of Conan
Local build is currently blocked by https://github.com/frankosterfeld/qtkeychain/issues/267 ... :shrug:
I did some cleanups, some work still needs to be done, thanks
Will be watching and learning ... ;-)
@AbrilRBS are we missing the qt option 'with_doubleconversion'?
https://github.com/conan-io/conan-center-index/blob/62d2d8fa803b2c395e32ed78d1d4e2f9f02be53c/recipes/qt/6.x.x/conanfile.py#L353-L354
@AbrilRBS are we missing the qt option 'with_doubleconversion'?
https://github.com/conan-io/conan-center-index/blob/62d2d8fa803b2c395e32ed78d1d4e2f9f02be53c/recipes/qt/6.x.x/conanfile.py#L353-L354
Sorry for the delay. The multiconfiguration option is off by default so double-conversion won't be built by default, but I can't seem to find where double-conversion is required to be used in qtkeychain, can you point me to the codepath that uses it? Thanks!
but I can't seem to find where
double-conversionis required to be used in qtkeychain, can you point me to the codepath that uses it?
I could not find it either ... by proposal to fix was more derived from the compiler error than from qtkeychain inner needs ...
by proposal to fix was more derived from the compiler error than from qtkeychain inner needs
I think we should find why this fails the way it does. If the proper solution is to have qt built with those extra options, then we'll merge with the check in the validate method checking for their values, but it would be preferrable if we understand if/how we need it, in order to be able to publish binaries for this :)
I think we should find why this fails the way it does. If the proper solution is to have qt built with those extra options, then we'll merge with the check in the validate method checking for their values, but it would be preferrable if we understand if/how we need it, in order to be able to publish binaries for this :)
@AbrilRBS can we drag somebody in here with deeper understanding of the qt build? I am just a consumer of the libraries and currently lack deeper understanding. THX you!