Skip to content

Update dependencies#4301

Merged
2dust merged 2 commits intomasterfrom
unknown repository
Jan 30, 2025
Merged

Update dependencies#4301
2dust merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Jan 30, 2025

No description provided.

@2dust 2dust merged commit e8d2c62 into 2dust:master Jan 30, 2025
1 check passed
@2dust
Copy link
Owner

2dust commented Jan 30, 2025

Pls fix it.

@ghost
Copy link
Author

ghost commented Jan 30, 2025

Pls fix it.

I think that if you can't fix it yourself, revert it so that it can be fixed later when you have a chance

@ghost
Copy link
Author

ghost commented Jan 30, 2025

Pls fix it.

@UjuiUjuMandan

@UjuiUjuMandan
Copy link
Contributor

[arm64-v8a] Executable     : libtun2socks.so
ld.lld: error: undefined symbol: SocksUdpClient_Init
>>> referenced by tun2socks.c:650 (./badvpn/tun2socks/tun2socks.c:650)
>>>               /tmp/tmp.sy1269icNc/tmp/local/arm64-v8a/objs/tun2socks/badvpn/tun2socks/tun2socks.o:(main)

ld.lld: error: undefined symbol: SocksUdpClient_Free
>>> referenced by tun2socks.c:726 (./badvpn/tun2socks/tun2socks.c:726)
>>>               /tmp/tmp.sy1269icNc/tmp/local/arm64-v8a/objs/tun2socks/badvpn/tun2socks/tun2socks.o:(main)

ld.lld: error: undefined symbol: SocksUdpClient_SubmitPacket
>>> referenced by tun2socks.c:1596 (./badvpn/tun2socks/tun2socks.c:1596)
>>>               /tmp/tmp.sy1269icNc/tmp/local/arm64-v8a/objs/tun2socks/badvpn/tun2socks/tun2socks.o:(device_read_handler_send)
cl***: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [/home/runner/.setup-ndk/r27/build/core/build-binary.mk:674: /tmp/tmp.sy1269icNc/tmp/local/arm64-v8a/libtun2socks.so] Error 1

@UjuiUjuMandan
Copy link
Contributor

UjuiUjuMandan commented Jan 31, 2025

@alphax-hue3682 Include newly added socks_udp_client folder in XTLS/badvpn@dc99ade to tun2socks.mk.

It should work, but build it yourself locally before making making a new commit.

@UjuiUjuMandan
Copy link
Contributor

I'll open another pr.

@UjuiUjuMandan UjuiUjuMandan mentioned this pull request Jan 31, 2025
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.

2 participants