breaker_machines 0.7.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5137f3692641c76411b86e725df03e901220a09d18a9bc1ba64fc1834d93b811
-  data.tar.gz: f5f0c05289c82dfe6aa1fe23322a531fbfd9faa1b8e9f29575eb5c801aacf712
+  metadata.gz: 2d890ea2732633801242760b868020a4ad4d517693a71a91767b27ce8cfde282
+  data.tar.gz: bb5a5b86d182b90cabdf0871d81f2b82ef58218e77613dc5f505067b6a39a63c
 SHA512:
-  metadata.gz: 1150d77d3823b038277fb4bd7ee2c28bb39ea0307fefe227f2346c220ceb961f89349bfadc0d8b7bbcea01d7c0bd319ad96177c2de0eb24b6df82f57ee8001f6
-  data.tar.gz: 3673de4be4d9784ea7135da34e0d02aae91c4b2f8a0c04bc3e058cb01e7f250feb9e6526c4f638d555cc7a3e35a4d3ffb75ffcd770db867442e234ecf43f58a7
+  metadata.gz: b84c67c557a067f768e9b16e73a0e5e9186e909d211632151b79a17558ac56c78d92a67fc9e86b90ef6fb6382eb2f00ab57dc984a7ff64d4e89291341155dbb5
+  data.tar.gz: a1e3853602dab814c12e3581939ba95060e0ded7b023c906833530418a3b5071e3af29083303be06e1f98cf4e96028b4e1a1a9657910e18a0e69543940e2b1c8

data/ext/breaker_machines_native/ffi/extconf.rb

@@ -1,20 +1,5 @@
 # frozen_string_literal: true
 
-# Skip native extension compilation on JRuby
-if RUBY_ENGINE == 'jruby'
-  puts 'Skipping native extension compilation on JRuby'
-  puts 'BreakerMachines will use pure Ruby backend'
-  makefile_content = "all:\n\t@echo 'Skipping native extension on JRuby'\n" \
-                     "install:\n\t@echo 'Skipping native extension on JRuby'\n"
-  File.write('Makefile', makefile_content)
-  exit 0
-end
-
-# Check if Cargo is available
-def cargo_available?
-  system('cargo --version > /dev/null 2>&1')
-end
-
 def create_noop_makefile(message)
   warn message
   warn 'BreakerMachines will fall back to pure Ruby backend.'
@@ -27,6 +12,16 @@ def create_noop_makefile(message)
   exit 0
 end
 
+# Skip native extension compilation on JRuby
+if RUBY_ENGINE == 'jruby'
+  create_noop_makefile('Skipping native extension on JRuby')
+end
+
+# Check if Cargo is available
+def cargo_available?
+  system('cargo --version > /dev/null 2>&1')
+end
+
 unless cargo_available?
   create_noop_makefile('Skipping native extension (Cargo not found)')
 end
@@ -55,7 +50,7 @@ begin
   if File.exist?(makefile_path)
     manifest_path = File.expand_path(__dir__)
     contents = File.read(makefile_path)
-    contents.gsub!(%r{^RB_SYS_CARGO_MANIFEST_DIR \?=.*$}, "RB_SYS_CARGO_MANIFEST_DIR ?= #{manifest_path}")
+    contents.gsub!(/^RB_SYS_CARGO_MANIFEST_DIR \?=.*$/, "RB_SYS_CARGO_MANIFEST_DIR ?= #{manifest_path}")
     File.write(makefile_path, contents)
   end
 rescue LoadError => e

data/lib/breaker_machines/native_extension.rb

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require 'rbconfig'
+require 'active_support/core_ext/object/blank'
+
 module BreakerMachines
   # Handles loading and status of the optional native extension
   module NativeExtension
@@ -9,29 +12,29 @@ module BreakerMachines
       def load!
         return @loaded if defined?(@loaded)
 
-        # Respect explicit ENV flag to disable native (useful for testing)
-        if ENV['BREAKER_MACHINES_NATIVE'] == '0'
+        # Native extension is opt-in: only load if explicitly enabled
+        unless ENV['BREAKER_MACHINES_NATIVE'] == '1'
           @loaded = false
           BreakerMachines.instance_variable_set(:@native_available, false)
