Merge pull request #971 from bnb-chain/develop

fix: make BLST PORTABLE for release binary

.github/workflows/pre-release.yml

@@ -6,6 +6,10 @@ on:
     tags:
       - pre-v*
 
+env:
+  CGO_CFLAGS: "-O -D__BLST_PORTABLE__"
+  CGO_CFLAGS_ALLOW: "-O -D__BLST_PORTABLE__"
+
 jobs:
   build:
     name: Build Release

.github/workflows/release.yml

@@ -5,6 +5,10 @@ on:
     # Publish `v1.2.3` tags as releases.
     tags:
       - v*
+
+env:
+  CGO_CFLAGS: "-O -D__BLST_PORTABLE__"
+  CGO_CFLAGS_ALLOW: "-O -D__BLST_PORTABLE__"
 
 jobs:
   build:

Dockerfile

@@ -2,6 +2,8 @@ FROM golang:1.19-alpine AS build-env
 
 # Set up dependencies
 ENV PACKAGES make cmake git libc-dev bash gcc linux-headers eudev-dev curl ca-certificates build-base libc-dev
+ENV CGO_CFLAGS="-O -D__BLST_PORTABLE__"
+ENV CGO_CFLAGS_ALLOW="-O -D__BLST_PORTABLE__"
 
 # Set working directory for the build
 WORKDIR /go/src/github.com/bnb-chain/node

README.md

@@ -45,6 +45,15 @@ $ git clone [email protected]:bnb-chain/node.git
 $ cd node && make build
 ```
 
+If you get such error when running the node with self built binary:
+```shell
+Caught SIGILL in blst_cgo_init, consult <blst>/bindinds/go/README.md.
+```
+please try to add the following environment variables and build again:
+```shell
+export CGO_CFLAGS="-O -D__BLST_PORTABLE__"
+export CGO_CFLAGS_ALLOW="-O -D__BLST_PORTABLE__"
+```
 
 To test that installation worked, try to run the cli tool: