Skip to content

Update document.go#97

Open
sergioangulo wants to merge 1 commit intomoovweb:masterfrom
sergioangulo:patch-1
Open

Update document.go#97
sergioangulo wants to merge 1 commit intomoovweb:masterfrom
sergioangulo:patch-1

Conversation

@sergioangulo
Copy link

https://tip.golang.org/doc/go1.12#cgo
(...)Mangled C names are no longer accepted in packages that use Cgo(...)

https://tip.golang.org/doc/go1.12#cgo
(...)Mangled C names are no longer accepted in packages that use Cgo(...)
@rhernandez-itemsoft
Copy link

Es un error de esta api. Debes cambiar esa linea de ese archivo que te marca error por la siguiente: nodePtr.doc = (*C.xmlDoc)(document.DocPtr())

image

@lzecca78
Copy link

when this pr will be merged? Currently is not possible to use this library with new version of go

@kivutar
Copy link

kivutar commented Nov 16, 2019

I need this too @jbowtie could you please merge it?

1 similar comment
@Turi-fly
Copy link

Turi-fly commented Dec 6, 2019

I need this too @jbowtie could you please merge it?

@jbowtie
Copy link
Contributor

jbowtie commented Dec 6, 2019

It's been merged in my fork for some time; I don't have commit rights on the moovweb version.

kyuden added a commit to WOVNio/gokogiri that referenced this pull request Jan 30, 2020
@pjain05
Copy link

pjain05 commented Jul 30, 2023

Can we please merge this PR as I'm also getting this same error. I also tried to fork this repo and fix it in my repo but still I'm getting this error? any workaround or solution?

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.

7 participants