Skip to content

Commit

Permalink
Merge pull request Homebrew#198835 from Homebrew/zeek-no-gcc5-update-…
Browse files Browse the repository at this point in the history
…test

zeek: remove unsupported gcc reference, update test
  • Loading branch information
chenrui333 authored Nov 24, 2024
2 parents 5f2a2ee + 70ba409 commit e03f2ab
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions Formula/z/zeek.rb
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ class Zeek < Formula
uses_from_macos "libxcrypt"
uses_from_macos "zlib"

fails_with gcc: "5"

def install
# Remove SDK paths from zeek-config. This breaks usage with other SDKs.
# https://github.com/Homebrew/homebrew-core/pull/74932
Expand Down Expand Up @@ -70,10 +68,10 @@ def install
assert_match "version #{version}", shell_output("#{bin}/zeek --version")
assert_match "ARP packet analyzer", shell_output("#{bin}/zeek --print-plugins")
system bin/"zeek", "-C", "-r", test_fixtures("test.pcap")
assert_predicate testpath/"conn.log", :exist?
refute_predicate testpath/"conn.log", :empty?
assert_predicate testpath/"http.log", :exist?
refute_predicate testpath/"http.log", :empty?
assert_path_exists testpath/"conn.log"
refute_empty (testpath/"conn.log").read
assert_path_exists testpath/"http.log"
refute_empty (testpath/"http.log").read
# For bottling MacOS SDK paths must not be part of the public include directories, see zeek/zeek#1468.
refute_includes shell_output("#{bin}/zeek-config --include_dir").chomp, "MacOSX"
end
Expand Down

0 comments on commit e03f2ab

Please sign in to comment.