-          BreakerMachines.log(:info, 'Native extension disabled via ENV')
           return false
         end
 
-        @loaded = true
-        require 'breaker_machines_native/breaker_machines_native'
-        BreakerMachines.instance_variable_set(:@native_available, true)
-        BreakerMachines.log(:info, 'Native extension loaded successfully')
-        true
-      rescue LoadError => e
-        @loaded = false
-        BreakerMachines.instance_variable_set(:@native_available, false)
+        errors = []
 
-        # Only log if it's not JRuby (expected failure) and logging is enabled
-        if RUBY_ENGINE != 'jruby'
-          BreakerMachines.log(:warn, "Native extension not available: #{e.message}")
-          BreakerMachines.log(:warn, 'Using pure Ruby backend (slower but functional)')
+        native_library_candidates.each do |require_path|
+          try_require(require_path)
+          @loaded = true
+          BreakerMachines.instance_variable_set(:@native_available, true)
+          BreakerMachines.log(:info, "Native extension loaded successfully (#{require_path})")
+          return true
+        rescue LoadError => e
+          errors << "#{require_path}: #{e.message}"
         end
 
+        @loaded = false
+        BreakerMachines.instance_variable_set(:@native_available, false)
+        BreakerMachines.log(:warn, "Native extension not available: #{errors.join(' | ')}") unless errors.empty?
+
         false
       end
 
@@ -39,6 +42,40 @@ module BreakerMachines
       def loaded?
         defined?(@loaded) && @loaded
       end
