From 590f77b111f5a7d352755bc0f531abbb55996793 Mon Sep 17 00:00:00 2001
From: Maxim Belkin <maxim.belkin@gmail.com>
Date: Tue, 21 Jul 2020 19:56:53 +0000
Subject: [PATCH] Unexpected linkage

---
 Library/Homebrew/linkage_checker.rb | 29 ++++++++++++++++++++++++++---
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/Library/Homebrew/linkage_checker.rb b/Library/Homebrew/linkage_checker.rb
index c97a691f10..b015b9be46 100644
--- a/Library/Homebrew/linkage_checker.rb
+++ b/Library/Homebrew/linkage_checker.rb
@@ -16,6 +16,7 @@ class LinkageChecker
     @system_dylibs    = Set.new
     @broken_dylibs    = Set.new
     @unexpected_broken_dylibs = nil
+    @unexpected_present_dylibs = nil
     @variable_dylibs  = Set.new
     @brewed_dylibs    = Hash.new { |h, k| h[k] = Set.new }
     @reverse_links    = Hash.new { |h, k| h[k] = Set.new }
@@ -57,6 +58,7 @@ class LinkageChecker
 
   def display_test_output(puts_output: true)
     display_items "Missing libraries", broken_dylibs_with_expectations, puts_output: puts_output
+    display_items "Unused missing linkage information", unexpected_present_dylibs, puts_output: puts_output
     display_items "Broken dependencies", @broken_deps, puts_output: puts_output
     display_items "Unwanted system libraries", @unwanted_system_dylibs, puts_output: puts_output
     display_items "Conflicting libraries", @version_conflict_deps, puts_output: puts_output
@@ -66,13 +68,15 @@ class LinkageChecker
     elsif unexpected_broken_dylibs.empty?
       puts "No unexpected broken library linkage detected."
     else
-      puts "Broken library linkage detected"
+      puts "Unexpected missing library linkage detected"
     end
+
+    puts "Unexpected non-missing linkage detected" if unexpected_present_dylibs.present?
   end
 
   def broken_library_linkage?
     issues = [@broken_deps, @unwanted_system_dylibs, @version_conflict_deps]
-    [issues, unexpected_broken_dylibs].flatten.any?(&:present?)
+    [issues, unexpected_broken_dylibs, unexpected_present_dylibs].flatten.any?(&:present?)
   end
 
   def unexpected_broken_dylibs
@@ -90,6 +94,21 @@ class LinkageChecker
     end
   end
 
+  def unexpected_present_dylibs
+    return @unexpected_present_dylibs if @unexpected_present_dylibs
+
+    @unexpected_present_dylibs = @formula.class.allowed_missing_libraries.reject do |allowed_missing_lib|
+      @broken_dylibs.any? do |broken_lib|
+        case allowed_missing_lib
+        when Regexp
+          allowed_missing_lib.match? broken_lib
+        when String
+          broken_lib.include? allowed_missing_lib
+        end
+      end
+    end
+  end
+
   def broken_dylibs_with_expectations
     output = {}
     @broken_dylibs.each do |broken_lib|
@@ -291,7 +310,11 @@ class LinkageChecker
       end
     else
       things.sort.each do |item|
-        output += "\n  #{item}"
+        output += if item.is_a? Regexp
+          "\n #{item.inspect}"
+        else
+          "\n  #{item}"
+        end
       end
     end
     puts output if puts_output