Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

sysdig 0.1.72 (new formula) #28116

Closed
wants to merge 4 commits into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions Library/Formula/sysdig.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
require "formula"

class Sysdig < Formula
homepage "http://www.sysdig.org/"
url 'https://github.com/draios/sysdig/archive/0.1.73.tar.gz'
sha1 '2f1133da0256c21c2642dbda57e047a3ae69765f'

head 'https://github.com/draios/sysdig.git', :branch => 'master'

depends_on "cmake" => :build

def install
ENV.libcxx if MacOS.version < :mavericks

mkdir "build" do
system "cmake", "..", *std_cmake_args
system "make", "install"
end
end

test do
system "#{bin}/sysdig", "-h"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this test be modified to do something more substantial then e.g. --version or --help? See cmake.rb for an example of a formula with a good test. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Throwing a few ideas for some more tests:

  • running sysdig -cl and make sure that the LUA chisels are listed (since embedding LUAjit is probably the most critical part of the OSX build)
  • uploading a sysdig trace file somewhere (we can host it on download.draios.com if you want or just send a pull request and I'll put it in our repo) and run a simple chisel and test that the output is consistent with what you expect

end
end