Skip to content

Add CPLErrContainer wrapper for ContourGenerate#102

Open
emar-kar wants to merge 1 commit into
lukeroth:masterfrom
emar-kar:lemarkar/fix-cerr-countour-generate
Open

Add CPLErrContainer wrapper for ContourGenerate#102
emar-kar wants to merge 1 commit into
lukeroth:masterfrom
emar-kar:lemarkar/fix-cerr-countour-generate

Conversation

@emar-kar

@emar-kar emar-kar commented Apr 8, 2024

Copy link
Copy Markdown

Recent update with new func ContourGenerate resulted in build errors on new versions of go.

github.com/lukeroth/gdal/algorithms.go:325:3: CPLErr(func() _Ctype_CPLErr {…}()).Err undefined (type CPLErr has no field or method Err)

This problem is related to #90 and was fixed in #92. Nevertheless ContourGenerate was using old format which caused some troubles.

@ARolek

ARolek commented Apr 27, 2024

Copy link
Copy Markdown

I just hit this same problem and this PR fixed it for me.

@canghel3

Copy link
Copy Markdown

this does fix the compile errors. would be nice if it was merged.

@chris

chris commented May 20, 2024

Copy link
Copy Markdown

Running into this as well. Using Go 1.22.3 on arm64 macOS.

@emar-kar

emar-kar commented Jun 3, 2024

Copy link
Copy Markdown
Author

@martende can you take a look, please?

@pdxrlj

pdxrlj commented Jun 14, 2024

Copy link
Copy Markdown

this does fix the compile errors. would be nice if it was merged.

@canghel3

Copy link
Copy Markdown

Any updates on this?

@Chenzo1001

Copy link
Copy Markdown

Still not merged yet....I've just modified local file by myself....

@canghel3

Copy link
Copy Markdown

Still not merged yet....I've just modified local file by myself....

I'd suggest using another library that is somewhat better maintained : https://github.com/airbusgeo/godal
This repo is dead.

@AliFlux

AliFlux commented Oct 17, 2024

Copy link
Copy Markdown

Hi, this package is completely broken. And this PR could fix it, any idea why this isnt merged yet?

@xiaxichen

Copy link
Copy Markdown

hi this pr is good please check it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants