Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

New Formula for netperf #18197

Closed
wants to merge 1 commit into from
Closed

New Formula for netperf #18197

wants to merge 1 commit into from

Conversation

tomburns
Copy link
Contributor

@tomburns tomburns commented Mar 3, 2013

Created a new formula for netperf. Test avoids disrupting any existing netserver processes; will use existing one if available.

Created a new formula for netperf. Test avoids disrupting any existing netserver processes; will use existing one if available.
@jacknagel
Copy link
Contributor

$ brew test ./netperf.rb -v
Testing netperf
==> netperf
netperf
complete_addrinfo: could not resolve 'localhost' port '0' af 2
    getaddrinfo returned 8 nodename nor servname provided, or not known
Error: netperf: failed

@tomburns
Copy link
Contributor Author

Fantastic; passed on three machines I installed on. This is a fairly tricky package to test due to limited options that exit 0, and I couldn't find an appropriate example browsing the repo. What's the standard for testing things with both a server and client component without meddling with a potentially extant process?

@adamv
Copy link
Contributor

adamv commented Mar 11, 2013

"omit the test", I suppose.

@MikeMcQuaid
Copy link
Member

Better to just run a command with --version than omitting it; at least that verifies it has linked.

@ghost ghost assigned adamv Apr 30, 2013
@adamv adamv closed this in 40715e2 May 2, 2013
handyman5 pushed a commit to handyman5/homebrew that referenced this pull request Oct 7, 2013
Closes Homebrew#18197.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants