Added coturn formula #43845
Added coturn formula #43845
Conversation
| class Coturn < Formula | ||
| desc "Free open source implementation of TURN and STUN Server" | ||
| homepage "https://github.com/coturn/coturn" | ||
| url "http://turnserver.open-sys.org/downloads/v#{version}/turnserver-#{version}.tar.gz" |
There was a problem hiding this comment.
We don't do this in Homebrew. The version should be inlined into the URL.
|
@DomT4 Ah, I misinterpreted the "version 4.4.5.4 is redundant with version scanned from URL" audit error. I've updated the formula with inlined version into the url and removed the version property. |
| end | ||
|
|
||
| test do | ||
| system "turnadmin", "-l" |
There was a problem hiding this comment.
This needs to be system bin/"turnadmin", "-l", presuming the binary is in bin. We use full paths in testing.
There was a problem hiding this comment.
Changed system "turnadmin", "-l" to system "#{bin}/turnadmin", "-l"
|
✅ Copy-pastable url Anything else? |
Free open source implementation of TURN and STUN Server
|
Travis-CI failed due to github rate-limiting, you might want to add a http://docs.travis-ci.com/user/environment-variables/#Encrypted-Variables Restart the job when github has calmed down, and you'll see that the formula builds just fine. |
|
We're aware of the Travis issues and working on them. Jenkins remains the primary CI at the moment. |
|
Thanks for your patience! Merged in 22cccc4. |
A free open source implementation of TURN and STUN Server
Which is used for reliable (WebRTC) connections.