Hi all, please disregard the pull request, I have closed it, it is easier for me to make the change locally. This way it doesn't affect your builds.

-jeff



On 2023-07-03 7:59 p.m., Regina Obe wrote:
FWIW Debian Buster I think ships with cmake 3.13 and it’s not that old. Old but not that old.

-1  to increase the cmake version until the pressing reason can be better explained or there are more pressing reasons.

As I recall I think Amazon had to upgrade their CMake to build the latest GEOS.

Note Debian Buster ships with 3.13 (which is old but not that old).

I also just approved this run of this pull request, and it broke some of our bots like the Ubuntu 20.04, which is kind of in the same boat as Debian Buster, old but not that old. Okay maybe in year when it’s time to release we can consider it ancient.

What I am curious to understand is why our VS windows 2022 build running cmake 3.26.4 https://github.com/libgeos/geos/actions/runs/5446890161/jobs/9908248625 <https://github.com/libgeos/geos/actions/runs/5446890161/jobs/9908248625>

Seems to be doing just fine without this patch am I missing something here?  What is different about this pull request error.

What version of VS is this using?

Thanks,

Regina

*From:*geos-devel [mailto:geos-devel-boun...@lists.osgeo.org] *On Behalf Of *Daniel Baston
*Sent:* Monday, July 3, 2023 5:13 PM
*To:* GEOS Development List <geos-devel@lists.osgeo.org>
*Subject:* Re: [geos-devel] cmake => 3.15?

I don't see a problem with requiring 3.15, but the issue isn't very clear to me. Is our CMake configuration using some feature of 3.15 while declaring that we only require 3.13?

Dan

On Mon, Jul 3, 2023 at 3:56 PM Paul Ramsey <pram...@cleverelephant.ca <mailto:pram...@cleverelephant.ca>> wrote:

    For this small fix with MSVC noise...

    https://github.com/libgeos/geos/pull/936
    <https://github.com/libgeos/geos/pull/936>

    It's still quite an old release, my dev env sits at 3.24 for example.

    P
_______________________________________________
geos-devel mailing list
geos-devel@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/geos-devel

Reply via email to