Merge pull request #6 from skolemlabs/fix/brew-openSSL

moved library path to library case, not cflags case
skolemlabs · Dec 17, 2021 · b0e9f33 · b0e9f33
2 parents 95ce7a6 + 796ff70
commit b0e9f33
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGES b/CHANGES
@@ -1,3 +1,6 @@
+* Fri Dec 17 2021 (1.2.5)
+  * Add library path fixes for brew for openssl library linking
+
 * Tue Nov 12 2019 (1.2.4)
   * Auto-initialize library in Mysql.escape
 

diff --git a/src/config/discover.ml b/src/config/discover.ml
@@ -35,12 +35,12 @@ let mysql_config_path =
 let c_flags =
   match execute_with_output (mysql_config_path ^ " --cflags") with
   | Some str ->
-    String.split_on_char ' ' str @ [ "-fPIC"; "-L/usr/local/opt/openssl/lib/" ]
+    String.split_on_char ' ' str @ [ "-fPIC" ]
   | None -> []
 
 let libs =
   match execute_with_output (mysql_config_path ^ " --libs") with
-  | Some str -> String.split_on_char ' ' str
+  | Some str -> String.split_on_char ' ' str @ [ "-L/usr/local/opt/openssl/lib/" ]
   | None -> []
 
 let replace_exact rep _with str =