Skip to content
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

C# bindings not compatible of SWIG 4.0.0 #1726

Closed
rouault opened this issue Jul 16, 2019 · 3 comments
Closed

C# bindings not compatible of SWIG 4.0.0 #1726

rouault opened this issue Jul 16, 2019 · 3 comments

Comments

@rouault
Copy link
Member

rouault commented Jul 16, 2019

Reported initially in #1725
Failures like

[00:32:58] 	swig.exe -DSWIG2_CSHARP -c++ -csharp -namespace OSGeo.GDAL  -dllimport gdal_wrap -I../../include/csharp -o gdal_wrap.cpp ../../include/gdal.i
[00:32:59] ..\..\include\csharp\swig_csharp_extensions.i(204) : Error: A deprecated csdestruct typemap was found for RasterIOExtraArg, please remove it and replace all csdestruct, csdestruct_derived and csfinalize typemaps by the csdispose, csdispose_derived, csdisposing and csdisposing_derived typemaps.
rouault added a commit that referenced this issue Jul 17, 2019
appveyor.yml: force installation of SWIG 3.0.12 since the C# bindings aren't compatible of SWIG 4.0 (fixes #1726)
@rouault
Copy link
Member Author

rouault commented Jul 17, 2019

Re-opening. Commit 8676b95 fixes #1725 not that one

@rouault rouault reopened this Jul 17, 2019
rouault added a commit that referenced this issue Jul 18, 2019
rouault added a commit that referenced this issue Jul 18, 2019
@runette
Copy link
Contributor

runette commented Jan 4, 2022

@rouault - I was reviewing issues to see if there was anything else that should be included in #5046 .

Since the CMake scripts are building on SWIG4 - should this issue be closed?

@rouault
Copy link
Member Author

rouault commented Jan 4, 2022

Let's close this, although it doesn't look we know why this works now...

@rouault rouault closed this as completed Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants