search

dennwc / inkview

Go SDK for Pocketbook
MIT License
34 stars 5 forks source link

Feature/upgrade go 1.15.6 #5

Closed Skeeve closed 3 years ago

Skeeve commented 3 years ago

Hi Denys. Can you please have a look?

I (tried to) upgrade the SDK to go version 1.15.6.

Unfortunately now go-sqlite cannot be compiled anymore.

I get

# runtime/cgo
cc1: warnings being treated as errors
_cgo_export.c:6: warning: ignoring #pragma GCC diagnostic
_cgo_export.c:7: warning: ignoring #pragma GCC diagnostic
_cgo_export.c:8: warning: ignoring #pragma GCC diagnostic

And have no clue yet what this means. If you leave out the "go get …" in the Dockerfile, the sdk seems to build.

The only change I did was to change the go-pb.patch to be applied to exec.go instead of build.go now.

Skeeve commented 3 years ago

Update: According to https://github.com/mattn/go-sqlite3/issues/907 it seems the version 4.1.2 of arm-none-linux-gnueabi-gcc is outdated fpr go 1.12 and onwards.

I think that version is installed in the pbsdk, right? I haven't found a way yet to upgrade to a newer arm-none-linux-gnueabi-gcc let alone find a newer version of it :(

dennwc commented 3 years ago

@Skeeve I found a workaround for it to support the latest Go, but it's in a private branch. I need to rebase it onto your latest changes.

Skeeve commented 3 years ago

I think I found a solution, using gcc-arm-linux-gnueabi.

I could compile the sqlitetst and run it successfully on my Touch HD in pbterm.

dennwc commented 3 years ago

That's awesome! :tada: I need to check on my device as well to be sure. But I think it will succeed.