Skip to content

fix: bump nim-libp2p and nim-lsquic#3707

Open
jakubgs wants to merge 3 commits intomasterfrom
bump-nim-lsquic
Open

fix: bump nim-libp2p and nim-lsquic#3707
jakubgs wants to merge 3 commits intomasterfrom
bump-nim-lsquic

Conversation

@jakubgs
Copy link
Contributor

@jakubgs jakubgs commented Jan 30, 2026

Fixes issues with windows builds:

And updates lsquic submodule URL:

Issue

@jakubgs jakubgs self-assigned this Jan 30, 2026
Fixes issues with windows builds:
vacp2p/nim-lsquic@4fb03ee7

And updates lsquic submodule URL:
vacp2p/nim-lsquic@37fb01e2

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs changed the title fix: windows builds and incorrect submodule URL fix: bump nim-libp2p and nim-lsquic Jan 30, 2026
@github-actions
Copy link

github-actions bot commented Jan 30, 2026

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:3707

Built from bddcda9

@jakubgs
Copy link
Contributor Author

jakubgs commented Feb 1, 2026

This doesn't look good:

DBG 2026-01-30 16:44:07.925+00:00 stopping sendNonPriorityTask               topics="libp2p pubsubpeer" tid=26432 file=pubsubpeer.nim:593 p=QmS*ugZrg9
DBG 2026-01-30 16:44:07.925+00:00 unsubscribing pubsub peer                  topics="libp2p pubsub" tid=26432 file=pubsub.nim:204 peerId=QmS*ugZrg9
Traceback (most recent call last)
/home/runner/work/logos-messaging-nim/logos-messaging-nim/tests/wakunode_rest/test_rest_store.nim(59) test_rest_store
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-unittest2/unittest2.nim(1170) suitePayload
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-unittest2/unittest2.nim(1089) runDirect
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-testutils/testutils/unittests.nim(16) runTest`gensym80
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-chronos/chronos/internal/asyncfutures.nim(660) waitFor
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-chronos/chronos/internal/asyncfutures.nim(635) pollFor
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-chronos/chronos/internal/asyncengine.nim(150) poll
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nim-libp2p/libp2p/protocols/pubsub/gossipsub.nim(291) addPeerAfterConnected
SIGSEGV: Illegal storage access. (Attempt to read from nil?)
Segmentation fault (core dumped)
stack trace: (most recent call last)
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nimbus-build-system/vendor/Nim/lib/system/nimscript.nim(427, 18)
/home/runner/work/logos-messaging-nim/logos-messaging-nim/waku.nimble(133, 3) testTask
/home/runner/work/logos-messaging-nim/logos-messaging-nim/waku.nimble(105, 3) test
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nimbus-build-system/vendor/Nim/lib/system/nimscript.nim(264, 7) exec
/home/runner/work/logos-messaging-nim/logos-messaging-nim/vendor/nimbus-build-system/vendor/Nim/lib/system/nimscript.nim(264, 7) Error: unhandled exception: FAILED: build/all_tests_waku [OSError]
make[1]: *** [Makefile:231: testwaku] Error 1

@Ivansete-status
Copy link
Collaborator

@jakubgs - I'll take a look at the issue

@Ivansete-status
Copy link
Collaborator

Ivansete-status commented Feb 2, 2026

The issue will get fixed by using current nim-libp2p v1.15.2, which has:
image

Thanks Richard!

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.

3 participants