MAINT: Address review comments

Also drop hopefully unnecessary brew work for macos

Co-authored-by: Eli Schwartz <eschwartz@archlinux.org>

highspy/meson.build

@@ -2,8 +2,7 @@ py_mod = import('python')
 py = py_mod.find_installation(pure: false)
 pyb11_dep = [
   py.dependency(),
-  dependency('pybind11',
-            required: true)
+  dependency('pybind11')
 ]
 _deps += pyb11_dep
 

meson.build

@@ -1,6 +1,8 @@
 project('highs', 'cpp',
   version : '1.5.3',
-  default_options : ['warning_level=1', 'cpp_std=c++17'])
+  default_options : ['warning_level=1',
+                     'cpp_std=c++17',
+                     'wrap_mode=forcefallback'])
 
 
 # Add C++ compiler options
@@ -105,7 +107,7 @@ if get_option('with_tests')
 endif
 
 # --------------------- Bindings
+
 if get_option('with_pybind11')
-  subproject('pybind11')
   subdir('highspy')
 endif

pyproject.toml

@@ -23,6 +23,7 @@ build-backend = "mesonpy"
 [tool.meson-python.args]
 setup = ['-Dwith_pybind11=True',
         '-Dhighsint64=False',
+        '-Dwrap_mode=forcefallback'
         ]
 dist = ['--include-subprojects']
 
@@ -39,7 +40,6 @@ repair-wheel-command = "auditwheel repair -w {dest_dir} {wheel}"
 [tool.cibuildwheel.macos]
 archs = ["x86_64 arm64"]
 environment = { RUNNER_OS="macOS" }
-before-all = "brew install pkg-config && brew install pybind11"
 
 # [tool.cibuildwheel.windows]
 # archs = ["AMD64"]