[gdal-dev] Removal of Python SWIG python generated files from git

[Kurt Schwehr]

I am also okay with removing the files. I think removing the generated
files will help the project's health.

At Google, we have used the swig generated files from git for the python
interfaces. It was helpful as we don't have a lot of control about which
swig version is available. However, it's now not that hard to run swig
separately from our bazel based build with a specific version of swig that
works and copying the files over.

Thanks!
-Kurt


On Thu, Mar 9, 2023 at 12:37 PM Frank Warmerdam <warmerdam at pobox.com> wrote:

> Folks,
>
> At one time we were very sensitive to the exact version of SWIG so
> providing pre-generated bindings removed a large class of versioning
> problems.
>
> I am ok with removing them.
>
> Best regards,
> Frank
>
>
> On Thu, Mar 9, 2023 at 3:14 PM Even Rouault <even.rouault at spatialys.com>
> wrote:
>
>>
>> > To play devil's advocate, were there any (perceived) benefits to this
>> > arrangement when originally introduced, other than not needing the
>> > SWIG binary to compile Python bindings?
>>
>> That predates the start of my involvement with GDAL, so my guess would
>> be that this was just what you mention: for the sake of simplicity of
>> people building GDAL, at a time where its only build requirement was the
>> basic tools autoconf, make and g++. Nowadays, getting SWIG isn't harder
>> than getting PROJ or any of the "optional" dependencies you generally
>> want to make a reasonably feature complete GDAL build.
>>
>> Even
>>
>> --
>> http://www.spatialys.com
>> My software is free, but my time generally not.
>>
>> _______________________________________________
>> gdal-dev mailing list
>> gdal-dev at lists.osgeo.org
>> https://lists.osgeo.org/mailman/listinfo/gdal-dev
>>
>
>
> --
>
> ---------------------------------------+--------------------------------------
> I set the clouds in motion - turn up   | Frank Warmerdam,
> warmerdam at pobox.com
> light and sound - activate the windows | +1 650-701-7823
> and watch the world go round - Rush    | Geospatial Software Developer
> _______________________________________________
> gdal-dev mailing list
> gdal-dev at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/gdal-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20230309/c64a5bb5/attachment-0001.htm>