fix go linking

.github/workflows/build.yml

@@ -130,6 +130,7 @@ jobs:
         env:
           CC: clang
         run: |
+          update-alternatives --set ld /usr/bin/lld
           make all
       - uses: actions/upload-artifact@v3
         with:

internal/pkg/all-in-one/all-in-one.go

@@ -9,7 +9,6 @@ import "C"
 import (
 	"fmt"
 	"time"
-	"unsafe"
 
 	"github.com/Jeffail/gabs"
 	"github.com/alicebob/miniredis/v2"
@@ -22,14 +21,6 @@ type AllInOne struct {
 	instance *C.AllInOneInstance
 }
 
-//export goErrorHandler
-func goErrorHandler(size C.int, start *C.char) {
-	dest := make([]byte, size)
-	copy(dest, (*(*[1024]byte)(unsafe.Pointer(start)))[:size:size])
-
-	println("Error from all in one runner: %s", string(dest))
-}
-
 func NewAllInOne() (*AllInOne, error) {
 	redis := miniredis.NewMiniRedis()
 	nats, err := server.NewServer(&server.Options{})
@@ -55,6 +46,7 @@ func (s *AllInOne) Start() error {
 	if !s.nats.ReadyForConnections(5 * time.Second) {
 		return fmt.Errorf("nats server didn't start after 5 seconds, please check if there is another service listening on the same port as nats")
 	}
+
 	handler := C.ErrorHandler(C.allInOneErrorHandler)
 	// Set the error handler
 	C.set_error_handler(handler)

internal/pkg/all-in-one/error_handler.h

@@ -2,7 +2,6 @@ extern void goErrorHandler(int, char*);
 
 typedef void (*ErrorHandler)(int, char*);
 
-__attribute__((weak))
 void allInOneErrorHandler(int size, char* string) {
   goErrorHandler(size, string);
 }

internal/pkg/all-in-one/handler.go

@@ -0,0 +1,13 @@
+package allinone
+
+import "C"
+import "unsafe"
+
+//go:linkname goErrorHandler c.goErrorHandler
+//export goErrorHandler
+func goErrorHandler(size C.int, start *C.char) {
+	dest := make([]byte, size)
+	copy(dest, (*(*[1024]byte)(unsafe.Pointer(start)))[:size:size])
+
+	println("Error from all in one runner: %s", string(dest))
+}