+
+      private
+
+      def native_library_candidates
+        dlext = RbConfig::CONFIG['DLEXT']
+        base_dir = File.expand_path('../breaker_machines_native', __dir__)
+        ruby_version = RbConfig::CONFIG['ruby_version']
+        arch = RbConfig::CONFIG['arch']
+        platform = Gem::Platform.local.to_s
+
+        matches = Dir.glob(File.join(base_dir, '**', "breaker_machines_native.#{dlext}"))
+
+        prioritized = matches.sort_by do |path|
+          score = 0
+          score -= 3 if path.include?(ruby_version)
+          score -= 2 if path.include?(arch)
+          score -= 1 if path.include?(platform)
+          [score, path]
+        end
+
+        candidates = prioritized.map { |full_path| require_path_for(full_path, dlext) }.presence
+        candidates ||= ['breaker_machines_native/breaker_machines_native']
+        candidates.uniq
+      end
+
+      def require_path_for(full_path, dlext)
+        root = File.expand_path('..', __dir__)
+        relative = full_path.sub(%r{^#{Regexp.escape(root)}/?}, '')
+        relative.sub(/\.#{Regexp.escape(dlext)}\z/, '')
+      end
+
+      def try_require(require_path)
+        require require_path
+      end
     end
   end
 end

data/lib/breaker_machines/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BreakerMachines
-  VERSION = '0.7.0'
+  VERSION = '0.9.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: breaker_machines
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.9.2
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -52,47 +52,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: state_machines
+  name: rb_sys
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.100.4
+        version: '0.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.100.4
+        version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: zeitwerk
+  name: state_machines
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: 0.100.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: 0.100.4
 - !ruby/object:Gem::Dependency
-  name: rb_sys
+  name: zeitwerk
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.7'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -165,51 +165,6 @@ files:
 - ext/breaker_machines_native/ffi/Cargo.toml
 - ext/breaker_machines_native/ffi/extconf.rb
 - ext/breaker_machines_native/ffi/src/lib.rs
-- ext/breaker_machines_native/target/debug/build/clang-sys-d961dfabd5f43fba/out/common.rs
-- ext/breaker_machines_native/target/debug/build/clang-sys-d961dfabd5f43fba/out/dynamic.rs
-- ext/breaker_machines_native/target/debug/build/clang-sys-d961dfabd5f43fba/out/macros.rs
-- ext/breaker_machines_native/target/debug/build/rb-sys-2bb7281aac8faec8/out/bindings-0.9.117-mri-arm64-darwin24-3.4.7.rs
-- ext/breaker_machines_native/target/debug/build/rb-sys-54cb99ea6aeab8bc/out/bindings-0.9.117-mri-arm64-darwin24-3.4.7.rs
-- ext/breaker_machines_native/target/debug/build/rb-sys-9e64a270c6421e93/out/bindings-0.9.117-mri-arm64-darwin24-3.4.7.rs
-- ext/breaker_machines_native/target/debug/build/rb-sys-e627030114d3fc19/out/bindings-0.9.117-mri-arm64-darwin24-3.4.7.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/Cargo.toml
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/examples/basic.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/builder.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/callbacks.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/circuit.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/errors.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/lib.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.1.0/src/storage.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/Cargo.toml
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/examples/basic.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/builder.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/callbacks.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/circuit.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/errors.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/lib.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.2.0/src/storage.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/Cargo.toml
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/examples/basic.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/builder.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/callbacks.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/circuit.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/errors.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/lib.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.5.0/src/storage.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/Cargo.toml
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/examples/basic.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/builder.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/bulkhead.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/callbacks.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/circuit.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/classifier.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/errors.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/lib.rs
-- ext/breaker_machines_native/target/package/breaker-machines-0.6.0/src/storage.rs
-- ext/breaker_machines_native/target/release/build/clang-sys-ef8ad8b846ac8b75/out/common.rs
-- ext/breaker_machines_native/target/release/build/clang-sys-ef8ad8b846ac8b75/out/dynamic.rs
-- ext/breaker_machines_native/target/release/build/clang-sys-ef8ad8b846ac8b75/out/macros.rs
-- ext/breaker_machines_native/target/release/build/rb-sys-064bf9961dd17810/out/bindings-0.9.117-mri-arm64-darwin24-3.4.7.rs
 - lib/breaker_machines.rb
 - lib/breaker_machines/async_circuit.rb
 - lib/breaker_machines/async_support.rb
@@ -266,7 +221,6 @@ homepage: https://github.com/seuros/breaker_machines
 licenses:
 - MIT
 metadata:
-  allowed_push_host: https://rubygems.org
   homepage_uri: https://github.com/seuros/breaker_machines
   source_code_uri: https://github.com/seuros/breaker_machines
   bug_tracker_uri: https://github.com/seuros/breaker_machines/issues

data/ext/breaker_machines_native/target/debug/build/clang-sys-d961dfabd5f43fba/out/common.rs

@@ -1,355 +0,0 @@
-// SPDX-License-Identifier: Apache-2.0
-
-use std::cell::RefCell;
-use std::collections::HashMap;
-use std::env;
-use std::path::{Path, PathBuf};
-use std::process::Command;
-
-use glob::{MatchOptions, Pattern};
-
-//================================================
-// Commands
-//================================================
-
-thread_local! {
-    /// The errors encountered by the build script while executing commands.
-    static COMMAND_ERRORS: RefCell<HashMap<String, Vec<String>>> = RefCell::default();
-}
-
-/// Adds an error encountered by the build script while executing a command.
-fn add_command_error(name: &str, path: &str, arguments: &[&str], message: String) {
-    COMMAND_ERRORS.with(|e| {
-        e.borrow_mut()
-            .entry(name.into())
-            .or_default()
-            .push(format!(
-                "couldn't execute `{} {}` (path={}) ({})",
-                name,
-                arguments.join(" "),
-                path,
-                message,
-            ))
-    });
-}
-
-/// A struct that prints the errors encountered by the build script while
-/// executing commands when dropped (unless explictly discarded).
-///
-/// This is handy because we only want to print these errors when the build
-/// script fails to link to an instance of `libclang`. For example, if
-/// `llvm-config` couldn't be executed but an instance of `libclang` was found
-/// anyway we don't want to pollute the build output with irrelevant errors.
-#[derive(Default)]
-pub struct CommandErrorPrinter {
-    discard: bool,
-}
-
-impl CommandErrorPrinter {
-    pub fn discard(mut self) {
-        self.discard = true;
-    }
-}
-
-impl Drop for CommandErrorPrinter {
-    fn drop(&mut self) {
-        if self.discard {
-            return;
-        }
-
-        let errors = COMMAND_ERRORS.with(|e| e.borrow().clone());
-
-        if let Some(errors) = errors.get("llvm-config") {
-            println!(
-                "cargo:warning=could not execute `llvm-config` one or more \
-                times, if the LLVM_CONFIG_PATH environment variable is set to \
-                a full path to valid `llvm-config` executable it will be used \
-                to try to find an instance of `libclang` on your system: {}",
-                errors
-                    .iter()
-                    .map(|e| format!("\"{}\"", e))
-                    .collect::<Vec<_>>()
-                    .join("\n  "),
-            )
-        }
-
-        if let Some(errors) = errors.get("xcode-select") {
-            println!(
-                "cargo:warning=could not execute `xcode-select` one or more \
-                times, if a valid instance of this executable is on your PATH \
-                it will be used to try to find an instance of `libclang` on \
-                your system: {}",
-                errors
-                    .iter()
-                    .map(|e| format!("\"{}\"", e))
-                    .collect::<Vec<_>>()
-                    .join("\n  "),
-            )
-        }
-    }
-}
-
-#[cfg(test)]
-lazy_static::lazy_static! {
-    pub static ref RUN_COMMAND_MOCK: std::sync::Mutex<
-        Option<Box<dyn Fn(&str, &str, &[&str]) -> Option<String> + Send + Sync + 'static>>,
-    > = std::sync::Mutex::new(None);
-}
-
-/// Executes a command and returns the `stdout` output if the command was
-/// successfully executed (errors are added to `COMMAND_ERRORS`).
-fn run_command(name: &str, path: &str, arguments: &[&str]) -> Option<String> {
-    #[cfg(test)]
-    if let Some(command) = &*RUN_COMMAND_MOCK.lock().unwrap() {
-        return command(name, path, arguments);
-    }
-
-    let output = match Command::new(path).args(arguments).output() {
-        Ok(output) => output,
-        Err(error) => {
-            let message = format!("error: {}", error);
-            add_command_error(name, path, arguments, message);
-            return None;
-        }
-    };
-
-    if output.status.success() {
-        Some(String::from_utf8_lossy(&output.stdout).into_owned())
-    } else {
-        let message = format!("exit code: {}", output.status);
-        add_command_error(name, path, arguments, message);
-        None
-    }
-}
-
-/// Executes the `llvm-config` command and returns the `stdout` output if the
-/// command was successfully executed (errors are added to `COMMAND_ERRORS`).
-pub fn run_llvm_config(arguments: &[&str]) -> Option<String> {
-    let path = env::var("LLVM_CONFIG_PATH").unwrap_or_else(|_| "llvm-config".into());
-    run_command("llvm-config", &path, arguments)
-}
-
-/// Executes the `xcode-select` command and returns the `stdout` output if the
-/// command was successfully executed (errors are added to `COMMAND_ERRORS`).
-pub fn run_xcode_select(arguments: &[&str]) -> Option<String> {
-    run_command("xcode-select", "xcode-select", arguments)
-}
-
-//================================================
-// Search Directories
-//================================================
-// These search directories are listed in order of
-// preference, so if multiple `libclang` instances
-// are found when searching matching directories,
-// the `libclang` instances from earlier
-// directories will be preferred (though version
-// takes precedence over location).
-//================================================
-
-/// `libclang` directory patterns for Haiku.
-const DIRECTORIES_HAIKU: &[&str] = &[
-    "/boot/home/config/non-packaged/develop/lib",
-    "/boot/home/config/non-packaged/lib",
-    "/boot/system/non-packaged/develop/lib",
-    "/boot/system/non-packaged/lib",
-    "/boot/system/develop/lib",
-    "/boot/system/lib",
-];
-
-/// `libclang` directory patterns for Linux (and FreeBSD).
-const DIRECTORIES_LINUX: &[&str] = &[
-    "/usr/local/llvm*/lib*",
-    "/usr/local/lib*/*/*",
-    "/usr/local/lib*/*",
-    "/usr/local/lib*",
-    "/usr/lib*/*/*",
-    "/usr/lib*/*",
-    "/usr/lib*",
-];
-
-/// `libclang` directory patterns for macOS.
-const DIRECTORIES_MACOS: &[&str] = &[
-    "/usr/local/opt/llvm*/lib/llvm*/lib",
-    "/Library/Developer/CommandLineTools/usr/lib",
-    "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib",
-    "/usr/local/opt/llvm*/lib",
-];
-
-/// `libclang` directory patterns for Windows.
-///
-/// The boolean indicates whether the directory pattern should be used when
-/// compiling for an MSVC target environment.
-const DIRECTORIES_WINDOWS: &[(&str, bool)] = &[
-    // LLVM + Clang can be installed using Scoop (https://scoop.sh).
-    // Other Windows package managers install LLVM + Clang to other listed
-    // system-wide directories.
-    ("C:\\Users\\*\\scoop\\apps\\llvm\\current\\lib", true),
-    ("C:\\MSYS*\\MinGW*\\lib", false),
-    ("C:\\Program Files*\\LLVM\\lib", true),
-    ("C:\\LLVM\\lib", true),
-    // LLVM + Clang can be installed as a component of Visual Studio.
-    // https://github.com/KyleMayes/clang-sys/issues/121
-    ("C:\\Program Files*\\Microsoft Visual Studio\\*\\VC\\Tools\\Llvm\\**\\lib", true),
-];
-
-/// `libclang` directory patterns for illumos
-const DIRECTORIES_ILLUMOS: &[&str] = &[
-    "/opt/ooce/llvm-*/lib",
-    "/opt/ooce/clang-*/lib",
-];
-
-//================================================
-// Searching
-//================================================
-
-/// Finds the files in a directory that match one or more filename glob patterns
-/// and returns the paths to and filenames of those files.
-fn search_directory(directory: &Path, filenames: &[String]) -> Vec<(PathBuf, String)> {
-    // Escape the specified directory in case it contains characters that have
-    // special meaning in glob patterns (e.g., `[` or `]`).
-    let directory = Pattern::escape(directory.to_str().unwrap());
-    let directory = Path::new(&directory);
-
-    // Join the escaped directory to the filename glob patterns to obtain
-    // complete glob patterns for the files being searched for.
-    let paths = filenames
-        .iter()
-        .map(|f| directory.join(f).to_str().unwrap().to_owned());
-
-    // Prevent wildcards from matching path separators to ensure that the search
-    // is limited to the specified directory.
-    let mut options = MatchOptions::new();
-    options.require_literal_separator = true;
-
-    paths
-        .map(|p| glob::glob_with(&p, options))
-        .filter_map(Result::ok)
-        .flatten()
-        .filter_map(|p| {
-            let path = p.ok()?;
-            let filename = path.file_name()?.to_str().unwrap();
-
-            // The `libclang_shared` library has been renamed to `libclang-cpp`
-            // in Clang 10. This can cause instances of this library (e.g.,
-            // `libclang-cpp.so.10`) to be matched by patterns looking for
-            // instances of `libclang`.
-            if filename.contains("-cpp.") {
-                return None;
-            }
-
-            Some((path.parent().unwrap().to_owned(), filename.into()))
-        })
-        .collect::<Vec<_>>()
-}
-
-/// Finds the files in a directory (and any relevant sibling directories) that
-/// match one or more filename glob patterns and returns the paths to and
-/// filenames of those files.
-fn search_directories(directory: &Path, filenames: &[String]) -> Vec<(PathBuf, String)> {
-    let mut results = search_directory(directory, filenames);
-
-    // On Windows, `libclang.dll` is usually found in the LLVM `bin` directory
-    // while `libclang.lib` is usually found in the LLVM `lib` directory. To
-    // keep things consistent with other platforms, only LLVM `lib` directories
-    // are included in the backup search directory globs so we need to search
-    // the LLVM `bin` directory here.
-    if target_os!("windows") && directory.ends_with("lib") {
-        let sibling = directory.parent().unwrap().join("bin");
-        results.extend(search_directory(&sibling, filenames));
-    }
-
-    results
-}
-
-/// Finds the `libclang` static or dynamic libraries matching one or more
-/// filename glob patterns and returns the paths to and filenames of those files.
-pub fn search_libclang_directories(filenames: &[String], variable: &str) -> Vec<(PathBuf, String)> {
-    // Search only the path indicated by the relevant environment variable
-    // (e.g., `LIBCLANG_PATH`) if it is set.
-    if let Ok(path) = env::var(variable).map(|d| Path::new(&d).to_path_buf()) {
-        // Check if the path is a matching file.
-        if let Some(parent) = path.parent() {
-            let filename = path.file_name().unwrap().to_str().unwrap();
-            let libraries = search_directories(parent, filenames);
-            if libraries.iter().any(|(_, f)| f == filename) {
-                return vec![(parent.into(), filename.into())];
-            }
-        }
-
-        // Check if the path is directory containing a matching file.
-        return search_directories(&path, filenames);
-    }
-
-    let mut found = vec![];
-
-    // Search the `bin` and `lib` directories in the directory returned by
-    // `llvm-config --prefix`.
-    if let Some(output) = run_llvm_config(&["--prefix"]) {
-        let directory = Path::new(output.lines().next().unwrap()).to_path_buf();
-        found.extend(search_directories(&directory.join("bin"), filenames));
-        found.extend(search_directories(&directory.join("lib"), filenames));
-        found.extend(search_directories(&directory.join("lib64"), filenames));
-    }
-
-    // Search the toolchain directory in the directory returned by
-    // `xcode-select --print-path`.
-    if target_os!("macos") {
-        if let Some(output) = run_xcode_select(&["--print-path"]) {
-            let directory = Path::new(output.lines().next().unwrap()).to_path_buf();
-            let directory = directory.join("Toolchains/XcodeDefault.xctoolchain/usr/lib");
-            found.extend(search_directories(&directory, filenames));
-        }
-    }
-
-    // Search the directories in the `LD_LIBRARY_PATH` environment variable.
-    if let Ok(path) = env::var("LD_LIBRARY_PATH") {
-        for directory in env::split_paths(&path) {
-            found.extend(search_directories(&directory, filenames));
-        }
-    }
-
-    // Determine the `libclang` directory patterns.
-    let directories: Vec<&str> = if target_os!("haiku") {
-        DIRECTORIES_HAIKU.into()
-    } else if target_os!("linux") || target_os!("freebsd") {
-        DIRECTORIES_LINUX.into()
-    } else if target_os!("macos") {
-        DIRECTORIES_MACOS.into()
-    } else if target_os!("windows") {
-        let msvc = target_env!("msvc");
-        DIRECTORIES_WINDOWS
-            .iter()
-            .filter(|d| d.1 || !msvc)
-            .map(|d| d.0)
-            .collect()
-    } else if target_os!("illumos") {
-        DIRECTORIES_ILLUMOS.into()
-    } else {
-        vec![]
-    };
-
-    // We use temporary directories when testing the build script so we'll
-    // remove the prefixes that make the directories absolute.
-    let directories = if test!() {
-        directories
-            .iter()
-            .map(|d| d.strip_prefix('/').or_else(|| d.strip_prefix("C:\\")).unwrap_or(d))
-            .collect::<Vec<_>>()
-    } else {
-        directories
-    };
-
-    // Search the directories provided by the `libclang` directory patterns.
-    let mut options = MatchOptions::new();
-    options.case_sensitive = false;
-    options.require_literal_separator = true;
-    for directory in directories.iter() {
-        if let Ok(directories) = glob::glob_with(directory, options) {
-            for directory in directories.filter_map(Result::ok).filter(|p| p.is_dir()) {
-                found.extend(search_directories(&directory, filenames));
-            }
-        }
-    }
-
-    found
-}