-
-
Notifications
You must be signed in to change notification settings - Fork 11.4k
Add support for pcap and tls support for sipp package #26708
Conversation
url 'http://downloads.sourceforge.net/project/sipp/sipp/3.4/sipp-3.3.990.tar.gz' | ||
sha1 'b2637cb72556595253bbdd4a68cc974c9ac1d92e' | ||
|
||
option 'pcap', 'Build with PCAP support' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason not to include this by default (or always include it?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't want to be presumptuous and change the established behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless it takes dramatically longer to build let's just make it default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I add a no-pcap option incase someone doesnt want it ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nah, they can always request/add it.
sha1 'b2637cb72556595253bbdd4a68cc974c9ac1d92e' | ||
|
||
option 'pcap', 'Build with PCAP support' | ||
option 'openssl', 'Build with OpenSSL support' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this.
Please read the Ruby Style Guide and update this pull request's changes to conform to it. |
@MikeMcQuaid - ok, I just updated. Hope this is better. |
happy to see this 👍 |
Closes Homebrew#26708. Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
Closes Homebrew#26708. Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
Add support for building with PCAP support ( mojolingo/sippy_cup#44 (comment) )