siege: fix OpenSSL linking

The --with-ssl configuration takes a path to OpenSSL, which wasn't being passed before. This change adds that as a recommended dependency and configures the path properly.

Also adds a test for https if the formula was build with openssl.

Closes #34105.

Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>

1 files changed, 5 insertions, 2 deletions

diff --git a/Library/Formula/siege.rb b/Library/Formula/siege.rb
index be72bd92d..4c1731722 100644
--- a/Library/Formula/siege.rb
+++ b/Library/Formula/siege.rb
@@ -4,6 +4,9 @@ class Siege < Formula
   homepage 'http://www.joedog.org/index/siege-home'
   url 'http://download.joedog.org/siege/siege-3.0.7.tar.gz'
   sha256 'c651e2ae871cc680eb375f128b4809e97ceecf367f6bd65c3df00603fbceed4e'
+  revision 1
+
+  depends_on 'openssl'
 
   bottle do
     sha1 "b20c74de2adcdae43e88fbca22fc01fcf58b5285" => :mavericks
@@ -18,7 +21,7 @@ class Siege < Formula
                           "--prefix=#{prefix}",
                           "--mandir=#{man}",
                           "--localstatedir=#{var}",
-                          "--with-ssl"
+                          "--with-ssl=#{Formula["openssl"].opt_prefix}"
     system "make install"
   end
 
@@ -40,6 +43,6 @@ class Siege < Formula
   end
 
   test do
-    system "#{bin}/siege", "--concurrent=1", "--reps=1", "http://google.com/"
+    system "#{bin}/siege", "--concurrent=1", "--reps=1", "https://www.google.com/"
   end
 end