-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
tiledb: add new recipe #19381
base: master
Are you sure you want to change the base?
tiledb: add new recipe #19381
Conversation
This comment has been minimized.
This comment has been minimized.
Triggered in conan-io/conan-center-index#19381 (comment) for `self.options.remove_deprecations`.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
7bd7007
to
102cbf6
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 102cbf6tiledb/2.16.3@#3424dfa3914ab293bc1ae9785f565c73
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 102cbf6tiledb/2.16.3@#3424dfa3914ab293bc1ae9785f565c73
|
This comment has been minimized.
This comment has been minimized.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
3ab6223
to
3705c96
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit da3b13dtiledb/2.17.4@#60cc8ccbf100b5b0189f1e951c3b7ff4
|
@RubenRBS Friendly ping. This PR has been green since November already. Should be a fairly straightforward one to review, I think. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 746c53ftiledb/2.21.0@#6830c80a6db90743c48ef964dfb0ff3f
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 96d1412tiledb/2.21.0@#9c0e325032d6df2e416a8708763a1332
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay, going thru old PRs now. I'll be trying to compile non default options for a bit today, will let you know and post logs should a combination fail :)
if self.options.serialization: | ||
self.options["libcurl"].with_zstd = True | ||
if self.options.s3: | ||
self.options["aws-sdk-cpp"].s3 = True | ||
self.options["aws-sdk-cpp"]["identity-management"] = True | ||
self.options["aws-sdk-cpp"].sts = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this not be part of the validate method too if these are hard requirements?
This comment has been minimized.
This comment has been minimized.
Rebuilding now |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 96910b7tiledb/2.22.0@#044583203e973cb8a2090307d174c9aa
|
Conan v1 pipeline ✔️All green in build 7 (
Conan v2 pipeline ✔️
All green in build 7 (
|
Hooks produced the following warnings for commit d057abatiledb/2.22.0@#3dfa5f9b3f06108e88a0c624ecc65219
|
Adds tiledb: https://github.com/TileDB-Inc/TileDB
Used by GDAL as an optional dependency.
Resolves #18684.
The baseline configuration works, but the recipe would benefit from the following:
azure-storage-blobs-cpp
package -azure
option is currently broken due to this.libmagic
to enable Windows support oftiledb
.