GDAL 3.9.x build #131
GDAL 3.9.x build #131
Conversation
pomadchin
force-pushed
the
update/gdal-3.9
branch
from
fe38d99 to
44fe512
Compare
pomadchin
force-pushed
the
update/gdal-3.9
branch
from
44fe512 to
8ba1488
Compare
pomadchin
force-pushed
the
update/gdal-3.9
branch
from
4e1f5b9 to
e9719fb
Compare
| "-of", "MEM", | ||
| "-of", "VRT", |
There was a problem hiding this comment.
It has always been VRT; GDAL used the last appended -of VRT to the param list. With MEM tests seem to be super slow btw.
There was a problem hiding this comment.
Interesting. I'm okay with this change, but my recollection is that these should write to in-memory structures which should be relatively fast.
pomadchin
force-pushed
the
update/gdal-3.9
branch
from
6c52025 to
83e303a
Compare
| "-of", "MEM", | ||
| "-of", "VRT", |
There was a problem hiding this comment.
Interesting. I'm okay with this change, but my recollection is that these should write to in-memory structures which should be relatively fast.
| options_array[i++] = "-of"; | ||
| options_array[i++] = "VRT"; | ||
| } | ||
|
|
There was a problem hiding this comment.
I assume that this also work with version of GDAL earlier than 3.9.x?
There was a problem hiding this comment.
Yes! < 3.9 GDAL was ignorant to duplicate args, and was using the last one passed. Now they have a more clear semantics of what args can be repeatable
|
GT seems to be happy with this change: locationtech/geotrellis#3540 |
🎉 🎉 🎉 |
|
Merging it in and cutting a release! |
Build against GDAL 3.9.x; windows is still behind.