Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linux Build Failing (error: no such module 'zlib') #1031

Open
chrisschlitt opened this issue May 31, 2024 · 6 comments
Open

Linux Build Failing (error: no such module 'zlib') #1031

chrisschlitt opened this issue May 31, 2024 · 6 comments
Labels

Comments

@chrisschlitt
Copy link

chrisschlitt commented May 31, 2024

Describe the bug

I get the following build issue on Linux in Docker:

#19 6.933 [89/975] Compiling Starscream Compression.swift
#19 6.933 /build/.build/checkouts/Starscream/Sources/Compression/WSCompression.swift:30:8: error: no such module 'zlib'
#19 6.933 import zlib
#19 6.933        ^
#19 6.942 error: fatalError

Steps to Reproduce

I'm running swift build -c release --static-swift-stdlib -Xlinker -ljemalloc when I get the error

Environment:

  • OS/Version: swift:5.10-jammy
  • Starscream Version 4.0.4
@zshannon
Copy link

zshannon commented Oct 9, 2024

@chrisschlitt did you find a workaround for this?

@chrisschlitt
Copy link
Author

I did not sadly, but still interested in a solution

@zackliu
Copy link

zackliu commented Oct 30, 2024

The same here. Any findings?

@zshannon
Copy link

I ended up using the URLRequest native websocket client and disabling the test cases on Linux :( (I saw other folks using WebSocketKit from Vapor but I didn't want to mix nio in)

@chrisschlitt
Copy link
Author

Mixing nio in was my exact hesitation. Interested to see how it goes removing that for their entirely rewritten websocketkit with vapor 5, but I haven’t heard any updates on that progress.

@zackliu
Copy link

zackliu commented Oct 31, 2024

I saw a guidance here https://github.com/apple/swift-nio-zlib-support?tab=readme-ov-file#deprecated but it needs Starscream to make some changes.
#844 This should be a good fix, why not merging it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants