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

Update Dockerfile. Add build ARG to avoid ('pread64' undeclared ) error #1

Closed
wants to merge 1 commit into from

Conversation

abdullahrkw
Copy link

I was building a docker image on my Windows machine and I got the following error
sqlite3-binding.c:35911:42: error: 'pread64' undeclared here (not in a function); did you mean 'pread'?

I searched a bit and applied suggestion (suggested here

After the change, I successfully built my Docker image on my Windows machine. The suggested change will also work for Linux machines.

I was building docker image on my windows machine and I got the following error
```sqlite3-binding.c:35911:42: error: 'pread64' undeclared here (not in a function); did you mean 'pread'?```

I searched a bit and apply suggestion (suggested [here](mattn/go-sqlite3#1164)

After the change, I was able to build my Docker image successfully on my windows machine. I think suggested change will also work for linux machine.
@phikal
Copy link
Collaborator

phikal commented Jan 4, 2024

It appears that this might have been an upstream issue in the SQLite database driver (c.f. mattn/go-sqlite3#1177). Could you try if updating the Go dependencies in go.mod resolves the issue for you?

Otherwise, you can always just build go-kgp yourself, as described in the HACKING file.

@phikal
Copy link
Collaborator

phikal commented Jan 6, 2024

FWIW it seems that after 73d4c5d the issue has been resolved, as I can reproduce the issue you had before that, but not since. I will therefore close this report. Thank you for brining the problem to my attention.

@phikal phikal closed this Jan 6, 2024
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.

None yet

2 participants