You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Has Makefile which builds nim-sds with generate-sds target
This target is used by other targets like:
$(GO_CMD_BUILDS)
test-functional
test-unit
Others
Consumes sds-go-bindings located in vendor folder:
vendor/github.com/waku-org/sds-go-bindings
Go can only build C dependencies using CGO
For this reason nim-sds needs to be built to generate C headers and library
sds-go-bindings
Has sds/Makefile which builds nim-sds
The build-android target builds nim-sds based on ARCH var.
The build target builds for local architecture.
Expectes nim-sds library to exist under:
third_party/nim-sds/build/libsds.{so,dylib,dll}
third_party/nim-sds/library/libsds.h
nim-sds
Has make targets for building libraries for various platforms
libsds-android-arm64
libsds-android-amd64
libsds-android-x86
libsds-android-arm
There two way in which it can be built:
Just run make way
Nix derivation via flake
Expected output loccations:
build/libsds.{so,dylib,dll}
library/libsds.h
I would recommend debugging why Nix wrapped GCC is not being used by CGO, because it should include the lib and header paths via NIX_CFLAGS_COMPILE, and NIX_LDFLAGS. You can debug that using export NIX_DEBUG=1 or up to 7.
❌ Patch coverage is 60.49383% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.55%. Comparing base (e061db0) to head (ac50dc1). ⚠️ Report is 5 commits behind head on develop.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
SDS is a protocol to improve reliability of community messages. SDS spec
A description to understand introduced changes without reading the code.
Important changes:
Relates #waku-org/pm#194