Compile both gocvx and gocv

jack139 / arcface-go

Go implementation of Arcface inference

BSD 3-Clause "New" or "Revised" License

17 stars 3 forks source link

Compile both gocvx and gocv #2

Open agorman opened 3 months ago

agorman commented 3 months ago

I'm having problems getting this to run in a program where I'm already using gocv. Any suggestions on how I can get this working?

/go/pkg/mod/github.com/jack139/arcface-go@v0.0.4/gocvx/core.cpp:5: multiple definition of `Mat_New'; /tmp/go-link-346442434/000022.o:/go/pkg/mod/gocv.io/x/gocv@v0.37.0/core.cpp:5: first defined here
/usr/bin/ld: /tmp/go-link-346442434/000048.o

jack139 commented 6 days ago

If you are using gocv, gocvx can be removed. Because I have pull request to gocv project. The compiling error show that the code is duplicated.

https://github.com/hybridgroup/gocv/commit/ddc82ce4188a0a3d0698c4c350ffb60f7c43ac55

agorman commented 6 days ago

@jack139 thanks. The issue is that gocvx is internally imported by arcface so I can't remove that import.