libspatialite: add v5.1.0, bump deps #21511
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
valgur
changed the title
|
🤖 Beep Boop! This pull request is making changes to 'recipes/libspatialite//'. 👋 @jwillikers you might be interested. 😉 |
ghost
mentioned this pull request
jwillikers
approved these changes
Nov 30, 2023
AbrilRBS
previously approved these changes
Nov 30, 2023
This comment has been minimized.
This comment has been minimized.
jwillikers
approved these changes
Dec 4, 2023
conan-center-bot
added
the
Missing dependencies
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
conan-center-bot
removed
the
Missing dependencies
This comment has been minimized.
This comment has been minimized.
valgur
force-pushed
the
update/libspatialite
branch
from
7faa4cf to
cb47b2c
Compare
This comment has been minimized.
This comment has been minimized.
valgur
force-pushed
the
update/libspatialite
branch
from
cb47b2c to
6f2d4a1
Compare
This comment has been minimized.
This comment has been minimized.
ericLemanissier
approved these changes
Dec 12, 2023
jwillikers
approved these changes
Dec 13, 2023
|
@uilianries Can you re-approve, please? |
This comment has been minimized.
This comment has been minimized.
valgur
force-pushed
the
update/libspatialite
branch
from
d0f1d7e to
9532e7a
Compare
This comment has been minimized.
This comment has been minimized.
valgur
changed the title
This comment has been minimized.
This comment has been minimized.
valgur
force-pushed
the
update/libspatialite
branch
from
9532e7a to
06c07f7
Compare
This comment has been minimized.
This comment has been minimized.
valgur
force-pushed
the
update/libspatialite
branch
from
06c07f7 to
9e575fc
Compare
Conan v1 pipeline ✔️All green in build 13 (
Conan v2 pipeline ✔️
All green in build 14 ( |
jwillikers
approved these changes
Jan 24, 2024
uilianries
approved these changes
Mar 1, 2024
| if self.options.get_safe("with_rttopo"): | ||
| self.requires("librttopo/1.1.0") | ||
| if self.options.with_libxml2: | ||
| self.requires("libxml2/2.10.3") | ||
| self.requires("libxml2/2.12.4") |
There was a problem hiding this comment.
GDAL uses libxml2/2.12.3. It will require another PR to align it. All other dependencies are aligned with GDAL requirements. Nice job.
| @@ -192,14 +198,17 @@ def _build_msvc(self): | |||
| if not self.options.with_minizip: | |||
| replace_in_file(self, gaiaconfig_msvc, "#define ENABLE_MINIZIP 1", "") | |||
|
|
|||
| # Workaround for a NMakeDeps define quoting issue: | |||
There was a problem hiding this comment.
Suggested change
| # Workaround for a NMakeDeps define quoting issue: | |
| # Workaround for a NMakeDeps define quoting issue. Fixed in Conan 1.63 and 2.0.15 |
So we can remember in the future to be removed. It's still a fresh bugfix.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #19111.